GregTech CE Unofficial

GregTech CE Unofficial

620k Downloads

GTCEu JEI Integration breaks on Java 21 (CleanroomMC)

Closed this issue · 7 comments

commented

GregTech CEu Version

2.8.10

Modpack Used

SuperSymmetry, modified to be a proper Cleanroom pack with Java 21.

Addons Installed

Gregicality Multiblocks 1.2.11
GregicProbe 1.4.2
GregTech Food Option 1.12.5

Environment

Multiplayer

New Worlds

Yes

Cross-Mod Interaction

Yes

Expected Behavior

GregTech Recipes show when clicked in JEI and the console does not log errors related to loading the JEI plugin.

Actual Behavior

Clicking on GregTech recipes in HEI does absolutely nothing. The console shows a few peculiar lines:

[Foundation]: gregtech.asm.visitors.JEIVisitor can't handle Java 21 class, please port the mod (if you are the author)!
[Had Enough Items]: Failed to register mod plugin: class gregtech.integration.jei.JustEnoughItemsModule

Steps to Reproduce

Get a fresh Cleanroom install. Very easy to do with PrismLauncher. Import their instance, set the Java path to a Java 21 JDK, make sure Fugue mod is installed. Then add HEI & GTCEu.

Additional Information

https://mclo.gs/rBrafEY

commented

We do not explicitly support Cleanroom Loader; if this issue occurs with forge please create a new issue.

commented

install fugue plz
also should report crl related issues on their repos

commented

install fugue plz also should report crl related issues on their repos

It is installed. This is an issue that can't be fixed by Fugue, requiring the mod developers to fix compatibility with Java 21 classes.

Edit: if you had actually read the error, you would be able to see that it was literally REPORTED BY CLEANROOM ITSELF.

commented

install fugue plz also should report crl related issues on their repos

It is installed. This is an issue that can't be fixed by Fugue, requiring the mod developers to fix compatibility with Java 21 classes.

Edit: if you had actually read the error, you would be able to see that it was literally REPORTED BY CLEANROOM ITSELF.

oh yeah nvm I can't read
seems like this could be caused by alfheim like that in #2775

also like the line [Foundation]: gregtech.asm.visitors.JEIVisitor can't handle Java 21 class, please port the mod (if you are the author)!
is just a warning (you may see losts of that), it should't cause much issues normally

commented

install fugue plz also should report crl related issues on their repos

It is installed. This is an issue that can't be fixed by Fugue, requiring the mod developers to fix compatibility with Java 21 classes.
Edit: if you had actually read the error, you would be able to see that it was literally REPORTED BY CLEANROOM ITSELF.

oh yeah nvm I can't read seems like this could be caused by alfheim like that in #2775

I don't just randomly open issues on peoples repos without doing my due diligence. It's a completely unrelated issue. I'll probably have to open a PR later. Right now I'm at work, I'll give it a shot later I guess

commented

install fugue plz also should report crl related issues on their repos

It is installed. This is an issue that can't be fixed by Fugue, requiring the mod developers to fix compatibility with Java 21 classes.
Edit: if you had actually read the error, you would be able to see that it was literally REPORTED BY CLEANROOM ITSELF.

oh yeah nvm I can't read seems like this could be caused by alfheim like that in #2775

I don't just randomly open issues on peoples repos without doing my due diligence. It's a completely unrelated issue. I'll probably have to open a PR later. Right now I'm at work, I'll give it a shot later I guess

I don't quite think it's a gregtech issue tbh
I ran susy on crl myself and it works well (Edit: the modlist in your log isn't even the original susy modlist)
You should really try with only gregtech and necessary mods instead, if that doesn't crash, try finding out a minimum combination of mods that could replicate this.

Also it's one of our goals to maintain crl compatibiltiy for susy, feel free to ping me at susy cord #bug-discussions channel
if you aren't there yet (since I can't find you by searching your dc name)

commented

install fugue plz also should report crl related issues on their repos

It is installed. This is an issue that can't be fixed by Fugue, requiring the mod developers to fix compatibility with Java 21 classes.
Edit: if you had actually read the error, you would be able to see that it was literally REPORTED BY CLEANROOM ITSELF.

oh yeah nvm I can't read seems like this could be caused by alfheim like that in #2775

I don't just randomly open issues on peoples repos without doing my due diligence. It's a completely unrelated issue. I'll probably have to open a PR later. Right now I'm at work, I'll give it a shot later I guess

I don't quite think it's a gregtech issue tbh
I ran susy on crl myself and it works well (Edit: the modlist in your log isn't even the original susy modlist)
You should really try with only gregtech and necessary mods instead, if that doesn't crash, try finding out a minimum combination of mods that could replicate this.

Also it's one of our goals to maintain crl compatibiltiy for susy, feel free to ping me at susy cord #bug-discussions channel
if you aren't there yet (since I can't find you by searching your dc name)

I apologize for being a bit abrasive yesterday, I was being yelled at by the players I was hosting the modpack for, I was at work and was not thrilled to have to guide people that can barely use a computer to revert an instance to java 8 non-cleanroom.

According to them, reverting to Java 8 didn't fix anything either. So that wasn't it. I'll try the Alfheim thing again later although I tried that yesterday and it didn't fix anything.

The reason its not the original modlist is because I always try to make whatever modpack they wanna play run on their potato rigs.

Again, I admire your patience. If anyone talked to me like I just did on a github issue I'd have lost my shit. I might join the discord after work if they still want to play the pack. Most of the time they get bored 3 days in and start something else.