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

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

Merge "Improve docs about //visibility:public" into main am: 2545872a am: c8a2c341 

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



Change-Id: I97d4e3b55e1dfed188f7845f3139ddeb33c399ee
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 6f67fe16 c8a2c341

sdk/update.go

+7 −5
Original line number Diff line number Diff line
@@ -1118,12 +1118,14 @@ func (s *snapshotBuilder) AddPrebuiltModule(member android.SdkMember, moduleType 
		// Since module sdks are generated from release branches and dropped to development

		// branches, there might be a visibility skew between the sources and prebuilts

		// of a specific module.

		// To reconcile this potential skew, change the visibility to public

		// To reconcile this potential skew, change the visibility to public.

		//

		// This is safe for (1) since these are stub libraries.

		// This is ok for (2) since these are host and test exports and are intended for

		// ART development.

		// TODO (b/361303067): This can be removed if ART uses full manifests.

		// This means dependencies can bypass visibility restrictions when prebuilts are used, so we rely

		// on source builds in CI to check them.

		//

		// TODO (b/361303067): This special case for category (2) can be removed if existing usages

		// of host/test prebuilts of modules like conscrypt,tzdata,i18n are switched to source builds.

		// It will also require ART switching to full manifests.

		m.AddProperty("visibility", []string{"//visibility:public"})

	}