-
Notifications
You must be signed in to change notification settings - Fork 334
Update to GraalVM 25 #14019
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
Update to GraalVM 25 #14019
Conversation
Otherwise engine-runner NI build fails.
Otherwise, NI build fails with "Substitution target for com.oracle.svm.core.posix.darwin.Target_java_util_prefs_FileSystemPreferences is not loaded".
|
NI build of So just by upgrading to GraalVM 25.0.0, the NI size decreased by roughly 6%. |
|
Failure in Fixed by bumping Mockito version in 6ac2d05 |
Which supports Java 25.
#14019 is blocked by this - the current google java format version is failing because it tries to call remove method from JDK - see https://github.com/enso-org/enso/actions/runs/17916525266/job/50940077628?pr=14019#step:5:245 # Important Notes The only notable change is in ff3fec5 . Every thing else is just automatic `sbt javafmtAll` formatting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
enso.{def,exp,lib} files must not be in the top directory. It's a packaging thing so probably somewhere in the distribution/resources directory?
Instructions to generate them should also be added either as a comment in the files (probably not possible) or in some docs related to distribution. PR comment is not really enough.
Right. PR comment is certainly not enough.... I cannot find it right now. Done in 0e2e247 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- CI seems to be green
- the hack with
enso.libhas been reviewed by Hubert - let's move on and switch to LTS version of GraalVM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- also generated from
enso.def - https://github.com/enso-org/enso/pull/14019/files#r2461574134
| @@ -0,0 +1,40 @@ | |||
| NAME "enso.exe" | |||
|
|
|||
| ; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are the instructions to generate enso.exp and enso.def files.
|
the test itself succeeds when executed locally in isolation with |
|
There is also failure which is real, isolated and reproducible. Related to some trickery around temporary directory on Mac. Fixed by 8031779 |
|
Running the tests on Mac locally: e.g. things are working and I believe we can integrate regardless of state of Mac CI (which was already almost green). Adding ready to merge label. |
Closes #13918
Pull Request Description
Update GraalVM from no longer supported 24.0.1 to 25.0.0 - e.g. the LTS version.
Important Notes
Highlights from GraalVM 25 release notes:
Arena.ofShared-H:+JDWPflag.Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
or the Snowflake database integration, a run of the Extra Tests has been scheduled.