Merge "Consolidate adding common java properties" (d7ffbb57) · Commits · e / os / android_build_soong

java/aar.go

+1 −4

Original line number	Diff line number	Diff line
		@@ -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)

java/app.go

+3 −12

Original line number	Diff line number	Diff line
		@@ -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,

java/java.go

+28 −35

Original line number	Diff line number	Diff line
		@@ -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 "":
		@@ -2051,12 +2066,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)

		@@ -2078,9 +2089,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)

		@@ -2248,12 +2257,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
		@@ -2266,12 +2271,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
		@@ -2312,10 +2313,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)

		@@ -2399,12 +2398,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)

		@@ -2420,10 +2415,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)

java/plugin.go

+2 −4

Original line number	Diff line number	Diff line
		@@ -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

java/robolectric.go

+1 −2

Original line number	Diff line number	Diff line
		@@ -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