Skip to content

Commit 22322ad

Browse files
authored
Merge pull request #214 from 3arthqu4ke/fix-xvfb
Removed xvfb patch because it is not needed anymore with mc-runtime-test 2.4.2
2 parents 2f54227 + 961dd9b commit 22322ad

8 files changed

Lines changed: 6 additions & 114 deletions

File tree

.github/workflows/run-matrix-in-memory.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ jobs:
9292
- name: Copy launcher jar
9393
run: cp launcher-jar/headlessmc-launcher.jar .
9494
- name: Run the MC client
95-
uses: 3arthqu4ke/mc-runtime-test@2.4.0
95+
uses: 3arthqu4ke/mc-runtime-test@2.4.1
9696
with:
9797
mc: ${{ matrix.version.mc }}
9898
modloader: ${{ matrix.version.modloader }}

.github/workflows/run-matrix-xvfb.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ jobs:
8888
- name: Copy launcher jar
8989
run: cp launcher-jar/headlessmc-launcher.jar .
9090
- name: Run the MC client
91-
uses: 3arthqu4ke/mc-runtime-test@2.4.0
91+
uses: 3arthqu4ke/mc-runtime-test@2.4.1
9292
with:
9393
mc: ${{ matrix.version.mc }}
9494
modloader: ${{ matrix.version.modloader }}

.github/workflows/run-matrix.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ jobs:
8888
- name: Copy launcher jar
8989
run: cp launcher-jar/headlessmc-launcher.jar .
9090
- name: Run the MC client
91-
uses: 3arthqu4ke/mc-runtime-test@2.4.0
91+
uses: 3arthqu4ke/mc-runtime-test@2.4.1
9292
with:
9393
mc: ${{ matrix.version.mc }}
9494
modloader: ${{ matrix.version.modloader }}

.github/workflows/run-specific-all-modloaders.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ jobs:
6969
- name: Copy launcher jar
7070
run: cp launcher-jar/headlessmc-launcher.jar .
7171
- name: Run the MC client
72-
uses: 3arthqu4ke/mc-runtime-test@2.4.0
72+
uses: 3arthqu4ke/mc-runtime-test@2.4.1
7373
with:
7474
mc: ${{ github.event.inputs.mc }}
7575
modloader: ${{ matrix.version.modloader }}

.github/workflows/run-specific.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ jobs:
6464
run: cp headlessmc-launcher-wrapper/build/libs/headlessmc-launcher.jar .
6565

6666
- name: Run the MC client
67-
uses: 3arthqu4ke/mc-runtime-test@2.4.0
67+
uses: 3arthqu4ke/mc-runtime-test@2.4.1
6868
with:
6969
mc: ${{ github.event.inputs.mc }}
7070
modloader: ${{ github.event.inputs.modloader }}

headlessmc-launcher/src/main/java/me/earth/headlessmc/launcher/instrumentation/InstrumentationHelper.java

Lines changed: 1 addition & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,9 @@
77
import me.earth.headlessmc.launcher.instrumentation.lwjgl.HmcLwjglTransformer;
88
import me.earth.headlessmc.launcher.instrumentation.modlauncher.BootstrapLauncherTransformer;
99
import me.earth.headlessmc.launcher.instrumentation.paulscode.PaulscodeTransformer;
10-
import me.earth.headlessmc.launcher.instrumentation.xvfb.XvfbLwjglTransformer;
1110
import me.earth.headlessmc.launcher.launch.LaunchOptions;
12-
import me.earth.headlessmc.launcher.version.Library;
13-
import me.earth.headlessmc.launcher.version.family.FamilyUtil;
14-
import org.jetbrains.annotations.VisibleForTesting;
1511

1612
import java.util.ArrayList;
17-
import java.util.List;
1813

1914
@CustomLog
2015
@UtilityClass
@@ -23,7 +18,7 @@ public class InstrumentationHelper {
2318
public static final String LWJGL_JAR = "headlessmc-lwjgl.jar";
2419

2520
public static Instrumentation create(LaunchOptions options) {
26-
val transformers = new ArrayList<Transformer>(8);
21+
val transformers = new ArrayList<Transformer>(7);
2722
if (options.isLwjgl()) {
2823
transformers.add(new HmcLwjglTransformer());
2924
transformers.add(new ResourceExtractor(options.getFiles(), LWJGL_JAR));
@@ -49,32 +44,7 @@ public static Instrumentation create(LaunchOptions options) {
4944
transformers.add(new BootstrapLauncherTransformer());
5045
}
5146

52-
if (options.isXvfb()) {
53-
addXvfbTransformer(options, transformers);
54-
}
55-
5647
return new Instrumentation(transformers, options.getFiles().getBase());
5748
}
5849

59-
@VisibleForTesting
60-
static void addXvfbTransformer(LaunchOptions options, List<Transformer> transformers) {
61-
log.error("Hello?!");
62-
Boolean oldLwjgl = FamilyUtil.iterateParents(options.getVersion(), version -> {
63-
for (Library library : version.getLibraries()) {
64-
log.error("Cehcking " + library.getName());
65-
if ("org.lwjgl.lwjgl".equals(library.getPackage()) && library.getVersionNumber().startsWith("2")) {
66-
log.error("Hello? " + library);
67-
return true;
68-
}
69-
}
70-
71-
return null;
72-
});
73-
74-
if (oldLwjgl != null && oldLwjgl) {
75-
log.info("Running with old lwjgl, using xvfb transformer");
76-
transformers.add(new XvfbLwjglTransformer());
77-
}
78-
}
79-
8050
}

headlessmc-launcher/src/main/java/me/earth/headlessmc/launcher/instrumentation/xvfb/XvfbLwjglTransformer.java

Lines changed: 0 additions & 48 deletions
This file was deleted.

headlessmc-launcher/src/test/java/me/earth/headlessmc/launcher/instrumentation/InstrumentationHelperTest.java

Lines changed: 0 additions & 30 deletions
This file was deleted.

0 commit comments

Comments
 (0)