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.

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

Exclude the classloader for jasperreports-plugin #3568

Merged
merged 1 commit into from Dec 12, 2023

Conversation

Postremus
Copy link

Basically the same fix as in #3079. Just new coordinates for the maven plugin.
Was reported over at Jasper-report-maven-plugin/issues/52

@mhagnumdw
Copy link

I tested it and it worked correctly.

Commands to generate the jar:

git clone git@github.com:projectlombok/lombok.git
cd lombok
git fetch origin pull/3568/head:pr-jasper
git switch pr-jasper
ant dist

In eclipse.ini I set -javaagent:/tmp/lombok/dist/lombok-1.18.31.jar and jasperreports-plugin worked as expected.

Any date to accept this PR? TKs!

@Rawi01
Copy link
Collaborator

Rawi01 commented Dec 11, 2023

@Postremus Can you also add yourself to the AUTHORS file?

@Postremus
Copy link
Author

@Rawi01 I updated the authors and changelog file.

AUTHORS Outdated Show resolved Hide resolved
@Rawi01 Rawi01 merged commit 0089374 into projectlombok:master Dec 12, 2023
@Postremus Postremus deleted the jasperreport branch December 12, 2023 07:58
@Postremus
Copy link
Author

Any chance to get a new release with this bug fix soon(ish)?

@mhagnumdw
Copy link

It hasn't even come out in the edge version yet. Any predictions for release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants