Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7686f94d authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Use copy of apex_available in AvailableToSameApexes" am: d7b50181 

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2559590



Change-Id: Ib6cd6bada1169984865e5c62c13fb026b18d7371
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 00dd119d d7b50181

android/apex.go

+3 −2
Original line number Diff line number Diff line
@@ -513,8 +513,9 @@ func (m *ApexModuleBase) checkApexAvailableProperty(mctx BaseModuleContext) { 
// exactly the same set of APEXes (and platform), i.e. if their apex_available

// properties have the same elements.

func AvailableToSameApexes(mod1, mod2 ApexModule) bool {

	mod1ApexAvail := SortedUniqueStrings(mod1.apexModuleBase().ApexProperties.Apex_available)

	mod2ApexAvail := SortedUniqueStrings(mod2.apexModuleBase().ApexProperties.Apex_available)

	// Use CopyOf to prevent non-determinism (b/275313114#comment1)

	mod1ApexAvail := SortedUniqueStrings(CopyOf(mod1.apexModuleBase().ApexProperties.Apex_available))

	mod2ApexAvail := SortedUniqueStrings(CopyOf(mod2.apexModuleBase().ApexProperties.Apex_available))

	if len(mod1ApexAvail) != len(mod2ApexAvail) {

		return false

	}

cc/cc_test.go

+0 −3
Original line number Diff line number Diff line
@@ -3578,9 +3578,6 @@ func TestVersionedStubs(t *testing.T) { 
}



func TestStubsForLibraryInMultipleApexes(t *testing.T) {

	// TODO(b/275313114): Test exposes non-determinism which should be corrected and the test

	// reenabled.

	t.Skip()

	t.Parallel()

	ctx := testCc(t, `

		cc_library_shared {