From 75f97c1194c82bdd59f0a8c2809e7bfa464b01d4 Mon Sep 17 00:00:00 2001 From: ajmichael Date: Wed, 30 Aug 2017 01:37:03 +0200 Subject: Make classJarOutput of AndroidResourceMergingAction read package from manifest if --packageForR is not specified. RELNOTES: None PiperOrigin-RevId: 166923102 --- .../devtools/build/android/AndroidResourceMergingAction.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/tools/android/java/com/google/devtools/build/android/AndroidResourceMergingAction.java b/src/tools/android/java/com/google/devtools/build/android/AndroidResourceMergingAction.java index 1fb31561ed..7beac4760a 100644 --- a/src/tools/android/java/com/google/devtools/build/android/AndroidResourceMergingAction.java +++ b/src/tools/android/java/com/google/devtools/build/android/AndroidResourceMergingAction.java @@ -13,6 +13,7 @@ // limitations under the License. package com.google.devtools.build.android; +import com.android.builder.core.VariantConfiguration; import com.android.builder.core.VariantType; import com.android.ide.common.internal.PngCruncher; import com.android.ide.common.internal.PngException; @@ -214,10 +215,15 @@ public class AndroidResourceMergingAction { logger.fine(String.format("Setup finished at %sms", timer.elapsed(TimeUnit.MILLISECONDS))); VariantType packageType = VariantType.LIBRARY; + String packageForR = options.packageForR; + if (packageForR == null) { + packageForR = + Strings.nullToEmpty( + VariantConfiguration.getManifestPackage(options.primaryManifest.toFile())); + } AndroidResourceClassWriter resourceClassWriter = - AndroidResourceClassWriter.createWith(aaptConfigOptions.androidJar, - generatedSources, - Strings.nullToEmpty(options.packageForR)); + AndroidResourceClassWriter.createWith( + aaptConfigOptions.androidJar, generatedSources, packageForR); resourceClassWriter.setIncludeClassFile(true); resourceClassWriter.setIncludeJavaFile(false); -- cgit v1.2.3