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

JDK9: -Xbootclasspath/p is no longer a supported #381

Closed
botic opened this issue Sep 25, 2017 · 4 comments
Closed

JDK9: -Xbootclasspath/p is no longer a supported #381

botic opened this issue Sep 25, 2017 · 4 comments
Labels
Milestone

Comments

@botic
Copy link
Member

botic commented Sep 25, 2017

Philipps-MacBook-Pro:~ philipp$ ringo -v
-Xbootclasspath/p is no longer a supported option.
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.
@botic botic added the java label Sep 25, 2017
@jgabios
Copy link

jgabios commented Nov 4, 2017

I changed -Xbootclasspath/p with -Xbootclasspath/a and it went passed. However I am stuck with: "java.lang.RuntimeException: No Context associated with current Thread" . It seems this one is getting solved here: #383

botic added a commit that referenced this issue Nov 4, 2017
This is a first workaround for JDK9 and targets #381
@botic
Copy link
Member Author

botic commented Nov 4, 2017

The --patch-module option replaces the -Xbootclasspath:/p option, which has been removed (see below).
The --patch-module option is intended only for testing and debugging. Its use in production settings is strongly discouraged.

What does Xbootclasspath do?

Sets the search path for bootstrap classes and resources. The default is to search for bootstrap classes and resources in the internal VM directories and .jar files.

The append and normal mode are still available, only the prepend has been removed due the new modular approach with Jigsaw.

So I will follow @jgabios approach to use the append option instead of prepend in Ringo 1.2.1. Please note that we will still need to verify that everything works with JDK 9+ and keep an eye on Jetty & Rhino.

See adb516a

@botic botic added this to the 1.2.1 milestone Nov 4, 2017
@grob
Copy link
Member

grob commented Nov 5, 2017

Correct me if i'm wrong, but the jar that is prepended by -Xbootclasspath:/p doesn't exist anymore - it's supposed to be rhino's js.jar, but iirc this changed when the build env switched to ivy. i think this option is a leftover from a long time ago and can be safely removed.

@botic
Copy link
Member Author

botic commented Nov 6, 2017

My problem is: I cannot judge this since there is no link to the OpenJDK bug it is intending to fix. My first thought was that this could also target some (now legacy?) setups. At least OpenJDK 8 works as expected and I agree, it should since there is no js.jar anymore with ivy.

@botic botic closed this as completed in dc8b8cc Nov 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants