Revert "Make some common properties overridden when used in defaults modules"
Bug: 68337468
Breaks HOST_PREFER_32BIT=true
This reverts commit 617016a6f2819210894492ef6f095992bf28bcb4.
Change-Id: Ifb20f41cdf93dbfd73a3aac5b055e595e334350b
diff --git a/android/androidmk.go b/android/androidmk.go
index 2cae1c3..759d328 100644
--- a/android/androidmk.go
+++ b/android/androidmk.go
@@ -231,10 +231,10 @@
if len(amod.commonProperties.Init_rc) > 0 {
fmt.Fprintln(&data.preamble, "LOCAL_INIT_RC := ", strings.Join(amod.commonProperties.Init_rc, " "))
}
- if Bool(amod.commonProperties.Proprietary) {
+ if amod.commonProperties.Proprietary {
fmt.Fprintln(&data.preamble, "LOCAL_PROPRIETARY_MODULE := true")
}
- if Bool(amod.commonProperties.Vendor) {
+ if amod.commonProperties.Vendor {
fmt.Fprintln(&data.preamble, "LOCAL_VENDOR_MODULE := true")
}
if amod.commonProperties.Owner != nil {
diff --git a/android/arch.go b/android/arch.go
index f8317bb..eaa35a4 100644
--- a/android/arch.go
+++ b/android/arch.go
@@ -312,12 +312,12 @@
var multilib string
switch class {
case Device:
- multilib = String(module.base().commonProperties.Target.Android.Compile_multilib)
+ multilib = module.base().commonProperties.Target.Android.Compile_multilib
case Host, HostCross:
- multilib = String(module.base().commonProperties.Target.Host.Compile_multilib)
+ multilib = module.base().commonProperties.Target.Host.Compile_multilib
}
if multilib == "" {
- multilib = String(module.base().commonProperties.Compile_multilib)
+ multilib = module.base().commonProperties.Compile_multilib
}
if multilib == "" {
multilib = module.base().commonProperties.Default_multilib
diff --git a/android/module.go b/android/module.go
index b8c9ab2..1cdc2dd 100644
--- a/android/module.go
+++ b/android/module.go
@@ -165,27 +165,27 @@
// are "32" (compile for 32-bit only), "64" (compile for 64-bit only), "both" (compile for both
// architectures), or "first" (compile for 64-bit on a 64-bit platform, and 32-bit on a 32-bit
// platform
- Compile_multilib *string `android:"arch_variant"`
+ Compile_multilib string `android:"arch_variant"`
Target struct {
Host struct {
- Compile_multilib *string
+ Compile_multilib string
}
Android struct {
- Compile_multilib *string
+ Compile_multilib string
}
}
Default_multilib string `blueprint:"mutated"`
// whether this is a proprietary vendor module, and should be installed into /vendor
- Proprietary *bool
+ Proprietary bool
// vendor who owns this module
Owner *string
// whether this module is device specific and should be installed into /vendor
- Vendor *bool
+ Vendor bool
// *.logtags files, to combine together in order to generate the /system/etc/event-log-tags
// file
@@ -519,7 +519,7 @@
return androidBaseContextImpl{
target: a.commonProperties.CompileTarget,
targetPrimary: a.commonProperties.CompilePrimary,
- vendor: Bool(a.commonProperties.Proprietary) || Bool(a.commonProperties.Vendor),
+ vendor: a.commonProperties.Proprietary || a.commonProperties.Vendor,
config: ctx.Config().(Config),
}
}