-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Use the 'io.freefair.aspectj' gradle plugin #7183
Use the 'io.freefair.aspectj' gradle plugin #7183
Conversation
Thanks for the PR @larsgrefer! We'd love to rely on proper AspectJ Gradle plugin. However, it appears this PR broke the build. Can you take a look at that? |
5c61639
to
df413db
Compare
@rwinch The build is fixed now. |
Is there a reason that we need to move the source code into a different source folder? I'd prefer to keep the code in the src/main/java folder. |
The If you like, this can be changed in the build script: sourceSets.main.aspectj.srcDir "src/main/java" Another option would be the |
I think I'd like to use Spring allows switching between proxy and compile time aspects with just configuration. It is a bit disruptive to switch the folder that the annotated Java code is in. |
df413db
to
df4d564
Compare
df4d564
to
974e3c1
Compare
For the For
|
Thanks for the PR @larsgrefer! This is now merged into master |
After I've migrated (or removed) the other gradle plugins in #7077 #7173 and #7174 the AspectJ plugin was the last remaining one. Instead of migrating the current groovy based plugin to java, I just used the AspectJ gradle plugin I've already written last year.