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

allow setting a ClassLoader Supplier #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

allow setting a ClassLoader Supplier #129

wants to merge 1 commit into from

Conversation

asquare
Copy link
Contributor

@asquare asquare commented Sep 5, 2015

Specifying a ClassLoader Supplier instead of just a ClassLoader
instance gives more flexibility to the embedder by allowing the
embedder to choose if classes loaded by multiple innovations of
loadJars are isolated from each-other.

Specifying a ClassLoader Supplier instead of just a ClassLoader
instance gives more flexibility to the embedder by allowing the
embedder to choose if classes loaded by multiple innovations of
loadJars are isolated from each-other.
@asquare
Copy link
Contributor Author

asquare commented Sep 8, 2015

please also back-port this patch to 0.8.5 and 0.8.6

@gbrail
Copy link
Contributor

gbrail commented Sep 14, 2015

I pushed 0.8.5.1 to Maven Central this morning, and it should show up there pretty soon. Can you try it out? Assuming it works I'll also do 0.8.6.1 if you also need that.

@asquare
Copy link
Contributor Author

asquare commented Sep 23, 2015

0.8.5.1 works fine. Please also apply this change to 0.8.6 and publish - we will need it sooner or later. Thanks

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