diff options
author | Mike Frysinger <vapier@chromium.org> | 2019-08-15 14:32:52 -0400 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-08-16 17:40:15 -0700 |
commit | ddb433c41731fa1dc251a8e6d3bcf170cad803c5 (patch) | |
tree | a4a93a8593ec90678213e31d2500deac09b17649 | |
parent | 4111c5852e8c56c0446fd02ab093a9d7264fad06 (diff) | |
download | platform_external_libbrillo-ddb433c41731fa1dc251a8e6d3bcf170cad803c5.tar.gz platform_external_libbrillo-ddb433c41731fa1dc251a8e6d3bcf170cad803c5.tar.bz2 platform_external_libbrillo-ddb433c41731fa1dc251a8e6d3bcf170cad803c5.zip |
common-mk: generate-pc: add a defines field
The linter doesn't like it when you put -D into cflags. We could
make the linter understand the AST for this specific case, but that
seems a bit error prone, so just extend the existing template to
handle defines in the same way as other build targets.
BUG=None
TEST=libbrillo pc still works
Change-Id: Ife556fa5d52d14108584e2c32a30eae887f52480
Reviewed-on: https://chromium-review.googlesource.com/1756106
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Cr-Mirrored-From: https://chromium.googlesource.com/chromiumos/platform2
Cr-Mirrored-Commit: 35c34057dfcc99d76c64b66d212a6ad8dff3f01d
-rw-r--r-- | BUILD.gn | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -248,7 +248,7 @@ generate_pkg_config("libbrillo_pc") { requires_private += sublib.all_dependent_pkg_deps } } - cflags = [ "-DUSE_RTTI_FOR_TYPE_TAGS" ] + defines = [ "USE_RTTI_FOR_TYPE_TAGS" ] libs = [ "-lbrillo-${libbase_ver}" ] } |