Consolidate adding common java properties
Use a method to add the properties that are present on all java modules.
Bug: 153485543
Test: m checkbuild
Change-Id: I7803b15eb0de810c8ab8d4b9acf2511935a26fb6
Merged-In: I7803b15eb0de810c8ab8d4b9acf2511935a26fb6
(cherry picked from commit ce6734e666)
This commit is contained in:
Colin Cross
parent
8d6286befe
commit
1c14b4ecf6
6 changed files with 37 additions and 64 deletions
|
|
@ -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)
|
||||
|
||||
|
|
|
|||
15
java/app.go
15
java/app.go
|
|
@ -970,11 +970,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,
|
||||
|
|
@ -1093,11 +1090,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,
|
||||
|
|
@ -1145,11 +1139,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,
|
||||
|
|
|
|||
63
java/java.go
63
java/java.go
|
|
@ -482,6 +482,21 @@ type Module struct {
|
|||
distFile android.Path
|
||||
}
|
||||
|
||||
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 "":
|
||||
|
|
@ -2024,12 +2039,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)
|
||||
|
||||
|
|
@ -2051,9 +2062,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)
|
||||
|
||||
|
|
@ -2221,12 +2230,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
|
||||
|
|
@ -2239,12 +2244,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
|
||||
|
|
@ -2285,10 +2286,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)
|
||||
|
||||
|
|
@ -2372,12 +2371,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)
|
||||
|
||||
|
|
@ -2393,10 +2388,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)
|
||||
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -1553,13 +1553,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)
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue