diff options
Diffstat (limited to 'android')
-rw-r--r-- | android/override_module.go | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/android/override_module.go b/android/override_module.go index 02db359f..119bca1c 100644 --- a/android/override_module.go +++ b/android/override_module.go @@ -134,10 +134,15 @@ func (b *OverridableModuleBase) setOverridesProperty(overridesProperty *[]string // Overrides a base module with the given OverrideModule. func (b *OverridableModuleBase) override(ctx BaseModuleContext, o OverrideModule) { + // Adds the base module to the overrides property, if exists, of the overriding module. See the + // comment on OverridableModuleBase.overridesProperty for details. + if b.overridesProperty != nil { + *b.overridesProperty = append(*b.overridesProperty, b.Name()) + } for _, p := range b.overridableProperties { for _, op := range o.getOverridingProperties() { if proptools.TypeEqual(p, op) { - err := proptools.PrependProperties(p, op, nil) + err := proptools.AppendProperties(p, op, nil) if err != nil { if propertyErr, ok := err.(*proptools.ExtendPropertyError); ok { ctx.PropertyErrorf(propertyErr.Property, "%s", propertyErr.Err.Error()) @@ -148,13 +153,6 @@ func (b *OverridableModuleBase) override(ctx BaseModuleContext, o OverrideModule } } } - // Adds the base module to the overrides property, if exists, of the overriding module. See the - // comment on OverridableModuleBase.overridesProperty for details. - if b.overridesProperty != nil { - *b.overridesProperty = append(*b.overridesProperty, b.Name()) - } - // The base module name property has to be updated separately for Name() to work as intended. - b.module.base().nameProperties.Name = proptools.StringPtr(o.Name()) } // Mutators for override/overridable modules. All the fun happens in these functions. It is critical |