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

Merge "Assert that platform_sdk_codename is set" am: 7a64621169 am: 04657453e7

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

Change-Id: I40d0e33b7dba5106348fb5a6ba7fa4033803d653
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Jiyong Park
2023-04-04 00:25:44 +00:00
committed by Automerger Merge Worker
parent aa8518fad3 04657453e7
commit ebbb91a105
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
android/config.go
View File

@@ -913,8 +913,16 @@ func (c *config) DefaultAppTargetSdk(ctx EarlyModuleContext) ApiLevel {
return c.PlatformSdkVersion()
}
codename := c.PlatformSdkCodename()
hostOnlyBuild := c.productVariables.DeviceArch == nil
if codename == "" {
return NoneApiLevel
// There are some host-only builds (those are invoked by build-prebuilts.sh) which
// don't set platform sdk codename. Platform sdk codename makes sense only when we
// are building the platform. So we don't enforce the below panic for the host-only
// builds.
if hostOnlyBuild {
return NoneApiLevel
}
panic("Platform_sdk_codename must be set")
}
if codename == "REL" {
panic("Platform_sdk_codename should not be REL when Platform_sdk_final is true")