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

Commit f4ae4f13 authored by Paul Duffin's avatar Paul Duffin
Browse files

Add support for transitive sdk members 

Allow an sdk member type to treat some of its dependencies as being
members of the sdk.

Needed for the java_system_modules type whose libs property are an
implementation detail of the system module and so should not be
explicitly listed in the sdk module but still have to be included in
the sdk snapshot.

Bug: 142940300
Test: m nothing
Change-Id: I90f37dae269ef64a6fe9debd0bbaf29a64dd74d8
parent f3c9d74c

android/sdk.go

+14 −2
Original line number Diff line number Diff line
@@ -264,6 +264,13 @@ type SdkMemberType interface { 
	// True if the member type supports the sdk/sdk_snapshot, false otherwise.

	UsableWithSdkAndSdkSnapshot() bool



	// Return true if modules of this type can have dependencies which should be

	// treated as if they are sdk members.

	//

	// Any dependency that is to be treated as a member of the sdk needs to implement

	// SdkAware and be added with an SdkMemberTypeDependencyTag tag.

	HasTransitiveSdkMembers() bool



	// Add dependencies from the SDK module to all the variants the member

	// contributes to the SDK. The exact set of variants required is determined

	// by the SDK and its properties. The dependencies must be added with the
@@ -293,6 +300,7 @@ type SdkMemberType interface { 
type SdkMemberTypeBase struct {

	PropertyName         string

	SupportsSdk          bool

	TransitiveSdkMembers bool

}



func (b *SdkMemberTypeBase) SdkPropertyName() string {
@@ -303,6 +311,10 @@ func (b *SdkMemberTypeBase) UsableWithSdkAndSdkSnapshot() bool { 
	return b.SupportsSdk

}



func (b *SdkMemberTypeBase) HasTransitiveSdkMembers() bool {

	return b.TransitiveSdkMembers

}



// Encapsulates the information about registered SdkMemberTypes.

type SdkMemberTypesRegistry struct {

	// The list of types sorted by property name.

sdk/update.go

+14 −8
Original line number Diff line number Diff line
@@ -105,23 +105,23 @@ func (gf *generatedFile) build(pctx android.PackageContext, ctx android.BuilderC 
// Collect all the members.

//

// The members are first grouped by type and then grouped by name. The order of

// the types is the order they are referenced in android.SdkMemberTypes. The

// names are in order in which the dependencies were added.

// the types is the order they are referenced in android.SdkMemberTypesRegistry.

// The names are in the order in which the dependencies were added.

func (s *sdk) collectMembers(ctx android.ModuleContext) []*sdkMember {

	byType := make(map[android.SdkMemberType][]*sdkMember)

	byName := make(map[string]*sdkMember)



	ctx.VisitDirectDeps(func(m android.Module) {

		tag := ctx.OtherModuleDependencyTag(m)

	ctx.WalkDeps(func(child android.Module, parent android.Module) bool {

		tag := ctx.OtherModuleDependencyTag(child)

		if memberTag, ok := tag.(android.SdkMemberTypeDependencyTag); ok {

			memberType := memberTag.SdkMemberType()



			// Make sure that the resolved module is allowed in the member list property.

			if !memberType.IsInstance(m) {

				ctx.ModuleErrorf("module %q is not valid in property %s", ctx.OtherModuleName(m), memberType.SdkPropertyName())

			if !memberType.IsInstance(child) {

				ctx.ModuleErrorf("module %q is not valid in property %s", ctx.OtherModuleName(child), memberType.SdkPropertyName())

			}



			name := ctx.OtherModuleName(m)

			name := ctx.OtherModuleName(child)



			member := byName[name]

			if member == nil {
@@ -130,8 +130,14 @@ func (s *sdk) collectMembers(ctx android.ModuleContext) []*sdkMember { 
				byType[memberType] = append(byType[memberType], member)

			}



			member.variants = append(member.variants, m.(android.SdkAware))

			member.variants = append(member.variants, child.(android.SdkAware))



			// If the member type supports transitive sdk members then recurse down into

			// its dependencies, otherwise exit traversal.

			return memberType.HasTransitiveSdkMembers()

		}



		return false

	})



	var members []*sdkMember