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

4.21.1 release doesn't embed JDK anymore #1187

Closed
martinlippert opened this issue Feb 11, 2024 · 3 comments
Closed

4.21.1 release doesn't embed JDK anymore #1187

martinlippert opened this issue Feb 11, 2024 · 3 comments
Assignees
Labels
for: eclipse something that is specific for Eclipse type: bug type: install-issue

Comments

@martinlippert
Copy link
Member

The JustJ JDK that we usually ship pre-installed into the Eclipse based distribution is not there anymore. It is also missing from the snapshot builds (so probably nothing specific to the release build).

This causes the launcher to pick up a (more or less) random JDK or does nothing if it can't find a pre-installed JDK (like in a new machine). In my case, I have a few JDKs installed via sdkman, but that only works if I launch STS from the command line. Double clicking the app icon doesn't work anymore.

@martinlippert
Copy link
Member Author

I haven't noticed this during the regular release testing probably because the launcher picked up a JDK from my machine that was around, which is not the case anymore on a different machine. We need to fix this to make sure that the IDE runs on the JDK that we embed and not a random other one.

@martinlippert
Copy link
Member Author

A few more observations from looking into this:

  • The JDK is not embedded anymore in macOS only. The Linux and Windows builds still includes the JDK.
  • The build logs indicate that the installation of the JustJ feature happens for all five platform builds and completes fine.
  • Running the build locally also results in the JDK being included.

I have no idea yet why the distributions for macOS that you download from the nightly download page do not include the JDK anymore (same for the 4.21.1 release). Strange...

@BoykoAlex
Copy link
Contributor

Fixed with: aa250f7
Installed JustJ into .app/Contents/Eclipse - worked fine in both Ubuntu and OSX (via local build), therefore, decided to go ahead with this solution (i.e. without platform specific paths)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for: eclipse something that is specific for Eclipse type: bug type: install-issue
Projects
None yet
Development

No branches or pull requests

2 participants