-
Notifications
You must be signed in to change notification settings - Fork 225
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
Shading fails on Java17 with UnsupportedOperationException: PermittedSubclasses requires ASM9 #479
Comments
Is there a fix or workaround for this? I'm also getting this error with Java 17. |
I guess I need to bump ASM here https://github.com/eed3si9n/jarjar-abrams/blob/develop/build.sbt#L26-L27 |
I took a look at the changes gradle made to support Java17, and it looks like it primarily comes down to updating |
I confirmed locally that updating ASM and changing |
IIUC, I'm not sure this is fixed when viewed from the context of sbt-assembly... It's not enough for the version of jarjar to be bumped; whoever is creating Can I substitute a patched version of jarjar-abrams in my build? :) |
Here's GitHub Actions https://github.com/sbt/sbt-assembly/actions/runs/6649535607 demonstrating scripted tests on JDK 21. |
👋 Do you mean this proves Even with the latest version of Applying the changes suggested above to I've opened a PR to gather feedback. |
I've confirmed that version 2.1.5 fixes my problem. Thanks! |
It appears as if for shading to work on Java 17 some dependencies must be updated. Definitely asm, possibly jar jar links as well. For reference, here's a full stack trace trying to shade
cats.**
into a spark project on Java 17:The text was updated successfully, but these errors were encountered: