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

JLine 3.26.1 (was 3.24.1) for sbt 1.10.x #7500

Draft
wants to merge 1 commit into
base: 1.10.x
Choose a base branch
from

Conversation

SethTisue
Copy link
Member

@SethTisue SethTisue commented Feb 21, 2024

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.

@eed3si9n
Copy link
Member

[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

@eed3si9n
Copy link
Member

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.

@SethTisue
Copy link
Member Author

so I should try to use https://github.com/marketplace/actions/github-action-for-graalvm, I guess? I'm unfamiliar with Graal stuff

@SethTisue
Copy link
Member Author

SethTisue commented Feb 21, 2024

note to self: Eugene's #7501 has some changes I should maybe pull in done

@xuwei-k
Copy link
Member

xuwei-k commented Feb 23, 2024

@hvesalai
Copy link
Contributor

@eed3si9n do you think this would be back-ported to 1.9.x also?

@eed3si9n
Copy link
Member

Not sure, but we should first get it working for 1.10.x?

@SethTisue
Copy link
Member Author

SethTisue commented Mar 15, 2024

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

@SethTisue
Copy link
Member Author

about Graal, Eugene has inquired at jline/jline3#954

@SethTisue
Copy link
Member Author

because JDK 23 isn't out yet, our normal channels to get GraalVM, like Coursier or SDKMAN do not list graal 23 yet

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...

@SethTisue
Copy link
Member Author

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.

@mkurz
Copy link
Member

mkurz commented May 6, 2024

Didn't make it into sbt 1.10.0 😢 Hoping for a patch release...

@SethTisue SethTisue changed the title JLine 3.25.1 (was 3.24.1) for sbt 1.10.0 JLine 3.25.1 (was 3.24.1) for sbt 1.10.x May 6, 2024
@SethTisue SethTisue changed the title JLine 3.25.1 (was 3.24.1) for sbt 1.10.x JLine 3.26.1 (was 3.24.1) for sbt 1.10.x May 6, 2024
@eed3si9n
Copy link
Member

eed3si9n commented May 6, 2024

I tried locally before releasing sbt 1.10.0, and afaict the situation has not changed with the new JLine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants