aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java29
1 files changed, 16 insertions, 13 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java
index e7325dcf6b..817e0a2bc9 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidToolsDefaultsJar.java
@@ -30,10 +30,10 @@ import java.util.regex.Pattern;
/**
* Implementation for the {@code android_tools_defaults_jar} rule.
*
- * <p>This rule is a sad, sad way to let people depend on {@code android.jar} when an
- * {@code android_sdk} rule is used. In an ideal world, people would say "depend on
- * the android_jar output group of $config.android_sdk", but, alas, neither depending on labels in
- * the configuration nor depending on a specified output group works.
+ * <p>This rule is a sad, sad way to let people depend on {@code android.jar} when an {@code
+ * android_sdk} rule is used. In an ideal world, people would say "depend on the android_jar output
+ * group of $config.android_sdk", but, alas, neither depending on labels in the configuration nor
+ * depending on a specified output group works.
*
* <p>So all this needs to be implemented manually. This rule is injected into the defaults package
* from {@link AndroidConfiguration.Options#getDefaultsRules()}.
@@ -47,20 +47,23 @@ public class AndroidToolsDefaultsJar implements RuleConfiguredTargetFactory {
throws InterruptedException, RuleErrorException {
if (!ruleContext.getLabel().getPackageName().equals("tools/defaults")) {
// Guard against extraordinarily inquisitive individuals.
- ruleContext.ruleError("The android_tools_defaults_jar rule should not be used in BUILD files."
- + " It is a rule internal to the build tool.");
+ ruleContext.ruleError(
+ "The android_tools_defaults_jar rule should not be used in BUILD files."
+ + " It is a rule internal to the build tool.");
return null;
}
TransitiveInfoCollection androidSdk = ruleContext.getPrerequisite(":android_sdk", Mode.TARGET);
- AndroidSdkProvider sdkProvider = androidSdk.getProvider(AndroidSdkProvider.class);
- Artifact androidJar = sdkProvider != null
- ? sdkProvider.getAndroidJar()
- : findAndroidJar(androidSdk.getProvider(FileProvider.class).getFilesToBuild());
+ AndroidSdkProvider sdkProvider = androidSdk.getProvider(AndroidSdkProvider.class);
+ Artifact androidJar =
+ sdkProvider != null
+ ? sdkProvider.getAndroidJar()
+ : findAndroidJar(androidSdk.getProvider(FileProvider.class).getFilesToBuild());
- NestedSet<Artifact> filesToBuild = androidJar == null
- ? NestedSetBuilder.<Artifact>emptySet(Order.STABLE_ORDER)
- : NestedSetBuilder.create(Order.STABLE_ORDER, androidJar);
+ NestedSet<Artifact> filesToBuild =
+ androidJar == null
+ ? NestedSetBuilder.<Artifact>emptySet(Order.STABLE_ORDER)
+ : NestedSetBuilder.create(Order.STABLE_ORDER, androidJar);
return new RuleConfiguredTargetBuilder(ruleContext)
.add(RunfilesProvider.class, RunfilesProvider.EMPTY)