StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove suffix based stub matching logic" into main

Browse Source
This commit is contained in:
Treehugger Robot
2024-08-23 03:42:12 +00:00
committed by Gerrit Code Review
parent fe5f57b27f fa3f0782f7
commit 65c530c17e
4 changed files with 90 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files

43
java/sdk_library.go
View File

@@ -1796,7 +1796,8 @@ type libraryProperties struct {
Dir *string
Tag *string
}
Is_stubs_module *bool
Is_stubs_module *bool
Stub_contributing_api *string
}
func (module *SdkLibrary) stubsLibraryProps(mctx android.DefaultableHookContext, apiScope *apiScope) libraryProperties {
@@ -1822,6 +1823,7 @@ func (module *SdkLibrary) stubsLibraryProps(mctx android.DefaultableHookContext,
// interop with older developer tools that don't support 1.9.
props.Java_version = proptools.StringPtr("1.8")
props.Is_stubs_module = proptools.BoolPtr(true)
props.Stub_contributing_api = proptools.StringPtr(apiScope.kind.String())
return props
}
@@ -2089,6 +2091,8 @@ func (module *SdkLibrary) topLevelStubsLibraryProps(mctx android.DefaultableHook
}
props.Compile_dex = compileDex
props.Stub_contributing_api = proptools.StringPtr(apiScope.kind.String())
if !Bool(module.sdkLibraryProperties.No_dist) && doDist {
props.Dist.Targets = []string{"sdk", "win_sdk"}
props.Dist.Dest = proptools.StringPtr(fmt.Sprintf("%v.jar", module.distStem()))
@@ -2425,36 +2429,25 @@ func (s *defaultNamingScheme) exportableSourceStubsLibraryModuleName(scope *apiS
var _ sdkLibraryComponentNamingScheme = (*defaultNamingScheme)(nil)
func hasStubsLibrarySuffix(name string, apiScope *apiScope) bool {
return strings.HasSuffix(name, apiScope.stubsLibraryModuleNameSuffix()) ||
strings.HasSuffix(name, apiScope.exportableStubsLibraryModuleNameSuffix())
}
func moduleStubLinkType(name string) (stub bool, ret sdkLinkType) {
name = strings.TrimSuffix(name, ".from-source")
// This suffix-based approach is fragile and could potentially mis-trigger.
// TODO(b/155164730): Clean this up when modules no longer reference sdk_lib stubs directly.
if hasStubsLibrarySuffix(name, apiScopePublic) {
if name == "hwbinder.stubs" || name == "libcore_private.stubs" {
// Due to a previous bug, these modules were not considered stubs, so we retain that.
return false, javaPlatform
}
func moduleStubLinkType(j *Module) (stub bool, ret sdkLinkType) {
kind := android.ToSdkKind(proptools.String(j.properties.Stub_contributing_api))
switch kind {
case android.SdkPublic:
return true, javaSdk
}
if hasStubsLibrarySuffix(name, apiScopeSystem) {
case android.SdkSystem:
return true, javaSystem
}
if hasStubsLibrarySuffix(name, apiScopeModuleLib) {
case android.SdkModule:
return true, javaModule
}
if hasStubsLibrarySuffix(name, apiScopeTest) {
case android.SdkTest:
return true, javaSystem
}
if hasStubsLibrarySuffix(name, apiScopeSystemServer) {
case android.SdkSystemServer:
return true, javaSystemServer
// Default value for all modules other than java_sdk_library-generated stub submodules
case android.SdkInvalid:
return false, javaPlatform
default:
panic(fmt.Sprintf("stub_contributing_api set as an unsupported sdk kind %s", kind.String()))
}
return false, javaPlatform
}
// java_sdk_library is a special Java library that provides optional platform APIs to apps.