Bug: Start scripts for Forge contain wrong nogui parameter

[Griefed]

• What is the current behavior?

Using the genersted start scripts for Forge fails, because it tried to run --nogui when it should run nogui instead.

The error is in ServerPackHandler.class with the --nogui ending in the String definition.

• If the current behavior is a bug, please provide the steps to reproduce

• Generate a server pack for Forge

• Let ServerPackCreator generate start scripts

• Use said start scripts

• What is the expected behavior?

The start scripts should start the server.

• Please tell us about your environment:

• Your Operating System: Any
• Your Java version: Any
• Your ServerPackCreator version: Alpha.6
• Your config file:
• Your latest log file:

• Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

[Griefed]

So, running some tests yielded interesting results:

Running with --nogui worked on both Windows and Linux
Running with nogui worked on both Windows and Linux, too!

[Griefed]

False alarm.
The issue was an incomplete Forge installation, probably due to problems on Forge's side, or some sort of internet-connection problem. The start scripts themselves were fine.

[kreezxil]

alpha.8, causes the gui to popup when run with --nogui, removing the -- makes it not come up. in all my dealings, forge didn't like the --, ie when it sees it, it ignores it.

[Griefed]

So, sometimes it likes it, sometimes it doesn't.
Which Forge version was affected in your case?

[kreezxil]

1.12.2

…

On Mon, Nov 22, 2021 at 9:16 AM Griefed ***@***.***> wrote: So, sometimes it likes it, sometimes it doesn't. Which Forge version was affected in your case? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#82 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA5TJCCVLKFPDZMSVOUNTKDUNJNF5ANCNFSM5FIOYKQA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[Griefed]

Fixed. Will be available in the next alpha release. I removed the -- from the nogui argument for Forge scripts.

[Griefed]

Available as of 3.0.0-alpha.10
Feel free to report any bugs. I'll close this feature request issue. 🙂

Enjoy! 👍
Cheers,
Griefed