diff --git a/java/aar.go b/java/aar.go index 28e388aaf..878704dda 100644 --- a/java/aar.go +++ b/java/aar.go @@ -506,11 +506,8 @@ func (a *AndroidLibrary) GenerateAndroidBuildActions(ctx android.ModuleContext) func AndroidLibraryFactory() android.Module { module := &AndroidLibrary{} + module.Module.addHostAndDeviceProperties() module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, &module.aaptProperties, &module.androidLibraryProperties) diff --git a/java/app.go b/java/app.go index 0fdede13a..ab7ec578d 100755 --- a/java/app.go +++ b/java/app.go @@ -963,11 +963,8 @@ func AndroidAppFactory() android.Module { module.Module.properties.Instrument = true module.Module.properties.Installable = proptools.BoolPtr(true) + module.addHostAndDeviceProperties() module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, &module.aaptProperties, &module.appProperties, &module.overridableAppProperties, @@ -1083,11 +1080,8 @@ func AndroidTestFactory() android.Module { module.appProperties.AlwaysPackageNativeLibs = true module.Module.dexpreopter.isTest = true + module.addHostAndDeviceProperties() module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, &module.aaptProperties, &module.appProperties, &module.appTestProperties, @@ -1135,11 +1129,8 @@ func AndroidTestHelperAppFactory() android.Module { module.appProperties.AlwaysPackageNativeLibs = true module.Module.dexpreopter.isTest = true + module.addHostAndDeviceProperties() module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, &module.aaptProperties, &module.appProperties, &module.appTestHelperAppProperties, diff --git a/java/java.go b/java/java.go index 13f9f2fbf..b343584a7 100644 --- a/java/java.go +++ b/java/java.go @@ -481,6 +481,21 @@ type Module struct { modulePaths []string } +func (j *Module) addHostProperties() { + j.AddProperties( + &j.properties, + &j.protoProperties, + ) +} + +func (j *Module) addHostAndDeviceProperties() { + j.addHostProperties() + j.AddProperties( + &j.deviceProperties, + &j.dexpreoptProperties, + ) +} + func (j *Module) OutputFiles(tag string) (android.Paths, error) { switch tag { case "": @@ -2044,12 +2059,8 @@ var javaHeaderLibsSdkMemberType android.SdkMemberType = &librarySdkMemberType{ func LibraryFactory() android.Module { module := &Library{} - module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, - &module.libraryProperties) + module.addHostAndDeviceProperties() + module.AddProperties(&module.libraryProperties) module.initModuleAndImport(&module.ModuleBase) @@ -2071,9 +2082,7 @@ func LibraryStaticFactory() android.Module { func LibraryHostFactory() android.Module { module := &Library{} - module.AddProperties( - &module.Module.properties, - &module.Module.protoProperties) + module.addHostProperties() module.Module.properties.Installable = proptools.BoolPtr(true) @@ -2241,12 +2250,8 @@ func (p *testSdkMemberProperties) AddToPropertySet(ctx android.SdkMemberContext, func TestFactory() android.Module { module := &Test{} - module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, - &module.testProperties) + module.addHostAndDeviceProperties() + module.AddProperties(&module.testProperties) module.Module.properties.Installable = proptools.BoolPtr(true) module.Module.dexpreopter.isTest = true @@ -2259,12 +2264,8 @@ func TestFactory() android.Module { func TestHelperLibraryFactory() android.Module { module := &TestHelperLibrary{} - module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, - &module.testHelperLibraryProperties) + module.addHostAndDeviceProperties() + module.AddProperties(&module.testHelperLibraryProperties) module.Module.properties.Installable = proptools.BoolPtr(true) module.Module.dexpreopter.isTest = true @@ -2305,10 +2306,8 @@ func JavaTestImportFactory() android.Module { func TestHostFactory() android.Module { module := &Test{} - module.AddProperties( - &module.Module.properties, - &module.Module.protoProperties, - &module.testProperties) + module.addHostProperties() + module.AddProperties(&module.testProperties) module.Module.properties.Installable = proptools.BoolPtr(true) @@ -2392,12 +2391,8 @@ func (j *Binary) DepsMutator(ctx android.BottomUpMutatorContext) { func BinaryFactory() android.Module { module := &Binary{} - module.AddProperties( - &module.Module.properties, - &module.Module.deviceProperties, - &module.Module.dexpreoptProperties, - &module.Module.protoProperties, - &module.binaryProperties) + module.addHostAndDeviceProperties() + module.AddProperties(&module.binaryProperties) module.Module.properties.Installable = proptools.BoolPtr(true) @@ -2413,10 +2408,8 @@ func BinaryFactory() android.Module { func BinaryHostFactory() android.Module { module := &Binary{} - module.AddProperties( - &module.Module.properties, - &module.Module.protoProperties, - &module.binaryProperties) + module.addHostProperties() + module.AddProperties(&module.binaryProperties) module.Module.properties.Installable = proptools.BoolPtr(true) diff --git a/java/plugin.go b/java/plugin.go index a5e8292e7..947c28626 100644 --- a/java/plugin.go +++ b/java/plugin.go @@ -24,10 +24,8 @@ func init() { func PluginFactory() android.Module { module := &Plugin{} - module.AddProperties( - &module.Module.properties, - &module.Module.protoProperties, - &module.pluginProperties) + module.addHostProperties() + module.AddProperties(&module.pluginProperties) InitJavaModule(module, android.HostSupported) return module diff --git a/java/robolectric.go b/java/robolectric.go index 3195615ff..64bd32570 100644 --- a/java/robolectric.go +++ b/java/robolectric.go @@ -215,10 +215,9 @@ func (r *robolectricTest) writeTestRunner(w io.Writer, module, name string, test func RobolectricTestFactory() android.Module { module := &robolectricTest{} + module.addHostProperties() module.AddProperties( - &module.Module.properties, &module.Module.deviceProperties, - &module.Module.protoProperties, &module.robolectricProperties) module.Module.dexpreopter.isTest = true diff --git a/java/sdk_library.go b/java/sdk_library.go index 63ebe6352..83bc04eb8 100644 --- a/java/sdk_library.go +++ b/java/sdk_library.go @@ -1535,13 +1535,8 @@ func (module *SdkLibrary) CreateInternalModules(mctx android.DefaultableHookCont } func (module *SdkLibrary) InitSdkLibraryProperties() { - module.AddProperties( - &module.sdkLibraryProperties, - &module.properties, - &module.dexpreoptProperties, - &module.deviceProperties, - &module.protoProperties, - ) + module.addHostAndDeviceProperties() + module.AddProperties(&module.sdkLibraryProperties) module.initSdkLibraryComponent(&module.ModuleBase)