Merge "Automatically set compile_dex:true and hostdex:true for APEX variants" am: 6655756230
am: 8a7f0e94c1
am: 7ce676e2a3
Change-Id: I981f5fc7e5b044ceb61b8aed357a7724eacb0b91
This commit is contained in:
commit
cbc961dfa6
2 changed files with 11 additions and 7 deletions
|
@ -453,7 +453,6 @@ func TestBasicApex(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
static_libs: ["myotherjar"],
|
||||
libs: ["mysharedjar"],
|
||||
// TODO: remove //apex_available:platform
|
||||
|
@ -468,7 +467,6 @@ func TestBasicApex(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
// TODO: remove //apex_available:platform
|
||||
apex_available: [
|
||||
"//apex_available:platform",
|
||||
|
@ -481,7 +479,6 @@ func TestBasicApex(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
}
|
||||
`)
|
||||
|
||||
|
@ -595,7 +592,6 @@ func TestDefaults(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
apex_available: [ "myapex" ],
|
||||
}
|
||||
|
||||
|
@ -2997,7 +2993,6 @@ func TestErrorsIfDepsAreNotEnabled(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
enabled: false,
|
||||
}
|
||||
`)
|
||||
|
@ -3528,7 +3523,6 @@ func TestCompatConfig(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: true,
|
||||
apex_available: [ "myapex" ],
|
||||
}
|
||||
`)
|
||||
|
@ -3557,6 +3551,7 @@ func TestRejectNonInstallableJavaLibrary(t *testing.T) {
|
|||
srcs: ["foo/bar/MyClass.java"],
|
||||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
compile_dex: false,
|
||||
}
|
||||
`)
|
||||
}
|
||||
|
@ -3643,7 +3638,6 @@ func TestSymlinksFromApexToSystem(t *testing.T) {
|
|||
sdk_version: "none",
|
||||
system_modules: "none",
|
||||
libs: ["myotherjar"],
|
||||
compile_dex: true,
|
||||
apex_available: [
|
||||
"myapex",
|
||||
"//apex_available:platform",
|
||||
|
|
10
java/java.go
10
java/java.go
|
@ -1475,6 +1475,16 @@ func (j *Module) compile(ctx android.ModuleContext, aaptSrcJar android.Path) {
|
|||
|
||||
j.implementationAndResourcesJar = implementationAndResourcesJar
|
||||
|
||||
// Enable dex compilation for the APEX variants, unless it is disabled explicitly
|
||||
if android.DirectlyInAnyApex(ctx, ctx.ModuleName()) && !j.IsForPlatform() {
|
||||
if j.deviceProperties.Compile_dex == nil {
|
||||
j.deviceProperties.Compile_dex = proptools.BoolPtr(true)
|
||||
}
|
||||
if j.deviceProperties.Hostdex == nil {
|
||||
j.deviceProperties.Hostdex = proptools.BoolPtr(true)
|
||||
}
|
||||
}
|
||||
|
||||
if ctx.Device() && j.hasCode(ctx) &&
|
||||
(Bool(j.properties.Installable) || Bool(j.deviceProperties.Compile_dex)) {
|
||||
// Dex compilation
|
||||
|
|
Loading…
Reference in a new issue