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

alternative classloader strategy #55

Merged
merged 1 commit into from
Feb 22, 2017
Merged

alternative classloader strategy #55

merged 1 commit into from
Feb 22, 2017

Conversation

imario42
Copy link
Contributor

Alternatively to creating a classloader for the application at runtime, allow to configure at fxlauncher classloader at boot-time and add the new classpath-entries to this classloader.
This should avoid problems where the jdk itself uses Class.forName (e.g. URLStreamHandler stuff) and might not find application-classes due to using the wrong classloader.

To activate this classloader one needs to add
-Djava.system.class.loader=fxlauncher.FxlauncherClassCloader
to the fxlauncher startup.
Without this argument the current behavior is still in place.

…e, allow to configure at fxlauncher classloader at boot-time and add the new classpath-entries to this classloader. this should avoid problems where the jdk itself does use Class.forName and might not find application-classes due to using the wrong classloader.
@edvin
Copy link
Owner

edvin commented Feb 22, 2017

That's a neat trick :) Thanks a lot!

@edvin edvin merged commit ce53a4b into edvin:master Feb 22, 2017
@edvin edvin mentioned this pull request Feb 27, 2017
@imario42
Copy link
Contributor Author

Yes, just tested again here and it seems to work.

@edvin
Copy link
Owner

edvin commented Feb 28, 2017

Perfect. Will do a release soon.

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.

2 participants