diff options
author | 2022-10-07 15:24:12 +0100 | |
---|---|---|
committer | 2023-01-11 12:47:10 +0000 | |
commit | d6be9e51ca5d935087d6735dd5d8dc2c40801ae7 (patch) | |
tree | 47f6773ca7ec3d9cb128c01dcb87b2f8fb58f48c | |
parent | e0598a1cbd20d410d33d7849a3ef9f06e683031e (diff) |
Target Java 17
Bug: 233029164
Test: TH
Change-Id: I750b866565c264eb4f4ac14ebc88be6fc9b1b458
-rw-r--r-- | android/config.go | 4 | ||||
-rw-r--r-- | java/java.go | 8 | ||||
-rw-r--r-- | java/sdk.go | 10 |
3 files changed, 4 insertions, 18 deletions
diff --git a/android/config.go b/android/config.go index d5ed883fa..255c836bf 100644 --- a/android/config.go +++ b/android/config.go @@ -723,10 +723,6 @@ func (c *config) IsEnvFalse(key string) bool { return value == "0" || value == "n" || value == "no" || value == "off" || value == "false" } -func (c *config) TargetsJava17() bool { - return c.IsEnvTrue("EXPERIMENTAL_TARGET_JAVA_VERSION_17") -} - // EnvDeps returns the environment variables this build depends on. The first // call to this function blocks future reads from the environment. func (c *config) EnvDeps() map[string]string { diff --git a/java/java.go b/java/java.go index 3b0ad8d9f..6e428cb1f 100644 --- a/java/java.go +++ b/java/java.go @@ -510,14 +510,8 @@ func getJavaVersion(ctx android.ModuleContext, javaVersion string, sdkContext an return normalizeJavaVersion(ctx, javaVersion) } else if ctx.Device() { return defaultJavaLanguageVersion(ctx, sdkContext.SdkVersion(ctx)) - } else if ctx.Config().TargetsJava17() { - // Temporary experimental flag to be able to try and build with - // java version 17 options. The flag, if used, just sets Java - // 17 as the default version, leaving any components that - // target an older version intact. - return JAVA_VERSION_17 } else { - return JAVA_VERSION_11 + return JAVA_VERSION_17 } } diff --git a/java/sdk.go b/java/sdk.go index b0da5afba..10ae3f6e8 100644 --- a/java/sdk.go +++ b/java/sdk.go @@ -57,14 +57,10 @@ func defaultJavaLanguageVersion(ctx android.EarlyModuleContext, s android.SdkSpe return JAVA_VERSION_8 } else if sdk.FinalOrFutureInt() <= 31 { return JAVA_VERSION_9 - } else if ctx.Config().TargetsJava17() { - // Temporary experimental flag to be able to try and build with - // java version 17 options. The flag, if used, just sets Java - // 17 as the default version, leaving any components that - // target an older version intact. - return JAVA_VERSION_17 - } else { + } else if sdk.FinalOrFutureInt() <= 32 { return JAVA_VERSION_11 + } else { + return JAVA_VERSION_17 } } |