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
Update ASM to 7.0 for full Java 11 support #785
Conversation
It includes full support for Java 11.
asm-xml is deleted since 7.0-beta. In asm 6.1, asm-xml is deprecated. |
Thanks @wreulicke, fixed it. |
I remember that we use asm-xml to keep backward compatibility in Eclipse manifest, see #400 and @iloveeclipse could you review this PR? I'm not sure we can remove |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change spotbugs/spotbugs/META-INF/MANIFEST-TEMPLATE.MF and remove xml package, diff:
git diff
diff --git a/spotbugs/META-INF/MANIFEST-TEMPLATE.MF b/spotbugs/META-INF/MANIFEST-TEMPLATE.MF
index 42c88a988..7863fbef1 100644
--- a/spotbugs/META-INF/MANIFEST-TEMPLATE.MF
+++ b/spotbugs/META-INF/MANIFEST-TEMPLATE.MF
@@ -64,8 +64,7 @@ Export-Package: edu.umd.cs.findbugs,
org.objectweb.asm.signature,
org.objectweb.asm.tree,
org.objectweb.asm.tree.analysis,
- org.objectweb.asm.util,
- org.objectweb.asm.xml
+ org.objectweb.asm.util
Bundle-RequiredExecutionEnvironment: JavaSE-1.8
Bundle-ManifestVersion: 2
Bundle-Name: spotbugs
We can remove xml package, we have not re-exported it, but we should remove it completely :-) |
Thanks, updated the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a reason why I've attached the diff :-)
Co-Authored-By: ijuma <ismael@juma.me.uk>
org.ow2.asm:asm-xml
which has been removed.Make sure these boxes are checked before submitting your PR -- thank you!
CHANGELOG.md
if you have changed SpotBugs code