Skip to content

Commit 3d12662

Browse files
committed
Fix GrEclipse for 4.18 through 4.26
1 parent 657e0bc commit 3d12662

File tree

4 files changed

+22
-16
lines changed

4 files changed

+22
-16
lines changed

lib-extra/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ version = rootProject.spotlessChangelog.versionNext
77
apply from: rootProject.file('gradle/java-setup.gradle')
88
apply from: rootProject.file('gradle/java-publish.gradle')
99

10-
String VER_SOLSTICE = '0.19.1'
10+
String VER_SOLSTICE = '0.19.2'
1111
dependencies {
1212
api project(':lib')
1313
// misc useful utilities

lib-extra/src/groovy/java/com/diffplug/spotless/extra/glue/groovy/GrEclipseFormatterStepImpl.java

Lines changed: 17 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import java.io.ByteArrayInputStream;
1919
import java.io.ByteArrayOutputStream;
2020
import java.io.IOException;
21+
import java.nio.file.Files;
2122
import java.util.ArrayList;
2223
import java.util.Collections;
2324
import java.util.List;
@@ -38,8 +39,10 @@
3839
import org.eclipse.jface.text.Document;
3940
import org.eclipse.jface.text.IDocument;
4041
import org.eclipse.jface.text.TextSelection;
42+
import org.eclipse.osgi.internal.location.EquinoxLocations;
4143
import org.eclipse.text.edits.TextEdit;
4244
import org.osgi.framework.Constants;
45+
import org.slf4j.LoggerFactory;
4346

4447
import dev.equo.solstice.NestedJars;
4548
import dev.equo.solstice.ShimIdeBootstrapServices;
@@ -49,20 +52,22 @@
4952
/** Spotless-Formatter step which calls out to the Groovy-Eclipse formatter. */
5053
public class GrEclipseFormatterStepImpl {
5154
static {
52-
System.setProperty("org.slf4j.simpleLogger.defaultLogLevel", "info");
5355
NestedJars.setToWarnOnly();
5456
NestedJars.onClassPath().confirmAllNestedJarsArePresentOnClasspath(CacheLocations.nestedJars());
55-
56-
var solstice = Solstice.findBundlesOnClasspath();
57-
solstice.warnAndModifyManifestsToFix();
58-
var props = Map.of("osgi.nl", "en_US",
59-
Constants.FRAMEWORK_STORAGE_CLEAN, Constants.FRAMEWORK_STORAGE_CLEAN_ONFIRSTINIT);
60-
solstice.openShim(props);
61-
ShimIdeBootstrapServices.apply(props, solstice.getContext());
62-
solstice.start("org.apache.felix.scr");
63-
solstice.startAllWithLazy(false);
64-
// solstice.start("org.eclipse.core.resources");
65-
solstice.start("org.codehaus.groovy.eclipse.core");
57+
try {
58+
var solstice = Solstice.findBundlesOnClasspath();
59+
solstice.warnAndModifyManifestsToFix();
60+
var props = Map.of("osgi.nl", "en_US",
61+
Constants.FRAMEWORK_STORAGE_CLEAN, Constants.FRAMEWORK_STORAGE_CLEAN_ONFIRSTINIT,
62+
EquinoxLocations.PROP_INSTANCE_AREA, Files.createTempDirectory("spotless-groovy").toAbsolutePath().toString());
63+
solstice.openShim(props);
64+
ShimIdeBootstrapServices.apply(props, solstice.getContext());
65+
solstice.start("org.apache.felix.scr");
66+
solstice.startAllWithLazy(false);
67+
solstice.start("org.codehaus.groovy.eclipse.core");
68+
} catch (IOException e) {
69+
throw new RuntimeException(e);
70+
}
6671
}
6772

6873
/**

lib-extra/src/main/java/com/diffplug/spotless/extra/EquoBasedStepBuilder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ EquoBasedStepBuilder.State get() throws Exception {
8888
var query = model(formatterVersion).query(P2ClientCache.PREFER_OFFLINE, P2QueryCache.ALLOW);
8989
var classpath = new ArrayList<File>();
9090
var mavenDeps = new ArrayList<String>();
91-
mavenDeps.add("dev.equo.ide:solstice:0.19.1");
91+
mavenDeps.add("dev.equo.ide:solstice:0.19.2");
9292
mavenDeps.add("com.diffplug.durian:durian-swt.os:4.1.1");
9393
mavenDeps.add("org.slf4j:slf4j-simple:1.7.36");
9494
mavenDeps.addAll(query.getJarsOnMavenCentral());

lib-extra/src/test/java/com/diffplug/spotless/extra/groovy/GrEclipseFormatterStepTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,13 @@
2323
import com.diffplug.spotless.TestProvisioner;
2424
import com.diffplug.spotless.extra.eclipse.EquoResourceHarness;
2525

26-
class GrEclipseFormatterStepTest extends EquoResourceHarness {
26+
public class GrEclipseFormatterStepTest extends EquoResourceHarness {
2727
private final static String INPUT = "class F{ def m(){} }";
2828
private final static String EXPECTED = "class F{\n\tdef m(){}\n}";
2929

3030
public GrEclipseFormatterStepTest() {
3131
super(GrEclipseFormatterStep.createBuilder(TestProvisioner.mavenCentral()), INPUT, EXPECTED);
32+
System.setProperty("org.gradle.logging.level", "info");
3233
}
3334

3435
@ParameterizedTest
@@ -38,6 +39,6 @@ void formatWithVersion(String version) throws Exception {
3839
}
3940

4041
private static Stream<String> formatWithVersion() {
41-
return Stream.of("4.21", GrEclipseFormatterStep.defaultVersion());
42+
return Stream.of("4.18", GrEclipseFormatterStep.defaultVersion());
4243
}
4344
}

0 commit comments

Comments
 (0)