Add RemoveOptionalPrebuiltPrefix() helper function
Test: m nothing Bug: 171061220 Change-Id: Iaca95efcaf3f02e066751c6e988d609ac40e048a
This commit is contained in:
parent
a7d6a89774
commit
d23c726b36
3 changed files with 9 additions and 2 deletions
|
@ -17,6 +17,7 @@ package android
|
|||
import (
|
||||
"fmt"
|
||||
"reflect"
|
||||
"strings"
|
||||
|
||||
"github.com/google/blueprint"
|
||||
"github.com/google/blueprint/proptools"
|
||||
|
@ -74,6 +75,12 @@ type Prebuilt struct {
|
|||
srcsPropertyName string
|
||||
}
|
||||
|
||||
// RemoveOptionalPrebuiltPrefix returns the result of removing the "prebuilt_" prefix from the
|
||||
// supplied name if it has one, or returns the name unmodified if it does not.
|
||||
func RemoveOptionalPrebuiltPrefix(name string) string {
|
||||
return strings.TrimPrefix(name, "prebuilt_")
|
||||
}
|
||||
|
||||
func (p *Prebuilt) Name(name string) string {
|
||||
return "prebuilt_" + name
|
||||
}
|
||||
|
|
|
@ -2178,7 +2178,7 @@ func baselineApexAvailable(apex, moduleName string) bool {
|
|||
func normalizeModuleName(moduleName string) string {
|
||||
// Prebuilt modules (e.g. java_import, etc.) have "prebuilt_" prefix added by the build
|
||||
// system. Trim the prefix for the check since they are confusing
|
||||
moduleName = strings.TrimPrefix(moduleName, "prebuilt_")
|
||||
moduleName = android.RemoveOptionalPrebuiltPrefix(moduleName)
|
||||
if strings.HasPrefix(moduleName, "libclang_rt.") {
|
||||
// This module has many arch variants that depend on the product being built.
|
||||
// We don't want to list them all
|
||||
|
|
2
cc/cc.go
2
cc/cc.go
|
@ -2749,7 +2749,7 @@ func orderStaticModuleDeps(staticDeps []StaticLibraryInfo, sharedDeps []SharedLi
|
|||
func baseLibName(depName string) string {
|
||||
libName := strings.TrimSuffix(depName, llndkLibrarySuffix)
|
||||
libName = strings.TrimSuffix(libName, vendorPublicLibrarySuffix)
|
||||
libName = strings.TrimPrefix(libName, "prebuilt_")
|
||||
libName = android.RemoveOptionalPrebuiltPrefix(libName)
|
||||
return libName
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue