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

Bugfix java 17 support per atlassian upstream #413

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

valentino-aguiar-gsa
Copy link
Contributor

See https://jira.atlassian.com/browse/JRASERVER-76224 for details on this. This snippet was taken directly from the setenv.sh script from JIRA 9.11.1.

Pull Request (PR) description

This PR updates setenv.sh with the fix from atlassian's upstream setenv.sh script. It simply checks if java 17 is in use, and if so, then it includes the JVM_OPENS parameters.

This Pull Request (PR) fixes the following issues

Fixes #412

@valentino-aguiar-gsa
Copy link
Contributor Author

valentino-aguiar-gsa commented Jan 12, 2024

The failing check failing appears to be unrelated (init.pp hasn't been modified by this pr).

@h-haaks
Copy link
Contributor

h-haaks commented Feb 12, 2024

@valentino-aguiar-gsa could you please rebase your branch?

@valentino-aguiar-gsa
Copy link
Contributor Author

Hi, rebased. Didn't notice, thanks!

@valentino-aguiar-gsa
Copy link
Contributor Author

Will adjust to only include the 1 commit shortly

@bastelfreak
Copy link
Member

@valentino-aguiar-gsa can you please rebase against our master branch to get rid of the merge commit?

See https://jira.atlassian.com/browse/JRASERVER-76224 for details on this. This snippet was taken directly from the setenv.sh script from JIRA 9.11.1.
@valentino-aguiar-gsa
Copy link
Contributor Author

Sorry about that, all sorted. Note to self, dont use the 'sync' option in github web ui.

@h-haaks h-haaks merged commit eaa728d into voxpupuli:master Feb 12, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JVM_OPENS not set for JIRA Java 17 support
3 participants