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

Updating Espresso support & docs to match GraalVM for JDK21 #8641

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

JaroslavTulach
Copy link
Member

@JaroslavTulach JaroslavTulach commented Dec 28, 2023

Pull Request Description

Updating instructions to use Enso with Espresso.

Important Notes

NI runner can be built, but doesn't initialize Espresso properly.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • All code follows the
    Scala,
    Java,
  • All code has been tested:
    • Instructions verified manually.

@JaroslavTulach JaroslavTulach force-pushed the wip/jtulach/Espresso21 branch 2 times, most recently from f603a33 to 8b89ec8 Compare December 29, 2023 05:07
@JaroslavTulach JaroslavTulach added the CI: No changelog needed Do not require a changelog entry for this PR. label Dec 29, 2023
@@ -523,24 +523,24 @@ impl RunContext {
}


// if build_native_runner {
Copy link
Member Author

Choose a reason for hiding this comment

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

This code has been introduced by

Looking at the commits in the PR it looks like @mwu-tow tried to add the test, but failed. Can we enable such test now? Do you know how to fix the failure, @mwu-tow?

Can we also expand the test to run twice and once use ENSO_JAVA=espresso, @mwu-tow?

@JaroslavTulach JaroslavTulach changed the title Use module layer to check for Espresso Updating Espresso support & docs to match GraalVM for JDK21 Dec 29, 2023
@JaroslavTulach JaroslavTulach added the CI: Clean build required CI runners will be cleaned before and after this PR is built. label Jan 2, 2024
docs/infrastructure/native-image.md Show resolved Hide resolved
@JaroslavTulach JaroslavTulach merged commit 542357a into develop Jan 5, 2024
34 checks passed
@JaroslavTulach JaroslavTulach deleted the wip/jtulach/Espresso21 branch January 5, 2024 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-compiler CI: Clean build required CI runners will be cleaned before and after this PR is built. CI: No changelog needed Do not require a changelog entry for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants