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
JLine 3.26.1 (was 3.24.1) for sbt 1.10.x #7500
base: 1.10.x
Are you sure you want to change the base?
Conversation
[info] /home/runner/.cache/coursier/jvm/graalvm-java11@22.2.0/bin/native-image -cp /home/runner/work/sbt/sbt/client/target/native-image-internal/manifest.jar --no-fallback --initialize-at-run-time=sbt.client -H:IncludeResourceBundles=jline.console.completer.CandidateListCompletionHandler -H:+ReportExceptionStackTraces -H:-ParseRuntimeOptions -H:Name=/home/runner/work/sbt/sbt/client/target/bin/sbtn sbt.client.Client /home/runner/work/sbt/sbt/client/target/bin/sbtn
WARNING: Unknown module: org.graalvm.nativeimage.llvm specified to --add-exports
WARNING: Unknown module: org.graalvm.nativeimage.llvm specified to --add-exports
WARNING: Unknown module: org.graalvm.nativeimage.llvm specified to --add-exports
Error: Could not find option 'UnlockExperimentalVMOptions' from 'jar:file:///home/runner/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-terminal-jna/3.25.1/jline-terminal-jna-3.25.1.jar!/META-INF/native-image/org.jline/jline-terminal-jna/native-image.properties'. Use -H:PrintFlags= to list all available options.
Could not find option 'UnlockExperimentalVMOptions' from 'jar:file:///home/runner/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-native/3.25.1/jline-native-3.25.1.jar!/META-INF/native-image/org.jline/jline-native/native-image.properties'. Use -H:PrintFlags= to list all available options.
Could not find option 'UnlockExperimentalVMOptions' from 'jar:file:///home/runner/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-terminal/3.25.1/jline-terminal-3.25.1.jar!/META-INF/native-image/org.jline/jline-terminal/native-image.properties'. Use -H:PrintFlags= to list all available options.
Could not find option 'UnlockExperimentalVMOptions' from 'jar:file:///home/runner/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-terminal-jansi/3.25.1/jline-terminal-jansi-3.25.1.jar!/META-INF/native-image/org.jline/jline-terminal-jansi/native-image.properties'. Use -H:PrintFlags= to list all available options.
Error: Image build request failed with exit status 1
[error] native-image command failed with exit code '1'
[error] (sbtClientProj / nativeImage) native-image command failed with exit code '1'
[error] Total time: 30 s, completed Feb 21, 2024 1:21:47 AM |
Per jline/jline3@2e9c4f9 JLine 3 use Graal Native Image 23.1.1, but because JDK 23 isn't out yet, our normal channels to get GraalVM, like Coursier or SDKMAN do not list graal 23 yet. |
so I should try to use https://github.com/marketplace/actions/github-action-for-graalvm, I guess? I'm unfamiliar with Graal stuff |
|
@eed3si9n do you think this would be back-ported to 1.9.x also? |
Not sure, but we should first get it working for 1.10.x? |
sorry I haven't had time lately to push this PR forward, but I haven't forgotten, either. (I also owe the scala/scala repo a similar PR.) once it's merged, it would be good to have an sbt 1.10.0-M2 that we could test with before maybe back porting to 1.9.x — feels like it's been a while since M1 |
about Graal, Eugene has inquired at jline/jline3#954 |
According to https://www.graalvm.org/release-calendar/ , "Note: Beginning with JDK 21, the release numbering scheme for language runtimes differs from the one used for GraalVM releases" So I think it's not relevant that JDK 23 isn't out yet, unless I'm confused. As for how to actually proceed, I've been trying to figure that out... |
JLine 3.26.0 is out — I'll update this PR to use it instead. It's supposed to fix the Graal Native Image issue. |
Didn't make it into sbt 1.10.0 😢 Hoping for a patch release... |
I tried locally before releasing sbt 1.10.0, and afaict the situation has not changed with the new JLine. |
wdyt @eed3si9n might it be this simple?
at https://github.com/jline/jline3/releases/tag/jline-parent-3.25.0 it says: "Merge Jansi library into JLine" and I'm unclear on whether that's somehow problematic for us here.
fyi @hamzaremmal
if this works out I can forward-port to
develop
branch, of course.