Treat system_ext as system container

Bug: b/365135457
Test: m
Change-Id: Ic8fd60c0e1045f190b6608931ca1d65a784026f0
This commit is contained in:
Dennis Shen
2024-09-12 22:05:08 +00:00
parent d4771aa267
commit 01efb83420
3 changed files with 10 additions and 2 deletions

View File

@@ -88,6 +88,13 @@ func (module *DeclarationsModule) DepsMutator(ctx android.BottomUpMutatorContext
ctx.PropertyErrorf("container", "missing container property") ctx.PropertyErrorf("container", "missing container property")
} }
// treating system_ext as system partition as we are combining them as one container
// TODO remove this logic once we start enforcing that system_ext cannot be specified as
// container in the container field.
if module.properties.Container == "system_ext" {
module.properties.Container = "system"
}
// Add a dependency on the aconfig_value_sets defined in // Add a dependency on the aconfig_value_sets defined in
// RELEASE_ACONFIG_VALUE_SETS, and add any aconfig_values that // RELEASE_ACONFIG_VALUE_SETS, and add any aconfig_values that
// match our package. // match our package.

View File

@@ -260,7 +260,7 @@ func TestMkEntriesMatchedContainer(t *testing.T) {
aconfig_declarations { aconfig_declarations {
name: "my_aconfig_declarations_bar", name: "my_aconfig_declarations_bar",
package: "com.example.package.bar", package: "com.example.package.bar",
container: "system_ext", container: "vendor",
srcs: ["bar.aconfig"], srcs: ["bar.aconfig"],
} }

View File

@@ -233,7 +233,8 @@ func getAconfigFilePaths(m *ModuleBase, aconfigFiles map[string]Paths) (paths Pa
} else if m.ProductSpecific() { } else if m.ProductSpecific() {
container = "product" container = "product"
} else if m.SystemExtSpecific() { } else if m.SystemExtSpecific() {
container = "system_ext" // system_ext and system partitions should be treated as one container
container = "system"
} }
paths = append(paths, aconfigFiles[container]...) paths = append(paths, aconfigFiles[container]...)