[ 
http://jira.codehaus.org/browse/MASPECTJ-94?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=262047#action_262047
 ] 

nodje commented on MASPECTJ-94:
-------------------------------

maven-dependency-plugin seems good to me.
Since all included JARS, be it weaveDependency or aspectLibrary, can contain 
aspect, I can't see any reason to make a difference between these two.

There should be a level of abstraction between the plugin and AJC, providing 
something in the line of ~ <includeJarDependency> that should be unpacked and 
included as source, so that we can a single consistent behavior across all 
aspect to apply.

I can't help but finding it very odd that ajc provides a -inpath and a 
-aspectpath and that both can contain aspect to be applied on source. There 
must be a good reason. I've open the question on Stackoverflow.

Anyway, even if it makes sense to keep weaveDependency or aspectLibrary, they 
both should be unpacked and included as source folder.

> <excludes> could also filter aspects included in <aspectLibrary>
> ----------------------------------------------------------------
>
>                 Key: MASPECTJ-94
>                 URL: http://jira.codehaus.org/browse/MASPECTJ-94
>             Project: Mojo AspectJ Plugin
>          Issue Type: Improvement
>    Affects Versions: 1.3.1
>         Environment: all
>            Reporter: nodje
>         Attachments: AcpectLibraryExclusionsAntStyle.zip
>
>
> I was trying to exclude an aspect contained in spring-aspects library, that I 
> was specifying as <aspectLibrary> in the config, when I realized it couldn't 
> be filtered.
> I've created a test that fails when aspectLibrary aspect are woven despite 
> their exclusion. It is to be added to the src/it tests.
> It's not very clean as it's based on an external aspect library 
> (spring-aspects), but it shows what is expected clearly I believe.
> Let me know if I'm being unclear.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to