FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: Permit overriding of JSA file location using ENV variable by aleksandrs-ledovskis · Pull Request #9073 · jruby/jruby · GitHub

/ jruby Public

fix: Permit overriding of JSA file location using ENV variable - #9073

Merged
headius merged 1 commit into
jruby:masterfrom
aleksandrs-ledovskis:fix/jruby_jsa_file_override
Dec 2, 2025
Merged

fix: Permit overriding of JSA file location using ENV variable#9073
headius merged 1 commit into
jruby:masterfrom
aleksandrs-ledovskis:fix/jruby_jsa_file_override

Conversation

Copy link
Copy Markdown
Contributor

The construct introduced in #8708 to "Allow overriding default JSA file location" doesn't work.

E.g. running docker run --rm -e 'JRUBY_JSA=/tmp' jruby:10.0.2.0 ruby --version fails with /opt/jruby/bin/ruby: 814: /opt/jruby/bin/jruby.sh: jruby_jsa_file: is read only.

The fix seems to be trivial.

enebo requested a review from mrnoname1000 November 17, 2025 15:21

headius commented Nov 26, 2025

Copy link
Copy Markdown
Member

Looks good to me. Waiting for review by @mrnoname1000.

headius added this to the JRuby 10.0.3.0 milestone Nov 26, 2025

mrnoname1000 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

headius merged commit 87dbf7d into jruby:master Dec 2, 2025
77 checks passed

headius commented Dec 2, 2025

Copy link
Copy Markdown
Member

Thank you!

aleksandrs-ledovskis deleted the fix/jruby_jsa_file_override branch December 14, 2025 12:26
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL