Skip to content

Commit 919bed6

Browse files
authored
Reland "Update template/test compileSdk, targetSdk, ndk versions" (#153795)
Relands flutter/flutter#152487. The difference is in the two new commits: -flutter/flutter@1354c6d -flutter/flutter@931788a, short version is that: - I forgot that I need to override the compileSdkVersion in the AGP 8.0 [instance of this test](https://github.com/flutter/flutter/blob/ef9cd32f5a2c0e27300fb9f4662d11da976087d7/dev/devicelab/bin/tasks/android_java17_dependency_smoke_tests.dart#L19) - A postsubmit integration test needed new lockfiles + the package attribute -> AGP namespace change. These were the only two postsubmit failures: [dashboard](https://flutter-dashboard.appspot.com/#/build).
1 parent b8f89f7 commit 919bed6

File tree

55 files changed

+522
-397
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+522
-397
lines changed

dev/benchmarks/microbenchmarks/android/gradle/wrapper/gradle-wrapper.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ distributionBase=GRADLE_USER_HOME
22
distributionPath=wrapper/dists
33
zipStoreBase=GRADLE_USER_HOME
44
zipStorePath=wrapper/dists
5-
distributionUrl=https\://services.gradle.org/distributions/gradle-7.6.3-all.zip
5+
distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-all.zip

dev/benchmarks/microbenchmarks/android/settings.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ pluginManagement {
2323

2424
plugins {
2525
id "dev.flutter.flutter-plugin-loader" version "1.0.0"
26-
id "com.android.application" version "7.3.0" apply false
27-
id "org.jetbrains.kotlin.android" version "1.7.10" apply false
26+
id "com.android.application" version "8.1.0" apply false
27+
id "org.jetbrains.kotlin.android" version "1.8.22" apply false
2828
}
2929

3030
include ":app"

dev/benchmarks/multiple_flutters/android/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ buildscript {
1010
mavenCentral()
1111
}
1212
dependencies {
13-
classpath "com.android.tools.build:gradle:7.3.0"
13+
classpath "com.android.tools.build:gradle:8.1.0"
1414
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
1515

1616
// Do not place your application dependencies here; they belong

dev/benchmarks/multiple_flutters/android/gradle/wrapper/gradle-wrapper.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
33
distributionPath=wrapper/dists
44
zipStoreBase=GRADLE_USER_HOME
55
zipStorePath=wrapper/dists
6-
distributionUrl=https\://services.gradle.org/distributions/gradle-7.6.3-all.zip
6+
distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-all.zip

dev/devicelab/bin/tasks/android_java11_dependency_smoke_tests.dart

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,23 +16,30 @@ import 'package:flutter_devicelab/framework/framework.dart';
1616
// - Kotlin: No methodology as of yet.
1717
// (*) - support range defined in packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts.
1818
List<VersionTuple> versionTuples = <VersionTuple>[
19-
VersionTuple(agpVersion: '7.0.1', gradleVersion: '7.0.2', kotlinVersion: '1.7.10'),
20-
VersionTuple(agpVersion: '7.1.0', gradleVersion: '7.2', kotlinVersion: '1.7.10'),
21-
VersionTuple(agpVersion: '7.2.0', gradleVersion: '7.3.3', kotlinVersion: '1.7.10'),
22-
VersionTuple(agpVersion: '7.3.0', gradleVersion: '7.4', kotlinVersion: '1.7.10'),
23-
VersionTuple(agpVersion: '7.4.0', gradleVersion: '7.5', kotlinVersion: '1.8.10'),
19+
VersionTuple(agpVersion: '7.0.1', gradleVersion: '7.0.2', kotlinVersion: '1.7.10', compileSdkVersion: '34'),
20+
VersionTuple(agpVersion: '7.1.0', gradleVersion: '7.2', kotlinVersion: '1.7.10', compileSdkVersion: '34'),
21+
VersionTuple(agpVersion: '7.2.0', gradleVersion: '7.3.3', kotlinVersion: '1.7.10', compileSdkVersion: '34'),
22+
VersionTuple(agpVersion: '7.3.0', gradleVersion: '7.4', kotlinVersion: '1.7.10', compileSdkVersion: '34'),
23+
VersionTuple(agpVersion: '7.4.0', gradleVersion: '7.5', kotlinVersion: '1.8.10', compileSdkVersion: '34'),
2424
];
2525

2626
// This test requires a Java version less than 17 due to the intentionally low
2727
// version of Gradle. We choose 11 because this was the primary version used in
28-
// CI before 17, and hence it is also hosted on CIPD.
28+
// CI before 17, and hence it is also hosted on CIPD. It also overrides to
29+
// compileSdkVersion 34 because compileSdk 35 requires AGP 8.0+.
2930
// https://docs.gradle.org/current/userguide/compatibility.html
3031
Future<void> main() async {
3132
/// The [FileSystem] for the integration test environment.
3233
const LocalFileSystem fileSystem = LocalFileSystem();
3334

34-
final Directory tempDir = fileSystem.systemTempDirectory.createTempSync('flutter_android_dependency_version_tests');
35+
final Directory tempDir = fileSystem
36+
.systemTempDirectory
37+
.createTempSync('flutter_android_dependency_version_tests');
3538
await task(() {
36-
return buildFlutterApkWithSpecifiedDependencyVersions(versionTuples: versionTuples, tempDir: tempDir, localFileSystem: fileSystem);
39+
return buildFlutterApkWithSpecifiedDependencyVersions(
40+
versionTuples: versionTuples,
41+
tempDir: tempDir,
42+
localFileSystem: fileSystem
43+
);
3744
});
3845
}

dev/devicelab/bin/tasks/android_java17_dependency_smoke_tests.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,9 @@ import 'package:flutter_devicelab/framework/framework.dart';
1515
// https://developer.android.com/build/releases/past-releases/agp-8-4-0-release-notes.
1616
// - Kotlin: No methodology as of yet.
1717
// (*) - support range defined in packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts.
18+
// Note that compileSdk 35 requires AGP 8.1.0+, so override to compileSdk 34 for AGP 8.0.
1819
List<VersionTuple> versionTuples = <VersionTuple>[
19-
VersionTuple(agpVersion: '8.0.0', gradleVersion: '8.0', kotlinVersion: '1.8.22'),
20+
VersionTuple(agpVersion: '8.0.0', gradleVersion: '8.0', kotlinVersion: '1.8.22', compileSdkVersion: '34'),
2021
VersionTuple(agpVersion: '8.1.0', gradleVersion: '8.0', kotlinVersion: '1.8.22'),
2122
VersionTuple(agpVersion: '8.2.0', gradleVersion: '8.2', kotlinVersion: '1.8.22'),
2223
VersionTuple(agpVersion: '8.3.0', gradleVersion: '8.4', kotlinVersion: '1.8.22'),

dev/devicelab/bin/tasks/module_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -449,7 +449,7 @@ class ModuleTest {
449449
Future<void> main() async {
450450
await task(combine(<TaskFunction>[
451451
// ignore: avoid_redundant_argument_values
452-
ModuleTest('module-gradle-7.6', gradleVersion: '7.6.3').call,
453-
ModuleTest('module-gradle-7.6', gradleVersion: '7.6-rc-2').call,
452+
ModuleTest('module-gradle-7.6', gradleVersion: '8.4').call,
453+
ModuleTest('module-gradle-7.6', gradleVersion: '8.4-rc-3').call,
454454
]));
455455
}

dev/devicelab/lib/framework/dependency_smoke_test_task_definition.dart

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,23 +58,27 @@ distributionUrl=https\://services.gradle.org/distributions/gradle-GRADLE_REPLACE
5858
''';
5959

6060
const String gradleReplacementString = 'GRADLE_REPLACE_ME';
61+
const String flutterCompileSdkString = 'flutter.compileSdkVersion';
6162

6263
/// A simple class containing a Kotlin, Gradle, and AGP version.
6364
class VersionTuple {
6465

6566
VersionTuple({
6667
required this.agpVersion,
6768
required this.gradleVersion,
68-
required this.kotlinVersion
69+
required this.kotlinVersion,
70+
this.compileSdkVersion,
6971
});
7072

7173
String agpVersion;
7274
String gradleVersion;
7375
String kotlinVersion;
76+
String? compileSdkVersion;
7477

7578
@override
7679
String toString() {
77-
return '(AGP version: $agpVersion, Gradle version: $gradleVersion, Kotlin version: $kotlinVersion)';
80+
return '(AGP version: $agpVersion, Gradle version: $gradleVersion, Kotlin version: $kotlinVersion'
81+
'${(compileSdkVersion == null) ? '' : ', compileSdk version: $compileSdkVersion)'}';
7882
}
7983
}
8084

@@ -93,7 +97,7 @@ Future<TaskResult> buildFlutterApkWithSpecifiedDependencyVersions({
9397
final Directory innerTempDir = tempDir.createTempSync(versions.gradleVersion);
9498
try {
9599
// Create a new flutter project.
96-
section('Create new app with Gradle ${versions.gradleVersion}, AGP ${versions.agpVersion}, and Kotlin ${versions.kotlinVersion}');
100+
section('Create new app with dependency versions: $versions');
97101
await flutter(
98102
'create',
99103
options: <String>[
@@ -105,6 +109,14 @@ Future<TaskResult> buildFlutterApkWithSpecifiedDependencyVersions({
105109

106110
final String appPath = '${innerTempDir.absolute.path}/dependency_checker_app';
107111

112+
if (versions.compileSdkVersion != null) {
113+
final File appGradleBuild = localFileSystem.file(localFileSystem.path.join(
114+
appPath, 'android', 'app', 'build.gradle'));
115+
final String appBuildContent = appGradleBuild.readAsStringSync()
116+
.replaceFirst(flutterCompileSdkString, versions.compileSdkVersion!);
117+
appGradleBuild.writeAsStringSync(appBuildContent);
118+
}
119+
108120
// Modify gradle version to passed in version.
109121
final File gradleWrapperProperties = localFileSystem.file(localFileSystem.path.join(
110122
appPath, 'android', 'gradle', 'wrapper', 'gradle-wrapper.properties'));

dev/integration_tests/android_custom_host_app/SampleApp/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
apply plugin: 'com.android.application'
66

77
android {
8+
namespace = "io.flutter.add2app"
89
compileSdk 34
910

1011
compileOptions {

dev/integration_tests/android_custom_host_app/SampleApp/src/main/AndroidManifest.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ Use of this source code is governed by a BSD-style license that can be
33
found in the LICENSE file. -->
44

55
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
6-
xmlns:tools="http://schemas.android.com/tools"
7-
package="io.flutter.add2app">
6+
xmlns:tools="http://schemas.android.com/tools">
87

98
<application android:allowBackup="false"
109
tools:ignore="GoogleAppIndexingWarning,MissingApplicationIcon">

0 commit comments

Comments
 (0)