diff options
| author | primiano <primiano@chromium.org> | 2016-03-15 12:47:28 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-03-15 19:50:07 +0000 |
| commit | d19a104896da1904ac7b013a5cd6b4ffc205098d (patch) | |
| tree | 87a96a37eabf10ba67a8b6dd6c07fdf65dad6de9 /base/allocator | |
| parent | cc9b0c5fa36368bb012b4b4d572ada52ceb1146b (diff) | |
| download | chromium_src-d19a104896da1904ac7b013a5cd6b4ffc205098d.zip chromium_src-d19a104896da1904ac7b013a5cd6b4ffc205098d.tar.gz chromium_src-d19a104896da1904ac7b013a5cd6b4ffc205098d.tar.bz2 | |
Make the allocator_features gyp target target-only
allocator_features uses buildflag_header to generate a header at
build time. In host/target builds, both the #host and #target
targets have a ninja rule for the same path, which is bad.
This CL makes allocator_features follow the same pattern of
base_debugging_flags (target-only, everything else explicitly refers
to the #target veriant)
I verified that the problem reproduces without this patch
by doing:
$ build/gyp_chromium -DOS=android -Duse_experimental_allocator_shim=1
$ ninja -w dupbuild=err -n -C out_android/Release/ all
ninja: warning: multiple rules generate gen/base/allocator/features.h. builds involving this target will not be correct;
And this CL fixes it.
BUG=593695
Review URL: https://codereview.chromium.org/1794943006
Cr-Commit-Position: refs/heads/master@{#381289}
Diffstat (limited to 'base/allocator')
| -rw-r--r-- | base/allocator/allocator.gyp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/base/allocator/allocator.gyp b/base/allocator/allocator.gyp index adc78fd..d906eea 100644 --- a/base/allocator/allocator.gyp +++ b/base/allocator/allocator.gyp @@ -380,9 +380,10 @@ }, # 'allocator' target. { # GN: //base/allocator:features + # When referenced from a target that might be compiled in the host + # toolchain, always refer to 'allocator_features#target'. 'target_name': 'allocator_features', 'includes': [ '../../build/buildflag_header.gypi' ], - 'toolsets': ['host', 'target'], 'variables': { 'buildflag_header_path': 'base/allocator/features.h', 'buildflag_flags': [ |