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.

Sign up for GitHub

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

Jump to bottom

MVEL is broken with Java 21 #2484

Closed
jan-zajic opened this issue Oct 12, 2023 · 2 comments
Closed

MVEL is broken with Java 21 #2484

jan-zajic opened this issue Oct 12, 2023 · 2 comments
Assignees
@tsegismont
Labels
bug
Milestone
4.5.0

Comments

@jan-zajic
Copy link
Contributor

jan-zajic commented Oct 12, 2023 •
edited

The new LTS release of Java 21 broke vertx-web-templ-mvel, more specifically the MVEL library. Updating MVEL to the version at least 2.5.0.Final fixes this issue.

Version

4.4.5

Context

java.lang.NoClassDefFoundError: java/lang/Compiler
at org.mvel2.compiler.AbstractParser.setupParser(AbstractParser.java:215)
at org.mvel2.compiler.AbstractParser.(AbstractParser.java:153)
at org.mvel2.MVEL.compileExpression(MVEL.java:831)

Steps to reproduce

  1. Use JDK 21
  2. Use vertx-web-templ-mvel
  3. Run some template
@jan-zajic jan-zajic added the bug label Oct 12, 2023
@tsegismont tsegismont self-assigned this Oct 13, 2023
@tsegismont tsegismont added this to the 4.5.0 milestone Oct 13, 2023
@tsegismont
Copy link
Contributor

Thanks for reporting this. Would you like to contribute the upgrade?

@jan-zajic jan-zajic mentioned this issue Oct 13, 2023
Merged
@jan-zajic
Copy link
Contributor Author

OK, I can do that, I've created PR #2485.

tsegismont pushed a commit that referenced this issue Oct 25, 2023
@jan-zajic @tsegismont
Fixes #2484
dd68261 
Signed-off-by: jzajic <jan.zajic@corpus.cz>
@tsegismont tsegismont mentioned this issue Dec 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsegismont tsegismont

Labels
bug
Milestone
4.5.0
Development

No branches or pull requests
2 participants
@tsegismont @jan-zajic