From d04edd1d7ff2ce37512dee1c202aa46bfa9096ea Mon Sep 17 00:00:00 2001 From: Gray Mackall <34871572+gmackall@users.noreply.github.com> Date: Tue, 7 May 2024 10:32:10 -0700 Subject: [PATCH] Add kotlinOptions jvmTarget to templates (#147326) Adds jvmTarget to gradle templates. Fixes https://github.com/flutter/flutter/issues/147185. Interestingly, we already had this in many of our examples/tests, see search results here https://github.com/search?q=repo%3Aflutter%2Fflutter%20jvmTarget&type=code. --- .../abstract_method_smoke_test/android/app/build.gradle | 2 +- .../android/app/build.gradle | 2 +- .../android_semantics_testing/android/app/build.gradle | 6 +++++- dev/integration_tests/channels/android/app/build.gradle | 4 ++++ dev/integration_tests/non_nullable/android/app/build.gradle | 2 +- .../release_smoke_test/android/app/build.gradle | 6 +++++- dev/integration_tests/spell_check/android/app/build.gradle | 2 +- dev/integration_tests/ui/android/app/build.gradle | 4 ++++ .../app_shared/android-java.tmpl/app/build.gradle.tmpl | 4 ++++ .../app_shared/android-kotlin.tmpl/app/build.gradle.tmpl | 4 ++++ .../templates/plugin/android-kotlin.tmpl/build.gradle.tmpl | 2 +- 11 files changed, 31 insertions(+), 7 deletions(-) diff --git a/dev/integration_tests/abstract_method_smoke_test/android/app/build.gradle b/dev/integration_tests/abstract_method_smoke_test/android/app/build.gradle index 3fc0256254f..7171eb700d4 100644 --- a/dev/integration_tests/abstract_method_smoke_test/android/app/build.gradle +++ b/dev/integration_tests/abstract_method_smoke_test/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() diff --git a/dev/integration_tests/android_embedding_v2_smoke_test/android/app/build.gradle b/dev/integration_tests/android_embedding_v2_smoke_test/android/app/build.gradle index 6c725414d64..e44d82de81d 100644 --- a/dev/integration_tests/android_embedding_v2_smoke_test/android/app/build.gradle +++ b/dev/integration_tests/android_embedding_v2_smoke_test/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() diff --git a/dev/integration_tests/android_semantics_testing/android/app/build.gradle b/dev/integration_tests/android_semantics_testing/android/app/build.gradle index 17ff7831b4c..25afe619dec 100644 --- a/dev/integration_tests/android_semantics_testing/android/app/build.gradle +++ b/dev/integration_tests/android_semantics_testing/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() @@ -25,6 +25,10 @@ android { targetCompatibility JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { minSdkVersion flutter.minSdkVersion targetSdkVersion flutter.targetSdkVersion diff --git a/dev/integration_tests/channels/android/app/build.gradle b/dev/integration_tests/channels/android/app/build.gradle index 04eed4c4bc7..17f8f1332ce 100644 --- a/dev/integration_tests/channels/android/app/build.gradle +++ b/dev/integration_tests/channels/android/app/build.gradle @@ -36,6 +36,10 @@ android { targetCompatibility JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html). applicationId "com.example.channels" diff --git a/dev/integration_tests/non_nullable/android/app/build.gradle b/dev/integration_tests/non_nullable/android/app/build.gradle index 45be794a1ac..9822c24687d 100644 --- a/dev/integration_tests/non_nullable/android/app/build.gradle +++ b/dev/integration_tests/non_nullable/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() diff --git a/dev/integration_tests/release_smoke_test/android/app/build.gradle b/dev/integration_tests/release_smoke_test/android/app/build.gradle index dad22850eee..357e24908a0 100644 --- a/dev/integration_tests/release_smoke_test/android/app/build.gradle +++ b/dev/integration_tests/release_smoke_test/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() @@ -35,6 +35,10 @@ android { targetCompatibility JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html). applicationId "com.example.release_smoke_test" diff --git a/dev/integration_tests/spell_check/android/app/build.gradle b/dev/integration_tests/spell_check/android/app/build.gradle index 258f3c0457c..40d896e8738 100644 --- a/dev/integration_tests/spell_check/android/app/build.gradle +++ b/dev/integration_tests/spell_check/android/app/build.gradle @@ -4,8 +4,8 @@ plugins { id "com.android.application" - id "dev.flutter.flutter-gradle-plugin" id "kotlin-android" + id "dev.flutter.flutter-gradle-plugin" } def localProperties = new Properties() diff --git a/dev/integration_tests/ui/android/app/build.gradle b/dev/integration_tests/ui/android/app/build.gradle index 4117aa2ab90..0b7c7957414 100644 --- a/dev/integration_tests/ui/android/app/build.gradle +++ b/dev/integration_tests/ui/android/app/build.gradle @@ -17,6 +17,10 @@ android { targetCompatibility JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { minSdkVersion flutter.minSdkVersion targetSdkVersion flutter.targetSdkVersion diff --git a/packages/flutter_tools/templates/app_shared/android-java.tmpl/app/build.gradle.tmpl b/packages/flutter_tools/templates/app_shared/android-java.tmpl/app/build.gradle.tmpl index ab0bc1bf60f..7f0bf6f4f3f 100644 --- a/packages/flutter_tools/templates/app_shared/android-java.tmpl/app/build.gradle.tmpl +++ b/packages/flutter_tools/templates/app_shared/android-java.tmpl/app/build.gradle.tmpl @@ -15,6 +15,10 @@ android { targetCompatibility = JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html). applicationId = "{{androidIdentifier}}" diff --git a/packages/flutter_tools/templates/app_shared/android-kotlin.tmpl/app/build.gradle.tmpl b/packages/flutter_tools/templates/app_shared/android-kotlin.tmpl/app/build.gradle.tmpl index ab0bc1bf60f..7f0bf6f4f3f 100644 --- a/packages/flutter_tools/templates/app_shared/android-kotlin.tmpl/app/build.gradle.tmpl +++ b/packages/flutter_tools/templates/app_shared/android-kotlin.tmpl/app/build.gradle.tmpl @@ -15,6 +15,10 @@ android { targetCompatibility = JavaVersion.VERSION_1_8 } + kotlinOptions { + jvmTarget = JavaVersion.VERSION_1_8 + } + defaultConfig { // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html). applicationId = "{{androidIdentifier}}" diff --git a/packages/flutter_tools/templates/plugin/android-kotlin.tmpl/build.gradle.tmpl b/packages/flutter_tools/templates/plugin/android-kotlin.tmpl/build.gradle.tmpl index 8838f3e3fb2..bd3c2d811bd 100644 --- a/packages/flutter_tools/templates/plugin/android-kotlin.tmpl/build.gradle.tmpl +++ b/packages/flutter_tools/templates/plugin/android-kotlin.tmpl/build.gradle.tmpl @@ -37,7 +37,7 @@ android { } kotlinOptions { - jvmTarget = "1.8" + jvmTarget = JavaVersion.VERSION_1_8 } sourceSets {