Skip to content

Commit 238ee28

Browse files
committed
Revert "Remove the workaround for Guava issue 6801"
This reverts commit 131829e.
1 parent 131829e commit 238ee28

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/GradleProvisioner.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2016-2025 DiffPlug
2+
* Copyright 2016-2024 DiffPlug
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -26,6 +26,7 @@
2626
import org.gradle.api.artifacts.Configuration;
2727
import org.gradle.api.artifacts.ConfigurationContainer;
2828
import org.gradle.api.artifacts.dsl.DependencyHandler;
29+
import org.gradle.api.attributes.Attribute;
2930
import org.gradle.api.attributes.Bundling;
3031
import org.gradle.api.attributes.Category;
3132
import org.gradle.api.initialization.dsl.ScriptHandler;
@@ -124,6 +125,9 @@ private static Provisioner forConfigurationContainer(Project project, Configurat
124125
config.attributes(attr -> {
125126
attr.attribute(Category.CATEGORY_ATTRIBUTE, project.getObjects().named(Category.class, Category.LIBRARY));
126127
attr.attribute(Bundling.BUNDLING_ATTRIBUTE, project.getObjects().named(Bundling.class, Bundling.EXTERNAL));
128+
// TODO: This is a copy-paste from org.gradle.api.attributes.java.TargetJvmEnvironment which is added in Gradle 7.0, remove this once we drop support for Gradle 6.x.
129+
// Add this attribute for resolving Guava dependency, see https://github.com/google/guava/issues/6801.
130+
attr.attribute(Attribute.of("org.gradle.jvm.environment", String.class), "standard-jvm");
127131
});
128132
return config.resolve();
129133
} catch (Exception e) {

testlib/src/main/java/com/diffplug/spotless/TestProvisioner.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import org.gradle.api.artifacts.Dependency;
3232
import org.gradle.api.artifacts.ResolveException;
3333
import org.gradle.api.artifacts.dsl.RepositoryHandler;
34+
import org.gradle.api.attributes.Attribute;
3435
import org.gradle.api.attributes.Bundling;
3536
import org.gradle.api.attributes.Category;
3637
import org.gradle.testfixtures.ProjectBuilder;
@@ -73,6 +74,9 @@ private static Provisioner createWithRepositories(Consumer<RepositoryHandler> re
7374
config.attributes(attr -> {
7475
attr.attribute(Category.CATEGORY_ATTRIBUTE, project.getObjects().named(Category.class, Category.LIBRARY));
7576
attr.attribute(Bundling.BUNDLING_ATTRIBUTE, project.getObjects().named(Bundling.class, Bundling.EXTERNAL));
77+
// TODO: This is a copy-paste from org.gradle.api.attributes.java.TargetJvmEnvironment which is added in Gradle 7.0, remove this once we drop support for Gradle 6.x.
78+
// Add this attribute for resolving Guava dependency, see https://github.com/google/guava/issues/6801.
79+
attr.attribute(Attribute.of("org.gradle.jvm.environment", String.class), "standard-jvm");
7680
});
7781
try {
7882
return config.resolve();

0 commit comments

Comments
 (0)