-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.
Description
Error: Require upper bound dependencies error for com.google.guava:failureaccess:1.0.1 paths to dependency are:
Error: +-com.google.cloud:google-cloud-notification:0.156.0-beta-SNAPSHOT
Error: +-com.google.cloud:google-cloud-storage:2.34.0
Error: +-com.google.guava:failureaccess:1.0.1
Error: and
Error: +-com.google.cloud:google-cloud-notification:0.156.0-beta-SNAPSHOT
Error: +-com.google.api.grpc:proto-google-cloud-pubsub-v1:1.109.2
Error: +-com.google.guava:failureaccess:1.0.1
Error: and
Error: +-com.google.cloud:google-cloud-notification:0.156.0-beta-SNAPSHOT
Error: +-com.google.guava:guava:33.1.0-jre
Error: +-com.google.guava:failureaccess:1.0.2
Error: ]
I added a temporary exclusion in guava
google-cloud-java/java-notification/pom.xml
Lines 53 to 62 in b885b13
| <dependency> | |
| <groupId>com.google.guava</groupId> | |
| <artifactId>guava</artifactId> | |
| <exclusions> | |
| <exclusion> | |
| <groupId>com.google.guava</groupId> | |
| <artifactId>failureaccess</artifactId> | |
| </exclusion> | |
| </exclusions> | |
| </dependency> |
We need to remove it once storage and pubsub get updated to the latest guava
cc: @blakeli0
Originally posted by @diegomarquezp in #10552 (comment)
Metadata
Metadata
Assignees
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.