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

JASSIST-271 Allow running on Java 11 by falling back to ClassLoader.defineClass for Java > 10 #198

Merged
merged 1 commit into from May 7, 2018

Conversation

beikov
Copy link
Contributor

@beikov beikov commented Apr 29, 2018

@chibash chibash merged commit 1108f94 into jboss-javassist:master May 7, 2018
@luengnat
Copy link

luengnat commented Jun 6, 2018

any plan to release this soon?

@chibash
Copy link
Member

chibash commented Jun 7, 2018

I can do so when you need. Immediately?

@chi9521
Copy link

chi9521 commented Jun 7, 2018

Yes, please, really need the fix. I could not find a way to solve the split package issue of java.transaction for Java 10. Even gradle compiles, Intellij IDEA will still show errors.

@joschi
Copy link

joschi commented Jun 20, 2018

@chibash Same here. It would be really great if you could release a new version of Javassist with this PR.

In my specific case it's about making Dropwizard run with Java 11-ea.
It works with current master but not with Javassist 3.22.0-GA.

joschi added a commit to dropwizard/dropwizard that referenced this pull request Jun 20, 2018
joschi added a commit to dropwizard/dropwizard that referenced this pull request Jun 20, 2018
joschi added a commit to dropwizard/dropwizard that referenced this pull request Jun 20, 2018
@chibash
Copy link
Member

chibash commented Jun 20, 2018

I've released it. It will be available from maven soon.

@joschi
Copy link

joschi commented Jun 20, 2018

@chibash Awesome, thank you very much!

joschi added a commit to dropwizard/dropwizard that referenced this pull request Jun 20, 2018
@beikov beikov deleted the JASSIST-271 branch June 21, 2018 13:54
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.

None yet

5 participants