Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: PS1 incorrectly uses "JAVAARGS" variable name where "JAVA_ARGS" is used for variables.txt #460

Closed
1 task done
Frontear opened this issue May 24, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@Frontear
Copy link

Frontear commented May 24, 2023

What happened?

Essentially, the PS1 has a typo that is trying to source the "JAVAARGS" variable from the variables.txt file. This is invalid since by default, the variables.txt file has a "JAVA_ARGS".

The suspecting file can be found here:

It should be noted that the shell file uses the correct variable, solidifying this as a simple typo. Nothing major but it does break PS1 for windows users.

Line in shell confirming the correct variable:

SERVER_RUN_COMMAND="-Dlog4j2.formatMsgNoLookups=true ${JAVA_ARGS} -jar ${LAUNCHER_JAR_LOCATION} nogui"

What did you expect to happen?

Well, I expected it to source the right variable

Version

Release 4.1.2

Relevant log output

No response

Anything else you would like to add?

No response

If you are using the webservice, in which browers did you encounter this bug?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Frontear Frontear added the bug Something isn't working label May 24, 2023
@Griefed
Copy link
Owner

Griefed commented May 24, 2023

Heya Frontear,

thanks for the report. I'll fix this asap! 😊

Cheers,
Griefed

@Griefed
Copy link
Owner

Griefed commented May 24, 2023

Also a huge thanks for the detailed report!

@Griefed
Copy link
Owner

Griefed commented May 24, 2023

Fixed in 4.1.3 😊

@Griefed Griefed closed this as completed May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants