From e3c22f7a3f30e945da8c839a898e053c6b092b78 Mon Sep 17 00:00:00 2001 From: Sam Saccone Date: Wed, 13 Nov 2019 15:23:05 +0000 Subject: [PATCH] Revert submission Reason for revert: b/144387414 Fixes: b/144387414 Change-Id: I21e15a4d149a5a8fefd0f84010600cb41b59db95 --- apex/apex.go | 4 +--- apex/apex_test.go | 12 +----------- java/androidmk.go | 5 ----- java/app.go | 11 ++++------- 4 files changed, 6 insertions(+), 26 deletions(-) diff --git a/apex/apex.go b/apex/apex.go index 1216d3334..477f917e6 100644 --- a/apex/apex.go +++ b/apex/apex.go @@ -1216,7 +1216,7 @@ func (a *apexBundle) GenerateAndroidBuildActions(ctx android.ModuleContext) { if am, ok := child.(android.ApexModule); ok { // We cannot use a switch statement on `depTag` here as the checked // tags used below are private (e.g. `cc.sharedDepTag`). - if cc.IsSharedDepTag(depTag) || cc.IsRuntimeDepTag(depTag) { + if cc.IsSharedDepTag(depTag) || cc.IsRuntimeDepTag(depTag) || java.IsJniDepTag(depTag) { if cc, ok := child.(*cc.Module); ok { if android.InList(cc.Name(), providedNativeSharedLibs) { // If we're using a shared library which is provided from other APEX, @@ -1253,8 +1253,6 @@ func (a *apexBundle) GenerateAndroidBuildActions(ctx android.ModuleContext) { filesInfo = append(filesInfo, apexFile{fileToCopy, moduleName, dirInApex, nativeTest, cc, nil}) return true } - } else if java.IsJniDepTag(depTag) { - // Do nothing for JNI dep. JNI libraries are always embedded in APK-in-APEX. } else if am.CanHaveApexVariants() && am.IsInstallableToApex() { ctx.ModuleErrorf("unexpected tag %q for indirect dependency %q", depTag, depName) } diff --git a/apex/apex_test.go b/apex/apex_test.go index 330dc3cdd..614164deb 100644 --- a/apex/apex_test.go +++ b/apex/apex_test.go @@ -2538,17 +2538,7 @@ func TestApexWithApps(t *testing.T) { ensureContains(t, copyCmds, "image.apex/app/AppFoo/AppFoo.apk") ensureContains(t, copyCmds, "image.apex/priv-app/AppFooPriv/AppFooPriv.apk") - - // JNI libraries are embedded inside APK - appZipRule := ctx.ModuleForTests("AppFoo", "android_common_myapex").Rule("zip") - libjniOutput := ctx.ModuleForTests("libjni", "android_arm64_armv8-a_core_shared_myapex").Module().(*cc.Module).OutputFile() - ensureListContains(t, appZipRule.Implicits.Strings(), libjniOutput.String()) - // ... uncompressed - if args := appZipRule.Args["jarArgs"]; !strings.Contains(args, "-L 0") { - t.Errorf("jni lib is not uncompressed for AppFoo") - } - // ... and not directly inside the APEX - ensureNotContains(t, copyCmds, "image.apex/lib64/libjni.so") + ensureContains(t, copyCmds, "image.apex/lib64/libjni.so") } func TestApexWithAppImports(t *testing.T) { diff --git a/java/androidmk.go b/java/androidmk.go index c97373926..05106806a 100644 --- a/java/androidmk.go +++ b/java/androidmk.go @@ -262,11 +262,6 @@ func (binary *Binary) AndroidMkEntries() android.AndroidMkEntries { } func (app *AndroidApp) AndroidMkEntries() android.AndroidMkEntries { - if !app.IsForPlatform() { - return android.AndroidMkEntries{ - Disabled: true, - } - } return android.AndroidMkEntries{ Class: "APPS", OutputFile: android.OptionalPathForPath(app.outputFile), diff --git a/java/app.go b/java/app.go index 6dd6d83c6..de12b73ea 100644 --- a/java/app.go +++ b/java/app.go @@ -78,9 +78,8 @@ type appProperties struct { // Store native libraries uncompressed in the APK and set the android:extractNativeLibs="false" manifest // flag so that they are used from inside the APK at runtime. Defaults to true for android_test modules unless - // sdk_version or min_sdk_version is set to a version that doesn't support it (<23), defaults to true for - // android_app modules that are embedded to APEXes, defaults to false for other module types where the native - // libraries are generally preinstalled outside the APK. + // sdk_version or min_sdk_version is set to a version that doesn't support it (<23), defaults to false for other + // module types where the native libraries are generally preinstalled outside the APK. Use_embedded_native_libs *bool // Store dex files uncompressed in the APK and set the android:useEmbeddedDex="true" manifest attribute so that @@ -217,8 +216,7 @@ func (a *AndroidApp) useEmbeddedNativeLibs(ctx android.ModuleContext) bool { ctx.PropertyErrorf("min_sdk_version", "invalid value %q: %s", a.minSdkVersion(), err) } - return (minSdkVersion >= 23 && Bool(a.appProperties.Use_embedded_native_libs)) || - !a.IsForPlatform() + return minSdkVersion >= 23 && Bool(a.appProperties.Use_embedded_native_libs) } // Returns whether this module should have the dex file stored uncompressed in the APK. @@ -242,7 +240,7 @@ func (a *AndroidApp) shouldUncompressDex(ctx android.ModuleContext) bool { func (a *AndroidApp) shouldEmbedJnis(ctx android.BaseModuleContext) bool { return ctx.Config().UnbundledBuild() || Bool(a.appProperties.Use_embedded_native_libs) || - !a.IsForPlatform() || a.appProperties.AlwaysPackageNativeLibs + a.appProperties.AlwaysPackageNativeLibs } func (a *AndroidApp) aaptBuildActions(ctx android.ModuleContext) { @@ -586,7 +584,6 @@ func AndroidAppFactory() android.Module { android.InitAndroidMultiTargetsArchModule(module, android.DeviceSupported, android.MultilibCommon) android.InitDefaultableModule(module) android.InitOverridableModule(module, &module.appProperties.Overrides) - android.InitApexModule(module) return module }