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.

Sign up for GitHub

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

Jump to bottom

JAVA_HOME not set properly when "system" is set to be global #422

Open
smlchen opened this issue Feb 20, 2024 · 0 comments · May be fixed by #430
Open

JAVA_HOME not set properly when "system" is set to be global #422

smlchen opened this issue Feb 20, 2024 · 0 comments · May be fixed by #430
Milestone
0.5.8

Comments

@smlchen
Copy link

smlchen commented Feb 20, 2024 •
edited 

> java --version
openjdk 21.0.2 2024-01-16 LTS
OpenJDK Runtime Environment Temurin-21.0.2+13 (build 21.0.2+13-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.2+13 (build 21.0.2+13-LTS, mixed mode)
> /usr/libexec/java_home
/Library/Java/JavaVirtualMachines/temurin-21.jdk/Contents/Home
> jenv --version
jenv 0.5.6
> jenv doctor
[OK]    JAVA_HOME variable probably set by jenv PROMPT
[OK]    Java binaries in path are jenv shims
[OK]    Jenv is correctly loaded
> jenv versions
* system (set by /Users/sachen/.jenv/version)
  corretto64-1.8.0.342
  oracle64-17.0.5
  temurin64-11.0.17
> echo "${JAVA_HOME}"
/Users/sachen/.jenv/versions/system
> ls /Users/sachen/.jenv/versions/system
"/Users/sachen/.jenv/versions/system": No such file or directory (os error 2)

My system Java was installed via homebrew
@smlchen smlchen changed the title JAVA_HOME not set property when "system" is set to be global JAVA_HOME not set properly when "system" is set to be global Feb 20, 2024
@andrewflbarnes andrewflbarnes mentioned this issue Mar 16, 2024
Merged
andrewflbarnes added a commit that referenced this issue Mar 16, 2024
@andrewflbarnes
Merge pull request #350 from fwcd/skip-javahome-system-jdk
56b2563 
fix: Skip setting `JAVA_HOME` when using system JDK #422
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 16, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
f38c8a4
@andrewflbarnes andrewflbarnes linked a pull request Mar 16, 2024 that will close this issue
Open
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 16, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
ef6540d
@andrewflbarnes andrewflbarnes mentioned this issue Mar 16, 2024
Open
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 16, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
7c5a5bb
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 18, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
5d6239c
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 19, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
31edd94
@andrewflbarnes andrewflbarnes mentioned this issue Mar 19, 2024
Open
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 20, 2024
@andrewflbarnes
fix: jenv#422 prevent export hook setting JAVA_HOME when system
26dcd20
@andrewflbarnes andrewflbarnes added this to the 0.5.8 milestone Mar 21, 2024
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 22, 2024
@andrewflbarnes
feat: jenv#422 handle system version better
47ca7f2 
Use the current env JAVA_HOME when using the system version, or use the
JAVA_HOME from when it was last set.
andrewflbarnes added a commit to andrewflbarnes/jenv that referenced this issue Mar 30, 2024
@andrewflbarnes
feat: jenv#422 handle system version better
08ab277 
Use java from the path with jenv shims and locations removed.
Set JAVA_HOME to be the linked java home from above, if it exists,
otherwise search for OS utility programs which can pinpoint the javahome
like libexec java_home or alternatives.
Fallback to whatever JAVA_HOME is currently set to if it can't otherwise
be resolved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.5.8
Development

Successfully merging a pull request may close this issue.

fix: #422 prevent export hook setting JAVA_HOME when system andrewflbarnes/jenv
2 participants
@andrewflbarnes @smlchen