search

mojohaus / appassembler

https://www.mojohaus.org/appassembler/
MIT License
93 stars 49 forks source link

Issue [#48] #58

Open nezda opened 7 years ago

nezda commented 7 years ago

[#48] just swap $JAVA_OPTS and @EXTRA_JVM_ARGUMENTS@ so env is last and has a chance for its values to take precedence

(I tested this works for me using <unixScriptTemplate> configuration option suggested in http://stackoverflow.com/a/27608001/689119 to override -Xmx with $JAVA_OPTS env)

nezda commented 7 years ago

oops, didn't run all the tests - figured out how with https://github.com/mojohaus/appassembler/blob/master/.travis.yml ... fixing

nezda commented 4 years ago

any chance this can get merged?

nezda commented 3 years ago

suppose I gotta make the tests pass again 😎...