diff options
author | 2016-04-26 00:06:33 +0000 | |
---|---|---|
committer | 2016-04-26 08:29:51 +0000 | |
commit | 35e29e5c707522d7c469947f007d4107a3e57e68 (patch) | |
tree | 562f9a17a98254bdccc53a3c798a0594c2fc42d7 /src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java | |
parent | bc604ab3b055ebf9b6070f0019cd74556493a939 (diff) |
Refactor CompilationSupport.registerFullyLinkedAction to be the responsibility of the caller to invoke, instead of creating the fully linked artifact by default
It makes more sense for the caller rule to opt-in to creating its own implicit output.
This also makes it easier for classes to use CompilationSupport without generating this artifact.
--
MOS_MIGRATED_REVID=120764517
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java b/src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java index f9cb428ec1..c01fbda176 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/AbstractJ2ObjcProtoAspect.java @@ -144,7 +144,9 @@ public abstract class AbstractJ2ObjcProtoAspect implements ConfiguredNativeAspec j2ObjcSource.getHeaderSearchPaths(), DEPENDENT_ATTRIBUTES); - new CompilationSupport(ruleContext).registerCompileAndArchiveActions(common); + new CompilationSupport(ruleContext) + .registerCompileAndArchiveActions(common) + .registerFullyLinkAction(common.getObjcProvider()); } NestedSet<Artifact> j2ObjcTransitiveHeaderMappingFiles = j2ObjcTransitiveHeaderMappingFiles( |