diff options
| author | 2021-04-24 11:34:46 +0100 | |
|---|---|---|
| committer | 2021-04-24 16:50:47 +0100 | |
| commit | 7b0259ff6204872a4e1027de248a3a23ac36f35e (patch) | |
| tree | 09a28dce15874c20022e6ae7d4f86b3ad903cced /sdk/update.go | |
| parent | cc3132e49616097fd06368b0cc38523764a8db99 (diff) | |
Make order of properties in snapshot module consistent
Previously the compile_multilib property was added after the member
list properties at the top level but before them in the target.os
level. This change makes it always be added before them.
Bug: 186290299
Test: m nothing
Change-Id: I01fe272933fd11f7f730cc379f60fb2ce70f2114
Diffstat (limited to 'sdk/update.go')
| -rw-r--r-- | sdk/update.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sdk/update.go b/sdk/update.go index 248cf5dbf..0bec50564 100644 --- a/sdk/update.go +++ b/sdk/update.go @@ -380,9 +380,6 @@ func (s *sdk) addSnapshotModule(ctx android.ModuleContext, builder *snapshotBuil extractor := newCommonValueExtractor(commonDynamicMemberProperties) extractCommonProperties(ctx, extractor, commonDynamicMemberProperties, dynamicMemberPropertiesContainers) - // Add properties common to all os types. - s.addMemberPropertiesToPropertySet(builder, snapshotModule, commonDynamicMemberProperties) - // Optimize other per-variant properties, besides the dynamic member lists. type variantProperties struct { Compile_multilib string `android:"arch_variant"` @@ -399,11 +396,14 @@ func (s *sdk) addSnapshotModule(ctx android.ModuleContext, builder *snapshotBuil commonVariantProperties := variantProperties{} extractor = newCommonValueExtractor(commonVariantProperties) extractCommonProperties(ctx, extractor, &commonVariantProperties, variantPropertiesContainers) + if commonVariantProperties.Compile_multilib != "" && commonVariantProperties.Compile_multilib != "both" { // Compile_multilib defaults to both so only needs to be set when it's // specified and not both. snapshotModule.AddProperty("compile_multilib", commonVariantProperties.Compile_multilib) } + // Add properties common to all os types. + s.addMemberPropertiesToPropertySet(builder, snapshotModule, commonDynamicMemberProperties) targetPropertySet := snapshotModule.AddPropertySet("target") |