Skip to main content
AI Assist is now on Stack Overflow. Start a chat to get instant answers from across the network. Sign up to save and share your chats.
added 12 characters in body
Source Link
kriegaex
  • 68.5k
  • 15
  • 120
  • 229

I forked your repo and fixed your Maven POM. You actually made a Maven beginner's mistake by pre-configuring plugins in the <pluginManagement> section but never actually declaring usage of the pre-configured plugins in the <plugins> section. In that case of course Maven does not use the AspectJ Maven plugin and thus does not weave any aspects either.

I sent you a pull request which you can just accept in order to make this work. There are several other things in your Maven settings which could use improvement, e.g.

  • if you really want to use AspectJ 1.9.5 and not the preconfigured 1.8.13 used in AspectJ Maven, there is a way to achieve that, but not the way you did it. Feel free to ask for more details if this is an issue.

  • if you want to use a JDK newer than 8 in order to run your Maven build. In that case you need to switch to a fork of AspectJ Maven because version 1.11 does not work with JDK 9+. I can also give you more details there if necessary. But with JDK 8 you should be fine.

I forked your repo and fixed your Maven POM. You actually made a Maven beginner's mistake by pre-configuring plugins in the <pluginManagement> section but never actually declaring usage of the pre-configured plugins in the <plugins> section. In that case of course Maven does not use the AspectJ Maven plugin and thus not weave any aspects.

I sent you a pull request which you can just accept in order to make this work. There are several other things in your Maven settings which could use improvement, e.g.

  • if you really want to use AspectJ 1.9.5 and not the preconfigured 1.8.13 used in AspectJ Maven, there is a way to achieve that, but not the way you did it. Feel free to ask for more details if this is an issue.

  • if you want to use a JDK newer than 8 in order to run your Maven build. In that case you need to switch to a fork of AspectJ Maven because version 1.11 does not work with JDK 9+. I can also give you more details there if necessary. But with JDK 8 you should be fine.

I forked your repo and fixed your Maven POM. You actually made a Maven beginner's mistake by pre-configuring plugins in the <pluginManagement> section but never actually declaring usage of the pre-configured plugins in the <plugins> section. In that case of course Maven does not use the AspectJ Maven plugin and thus does not weave any aspects either.

I sent you a pull request which you can just accept in order to make this work. There are several other things in your Maven settings which could use improvement, e.g.

  • if you really want to use AspectJ 1.9.5 and not the preconfigured 1.8.13 used in AspectJ Maven, there is a way to achieve that, but not the way you did it. Feel free to ask for more details if this is an issue.

  • if you want to use a JDK newer than 8 in order to run your Maven build. In that case you need to switch to a fork of AspectJ Maven because version 1.11 does not work with JDK 9+. I can also give you more details there if necessary. But with JDK 8 you should be fine.

Source Link
kriegaex
  • 68.5k
  • 15
  • 120
  • 229

I forked your repo and fixed your Maven POM. You actually made a Maven beginner's mistake by pre-configuring plugins in the <pluginManagement> section but never actually declaring usage of the pre-configured plugins in the <plugins> section. In that case of course Maven does not use the AspectJ Maven plugin and thus not weave any aspects.

I sent you a pull request which you can just accept in order to make this work. There are several other things in your Maven settings which could use improvement, e.g.

  • if you really want to use AspectJ 1.9.5 and not the preconfigured 1.8.13 used in AspectJ Maven, there is a way to achieve that, but not the way you did it. Feel free to ask for more details if this is an issue.

  • if you want to use a JDK newer than 8 in order to run your Maven build. In that case you need to switch to a fork of AspectJ Maven because version 1.11 does not work with JDK 9+. I can also give you more details there if necessary. But with JDK 8 you should be fine.