Merge "Rewrite macOS SDK selection" am: 1c6de4bdac
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1864059 Change-Id: I0f360745b26dad800f0a5bd697c4a5745dfb8883
This commit is contained in:
@@ -56,9 +56,7 @@ var (
|
||||
"10.13",
|
||||
"10.14",
|
||||
"10.15",
|
||||
"11.0",
|
||||
"11.1",
|
||||
"11.3",
|
||||
"11",
|
||||
}
|
||||
|
||||
darwinAvailableLibraries = append(
|
||||
@@ -138,7 +136,7 @@ func getMacTools(ctx android.PathContext) *macPlatformTools {
|
||||
return ""
|
||||
}
|
||||
|
||||
bytes, err := exec.Command(xcrunTool, args...).Output()
|
||||
bytes, err := exec.Command(xcrunTool, append([]string{"--sdk", "macosx"}, args...)...).Output()
|
||||
if err != nil {
|
||||
macTools.err = fmt.Errorf("xcrun %q failed with: %q", args, err)
|
||||
return ""
|
||||
@@ -147,27 +145,19 @@ func getMacTools(ctx android.PathContext) *macPlatformTools {
|
||||
return strings.TrimSpace(string(bytes))
|
||||
}
|
||||
|
||||
xcrunSdk := func(arg string) string {
|
||||
if selected := ctx.Config().Getenv("MAC_SDK_VERSION"); selected != "" {
|
||||
if !inList(selected, darwinSupportedSdkVersions) {
|
||||
macTools.err = fmt.Errorf("MAC_SDK_VERSION %s isn't supported: %q", selected, darwinSupportedSdkVersions)
|
||||
return ""
|
||||
}
|
||||
|
||||
return xcrun("--sdk", "macosx"+selected, arg)
|
||||
sdkVersion := xcrun("--show-sdk-version")
|
||||
sdkVersionSupported := false
|
||||
for _, version := range darwinSupportedSdkVersions {
|
||||
if version == sdkVersion || strings.HasPrefix(sdkVersion, version+".") {
|
||||
sdkVersionSupported = true
|
||||
}
|
||||
|
||||
for _, sdk := range darwinSupportedSdkVersions {
|
||||
bytes, err := exec.Command(xcrunTool, "--sdk", "macosx"+sdk, arg).Output()
|
||||
if err == nil {
|
||||
return strings.TrimSpace(string(bytes))
|
||||
}
|
||||
}
|
||||
macTools.err = fmt.Errorf("Could not find a supported mac sdk: %q", darwinSupportedSdkVersions)
|
||||
return ""
|
||||
}
|
||||
if !sdkVersionSupported {
|
||||
macTools.err = fmt.Errorf("Unsupported macOS SDK version %q not in %v", sdkVersion, darwinSupportedSdkVersions)
|
||||
return
|
||||
}
|
||||
|
||||
macTools.sdkRoot = xcrunSdk("--show-sdk-path")
|
||||
macTools.sdkRoot = xcrun("--show-sdk-path")
|
||||
|
||||
macTools.arPath = xcrun("--find", "ar")
|
||||
macTools.stripPath = xcrun("--find", "strip")
|
||||
|
Reference in New Issue
Block a user