-
Notifications
You must be signed in to change notification settings - Fork 950
Fix VM argument passing by .sbtopts file and JAVA_TOOL_OPTIONS environmental variable #7393
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
Conversation
Fixes #7387. |
Thanks for the contribution! |
Happy to contribute! I see that test |
Aha, it is the |
@eed3si9n Could you approve the workflow again? I think I fixed the failing test. |
Okay, so it seems like a bunch of tests in |
…nmental variable, improve launcher script integration test setup
833abe1
to
e1314e0
Compare
@eed3si9n Should be good now, managed to execute the tests locally fine. Had to restore those python scripts in |
@eed3si9n Adding a test for |
Looks like [error] (Compile / compileIncremental) sbt.internal.inc.InvalidComponent: Expected single file for component 'org.scala-sbt-compiler-bridge_2.12-1.9.2-bin_2.12.18__61.0', found: /home/runner/.sbt/boot/scala-2.12.18/org.scala-sbt/sbt/1.9.1/org.scala-sbt-compiler-bridge_2.12-1.9.2-bin_2.12.18__61.0/org.scala-sbt-compiler-bridge_2.12-1.9.2-bin_2.12.18__61.0.jar, /home/runner/.sbt/boot/scala-2.12.18/org.scala-sbt/sbt/1.9.1/org.scala-sbt-compiler-bridge_2.12-1.9.2-bin_2.12.18__61.0/org.scala-sbt-compiler-bridge_2.12-1.9.2-bin_2.12.18__61.0-1.9.2_20230626T014432.jar You might need to temporarily add
in ci.yml. |
Sending #7394 to sidestep this issue. |
Also contains few improvements to launcher script integration test setup.