-
Notifications
You must be signed in to change notification settings - Fork 18
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
Asm7 java10and11 #54
base: master
Are you sure you want to change the base?
Asm7 java10and11 #54
Conversation
The library is split in multiple jars since 6.x. This requires some changes in the build description.
This is required to make the 'ant test' tests pass.
I am using JDK 11. I applied your patches. The default compilation fails when building the javadoc jars. |
This is no regression. The previous version also triggered 8 errors and 1 warning with JDK8, when building the documentation. |
Could that be merged to keep this project usable for newer Java settings? |
Is this project still alive? |
Hey!!! please merge, need java 1.8 too... |
@nhatminhle May I ask whether you see these notifications? We would "just" ask you to merge this pull request. |
@miasma Is this project dead? 🥲 |
Update to ASM7 jars. Should fix issues with Java 10 and 11. Tested (ant test) with OpenJDK 8,10, and 11.