diff options
author | 2015-06-26 19:05:59 +0000 | |
---|---|---|
committer | 2015-06-29 16:37:48 +0000 | |
commit | b50b8ebf6255be9cbf7e024cb45c3d08ee83180a (patch) | |
tree | 6a4ee6465a88d3e161f2ad33475bbdcd78e9d108 /src/main | |
parent | 89f7ff3a277c944e51fec54d19dc1873802b158f (diff) |
experimental_ios_test: Support data dependencies
This CL makes me so unhappy. As a follow up, I may rename BaseRule and/or RuleBase. Because seriously, you guys, naming.
--
MOS_MIGRATED_REVID=96990894
Diffstat (limited to 'src/main')
3 files changed, 18 insertions, 6 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ExperimentalIosTest.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ExperimentalIosTest.java index cad88fe005..069963c298 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/ExperimentalIosTest.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ExperimentalIosTest.java @@ -38,7 +38,8 @@ public final class ExperimentalIosTest extends IosTest { public ConfiguredTarget create(RuleContext ruleContext, ObjcCommon common, XcodeProvider xcodeProvider, NestedSet<Artifact> filesToBuild) throws InterruptedException { - Runfiles.Builder runfilesBuilder = new Runfiles.Builder(); + Runfiles.Builder runfilesBuilder = new Runfiles.Builder() + .addRunfiles(ruleContext, RunfilesProvider.DEFAULT_RUNFILES); NestedSetBuilder<Artifact> filesToBuildBuilder = NestedSetBuilder.<Artifact>stableOrder(); filesToBuildBuilder.addTransitive(filesToBuild); diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcProtoLibraryRule.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcProtoLibraryRule.java index c70a25e735..2163ae0d4a 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcProtoLibraryRule.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcProtoLibraryRule.java @@ -47,7 +47,7 @@ public class ObjcProtoLibraryRule implements RuleDefinition { The directly depended upon proto_library rules. ${SYNOPSIS} <!-- #END_BLAZE_RULE.ATTRIBUTE -->*/ - .add(attr("deps", LABEL_LIST) + .override(attr("deps", LABEL_LIST) .allowedRuleClasses("proto_library", "filegroup") .legacyAllowAnyFileType()) /* <!-- #BLAZE_RULE(objc_proto_library).ATTRIBUTE(options_file) --> @@ -85,7 +85,9 @@ public class ObjcProtoLibraryRule implements RuleDefinition { return RuleDefinition.Metadata.builder() .name("objc_proto_library") .factoryClass(ObjcProtoLibrary.class) - .ancestors(BaseRuleClasses.BaseRule.class, ObjcRuleClasses.ObjcProtoRule.class) + .ancestors( + BaseRuleClasses.RuleBase.class, + ObjcRuleClasses.ObjcProtoRule.class) .build(); } } diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcRuleClasses.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcRuleClasses.java index 217c6851c7..787decdbac 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcRuleClasses.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcRuleClasses.java @@ -28,6 +28,7 @@ import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableMap; import com.google.common.collect.ImmutableSet; import com.google.devtools.build.lib.actions.Artifact; +import com.google.devtools.build.lib.analysis.BaseRuleClasses; import com.google.devtools.build.lib.analysis.RuleConfiguredTarget.Mode; import com.google.devtools.build.lib.analysis.RuleConfiguredTargetBuilder; import com.google.devtools.build.lib.analysis.RuleContext; @@ -608,7 +609,7 @@ public class ObjcRuleClasses { The list of targets that are linked together to form the final bundle. ${SYNOPSIS} <!-- #END_BLAZE_RULE.ATTRIBUTE -->*/ - .add(attr("deps", LABEL_LIST) + .override(attr("deps", LABEL_LIST) .direct_compile_time_input() .allowedRuleClasses(ALLOWED_DEPS_RULE_CLASSES) .allowedFileTypes()) @@ -641,7 +642,11 @@ public class ObjcRuleClasses { return RuleDefinition.Metadata.builder() .name("$objc_compiling_rule") .type(RuleClassType.ABSTRACT) - .ancestors(CompileDependencyRule.class, OptionsRule.class, CoptsRule.class) + .ancestors( + BaseRuleClasses.RuleBase.class, + CompileDependencyRule.class, + OptionsRule.class, + CoptsRule.class) .build(); } } @@ -819,7 +824,11 @@ public class ObjcRuleClasses { .name("$ios_test_base_rule") .type(RuleClassType.ABSTRACT) .ancestors( - ReleaseBundlingRule.class, LinkingRule.class, XcodegenRule.class, SimulatorRule.class) + CompilingRule.class, + ReleaseBundlingRule.class, + LinkingRule.class, + XcodegenRule.class, + SimulatorRule.class) .build(); } } |