Skip to content

Commit d9509b5

Browse files
authored
Upgrade guava version to 33.4.8 (#12219)
Guava seems to call a deprecated sun.misc.Unsafe::objectFieldOffset method which might be removed in a future JDK release. There are still a few things to do for -android versions, which are tracked in google/guava#7742, see details at google/guava#7811 Fixes #12215
1 parent 4f6948f commit d9509b5

File tree

27 files changed

+331
-16
lines changed

27 files changed

+331
-16
lines changed

MODULE.bazel

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ IO_GRPC_GRPC_JAVA_ARTIFACTS = [
1515
"com.google.auto.value:auto-value:1.11.0",
1616
"com.google.code.findbugs:jsr305:3.0.2",
1717
"com.google.code.gson:gson:2.11.0",
18-
"com.google.errorprone:error_prone_annotations:2.30.0",
18+
"com.google.errorprone:error_prone_annotations:2.36.0",
1919
"com.google.guava:failureaccess:1.0.1",
20-
"com.google.guava:guava:33.3.1-android",
20+
"com.google.guava:guava:33.4.8-android",
2121
"com.google.re2j:re2j:1.8",
2222
"com.google.s2a.proto.v2:s2a-proto:0.1.2",
2323
"com.google.truth:truth:1.4.2",
@@ -41,7 +41,7 @@ IO_GRPC_GRPC_JAVA_ARTIFACTS = [
4141
"io.opencensus:opencensus-contrib-grpc-metrics:0.31.0",
4242
"io.perfmark:perfmark-api:0.27.0",
4343
"junit:junit:4.13.2",
44-
"org.checkerframework:checker-qual:3.12.0",
44+
"org.checkerframework:checker-qual:3.49.5",
4545
"org.codehaus.mojo:animal-sniffer-annotations:1.24",
4646
]
4747
# GRPC_DEPS_END

core/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
buildscript {
22
dependencies {
3-
classpath 'com.google.guava:guava:30.0-android'
3+
classpath 'com.google.guava:guava:33.4.8-android'
44
}
55
}
66

cronet/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ dependencies {
4646
libraries.cronet.api
4747
implementation project(':grpc-core')
4848
implementation libraries.guava
49+
implementation 'org.checkerframework:checker-qual:3.49.5'
4950
testImplementation project(':grpc-testing')
5051

5152
testImplementation libraries.cronet.embedded
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,17 @@
1+
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
}
16+
117
include ':app'
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,17 @@
1+
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
}
16+
117
include ':app'
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,17 @@
1+
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
}
16+
117
include ':app'
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,17 @@
1+
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
}
16+
117
include ':app'

examples/example-alts/settings.gradle

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,18 @@
11
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
216
repositories {
317
gradlePluginPortal()
418
}
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,17 @@
1+
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
}
16+
117
rootProject.name = 'example-debug'

examples/example-dualstack/settings.gradle

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,18 @@
11
pluginManagement {
2+
// https://issuetracker.google.com/issues/342522142#comment8
3+
// use D8/R8 8.0.44 or 8.1.44 with AGP 7.4 if needed.
4+
buildscript {
5+
repositories {
6+
mavenCentral()
7+
maven {
8+
url = uri("https://storage.googleapis.com/r8-releases/raw")
9+
}
10+
}
11+
dependencies {
12+
classpath("com.android.tools:r8:8.1.44")
13+
}
14+
}
15+
216
repositories {
317
gradlePluginPortal()
418
}

0 commit comments

Comments
 (0)