diff options
| author |  Googler <noreply@google.com> | 2016-10-21 17:48:41 +0000 |
|---|---|---|
| committer |  John Cater <jcater@google.com> | 2016-10-24 19:27:15 +0000 |
| commit | 0ac7795efd5695a052b6ab0019945e6b4d9a5a99 (patch) | |
| tree | 5d1edb375bbd5ad35eae815cbe84161a85385ce1 /src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java | |
| parent | 4ed83bd49ab43b71ad020fe54ad26337a86498b9 (diff) | |
Don't mark modules as used if they would only contribute textual_hdrs. There is
not much to gain by providing textual_hdrs through modules and, dependent on the
setup, such modules can pull in large dependency trees (currently pruning of
header modules isn't activated for the modules builds themselves and so we
always add all dependent modules to the inputs).
--
MOS_MIGRATED_REVID=136849158
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java index c1bd2b1789..dc6dd1faff 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java @@ -1128,6 +1128,9 @@ public final class CcLibraryHelper { contextBuilder.addDeclaredIncludeSrcs(publicHeaders); contextBuilder.addDeclaredIncludeSrcs(publicTextualHeaders); contextBuilder.addDeclaredIncludeSrcs(privateHeaders); + contextBuilder.addModularHdrs(publicHeaders); + contextBuilder.addModularHdrs(privateHeaders); + contextBuilder.addTextualHdrs(publicTextualHeaders); contextBuilder.addPregreppedHeaderMap( CppHelper.createExtractInclusions(ruleContext, semantics, publicHeaders)); contextBuilder.addPregreppedHeaderMap( |