Fix prebuilts of overridden apps
AndroidApp had its own HideFromMake method and flag that shadowed
the one in ModuleBase. This caused performOverrideMutator to set the
AndroidApp flag, but ModuleBase.skipInstall to read the ModuleBase
flag, resulting in a conflicting install rule being created. Remove
AndroidApp's HideFromMake in favor of the ModuleBase one.
Bug: 232788722
Test: TestOverrideAndroidAppWithPrebuilt
Change-Id: I8c0dfcb50ff4dc1e4d0574f150b10d79908f46aa
Merged-In: I8c0dfcb50ff4dc1e4d0574f150b10d79908f46aa
(cherry picked from commit aaa0c1ffcd
)
This commit is contained in:
@@ -99,7 +99,6 @@ type appProperties struct {
|
||||
|
||||
// cc.Coverage related properties
|
||||
PreventInstall bool `blueprint:"mutated"`
|
||||
HideFromMake bool `blueprint:"mutated"`
|
||||
IsCoverageVariant bool `blueprint:"mutated"`
|
||||
|
||||
// Whether this app is considered mainline updatable or not. When set to true, this will enforce
|
||||
@@ -889,10 +888,6 @@ func (a *AndroidApp) SetPreventInstall() {
|
||||
a.appProperties.PreventInstall = true
|
||||
}
|
||||
|
||||
func (a *AndroidApp) HideFromMake() {
|
||||
a.appProperties.HideFromMake = true
|
||||
}
|
||||
|
||||
func (a *AndroidApp) MarkAsCoverageVariant(coverage bool) {
|
||||
a.appProperties.IsCoverageVariant = coverage
|
||||
}
|
||||
|
Reference in New Issue
Block a user