-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
jetty.sh does not read JAVA_OPTIONS anymore #8152
Comments
The removal of JAVA_OPTIONS was because of the nature of how configurations could fork a JVM. The changes in jetty.sh reflect this, and an effort to build the true command line based on your Configuration. |
OK, thanks for the information. I'm not sure it will work for us, since we are sourcing external config files to get the option list, but I will look at the doc now that I know the entry point. Thanks for the pointer. |
Co-authored-by: Francois @fanf42 Armand <[email protected]>
Jetty version(s)
Jetty 10.x since commit c898666, ie
git tag --contains=c89866682c14ca1da0ce482eeca0953375b25da8
jetty-10.0.8
jetty-10.0.9
jetty-11.0.8
jetty-11.0.9
Java version/vendor
(use: java -version)
Any
OS type/version
Any
Description
Since commit c898666,
JAVA_OPTIONS
is not read anywhere injetty.sh
script.I believe the goal of the refactoring was to add
JETTY_SYS_PROPS
and then addJAVA_OPTIONS
inRUN_ARGS
, but it was not doneThe text was updated successfully, but these errors were encountered: