diff options
author | Jaewoong Jung <jungjw@google.com> | 2019-06-19 11:22:25 -0700 |
---|---|---|
committer | Jaewoong Jung <jungjw@google.com> | 2019-06-19 15:04:27 -0700 |
commit | 3ad00bf196544f686eaf8b9caf8ef44d3e991744 (patch) | |
tree | a58a28229b9eb337ce08dbacd1caae6775c863e2 | |
parent | 6df90831b7825bcb3dc233dbc5b1e47e4316d990 (diff) | |
download | build_soong-3ad00bf196544f686eaf8b9caf8ef44d3e991744.tar.gz build_soong-3ad00bf196544f686eaf8b9caf8ef44d3e991744.tar.bz2 build_soong-3ad00bf196544f686eaf8b9caf8ef44d3e991744.zip |
OverridableModuleBase shouldn't embed ModuleBase.
Individual module structs normally embed ModuleBase themselves. This
caused duplicate embedding, and so made common properties like notice
inaccessible.
Bug: 135460391
Test: app_test.go, TreeHugger
Change-Id: I5683b194e3de909d480c083a931cc7e871de1d74
Merged-In: I5683b194e3de909d480c083a931cc7e871de1d74
-rw-r--r-- | android/override_module.go | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/android/override_module.go b/android/override_module.go index 119bca1c..ba66182e 100644 --- a/android/override_module.go +++ b/android/override_module.go @@ -90,8 +90,6 @@ type OverridableModule interface { // Base module struct for overridable module types type OverridableModuleBase struct { - ModuleBase - // List of OverrideModules that override this base module overrides []OverrideModule // Used to parallelize registerOverrideMutator executions. Note that only addOverride locks this @@ -137,7 +135,7 @@ 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()) + *b.overridesProperty = append(*b.overridesProperty, ctx.ModuleName()) } for _, p := range b.overridableProperties { for _, op := range o.getOverridingProperties() { |