You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2018/06/10 11:31:35 UTC

[GitHub] slachiewicz commented on issue #169: [MNG-6344] Upgrade Guice to 4.2 wit AOP (cglib)

slachiewicz commented on issue #169: [MNG-6344] Upgrade Guice to 4.2 wit AOP (cglib)
URL: https://github.com/apache/maven/pull/169#issuecomment-396041950
 
 
   Based on [Guice optional AOP](https://github.com/google/guice/wiki/OptionalAOP#optional-aop)
   
   > In Guice 2.0 and later, AOP is optional. If your platform doesn't support bytecode generation, you can download a version of Guice that doesn't include AOP support. This is most useful for mobile platforms like Android. This version also lacks fast reflection and line numbers in error messages. For this reason, we recommend Guice+AOP even in applications that don't use method interceptors.
   
   Only risk I can see is inclusion of additional dependency to CGLIB and ASM

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services