[Install Bug]: Fabric - Minecraft Java

[59n]

Panel Version

1.0.0-beta27

Wings Version

1.0.0-beta18

Service

minecraft/java/fabric

Modified

Yes, I modified the egg (will provide details below)

Expected Behavior

You expect to pull the egg JSON from github and it to work out of the box

Actual Behavior

My server was not setup with a ram limit nor are my wings, so the startup variable "-Xmx{{SERVER_MEMORY}" was defaulting to 0 which was causing startup errors, I have removed it and it fixed the issue

Steps To Reproduce

Step 1, remove -Xmx{{SERVER_MEMORY} variable from default and add it to a optional startup script
Step 2, optionally, replace with different variable such as -XX:MaxRAMPercentage=95.0
Step 3, start server and works

Install logs

nan

[SirEndii]

Iirc the logs even tell you that this won't work
But iirc only the neoforge or forge egg show that

[rmartinoscar]

This is the most common java error Xmx cannot be 0 so either set an actual ram limit or manually edit the egg on your side to remove the argument but this won't be changed for any egg that has support for any Java version prior to 10.

Java 8 doesn't have -XX:MaxRAMPercentage so we'd need to detect java version and use something like -XX:MaxRAMFraction instead.

[midget3111]

I'm also encountering this issue, but changing the variable doesn't seem to sort it. Wing has no specified ram limit.

[59n]

I have managed to find a working fabric egg, the current one is using a minimalistic docker image, try this and see if it works

fabric.json

[quill-g]

I have managed to find a working fabric egg, the current one is using a minimalistic docker image, try this and see if it works

fabric.json

This egg works great! Except for the eula accept popup which doesn't for some reason. I had to manually accept the eula in the eula.txt file.
I can confirm the fabric egg in this repo is completely nonfunctional, it seems like it has had a lot less care put into it compared to the other minecraft eggs.