-
Notifications
You must be signed in to change notification settings - Fork 383
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
Interest in Java Compiler API based compiler #203
Comments
A JSR 199 based report compiler would be a useful and convenient feature for JasperReports. We did consider it at some point, but if I recall correctly we found something that was not working as we expected it. Thank you for the contribution. |
@dadza do you know what caused issues? |
It was a long time ago, but I think the problem was that we didn't find a way to have the compiler resolve classes from the context classloader, which is something that the ECJ based report compiler supported. That's not a mandatory feature though. |
I have been working on a compiler based on the built-in JSR 199
JavaCompiler
API. This has the advantage that anybody deploying on a JDK (since Java 9 there is more JRE) or has thejava.compiler
module needs no external dependency. There is also an EJC based implementation.I would be willing to create a PR and donate the code. The code has only received minimal testing.
The text was updated successfully, but these errors were encountered: