Skip to content

Conversation

ptrdom
Copy link
Contributor

@ptrdom ptrdom commented Oct 5, 2023

Also contains few improvements to launcher script integration test setup.

@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

Fixes #7387.

@eed3si9n
Copy link
Member

eed3si9n commented Oct 5, 2023

Thanks for the contribution!

@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

Happy to contribute!

I see that test SbtScriptTest sbt -D arguments is hanging on all OS, I can replicate that locally, but only if I execute all tests - executing only sbt -D arguments makes it succeed. Needs more investigation, probably improvements to the test suite too.

@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

Aha, it is the sbt --sbt-version that actually hangs. It seems it just boots the sbt and process does not exit. I wonder how did that test managed to pass in 1.9.x and 1.10.x branches on the remote.

@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

@eed3si9n Could you approve the workflow again? I think I fixed the failing test.

@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

Okay, so it seems like a bunch of tests in SbtScriptTest just hang because spawned sbt process does not exit. Will investigate further.

@ptrdom ptrdom force-pushed the windows-launcher-vm-options-fix branch from 833abe1 to e1314e0 Compare October 5, 2023 18:27
@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 5, 2023

@eed3si9n Should be good now, managed to execute the tests locally fine. Had to restore those python scripts in itegration-test/bin, no idea what is so magic about them, but passing them as JAVA_CMD both makes tests pass and execute fast.

@ptrdom ptrdom changed the title Fix VM argument passing by .sbtopts file and JAVA_TOOL_OPTIONS environmental variable in Windows Fix VM argument passing by .sbtopts file and JAVA_TOOL_OPTIONS environmental variable Oct 6, 2023
@ptrdom
Copy link
Contributor Author

ptrdom commented Oct 6, 2023

@eed3si9n Adding a test for JAVA_TOOL_OPTIONS made linux/mac runners fail, so I pushed a fix for sbt.sh too.

@eed3si9n
Copy link
Member

eed3si9n commented Oct 6, 2023

Looks like $HOME/.sbt/boot/ directory got corrupted?

[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

rm -rf $HOME/.sbt/boot/

in ci.yml.

@eed3si9n
Copy link
Member

eed3si9n commented Oct 6, 2023

Sending #7394 to sidestep this issue.

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

Successfully merging this pull request may close these issues.

2 participants