[ http://jira.codehaus.org/browse/MPASPECTJ-19?page=comments#action_38108 ]
     
Vincent Massol commented on MPASPECTJ-19:
-----------------------------------------

Thanks Matt. I'm getting the following which does not sound right (but then 
I've not read the tests so this may be normal):

C:\dev\maven\maven-1\plugins\trunk\aspectj>maven plugin:test
 __  __
|  \/  |__ _Apache__ ___
| |\/| / _` \ V / -_) ' \  ~ intelligent projects ~
|_|  |_\__,_|\_/\___|_||_|  v. 1.0.2

[warning] couldn't find aspectjrt.jar on classpath, checked: 
c:\apps\maven-1.0.2\lib\endorsed\xerces-2.4.0.jar;c:\apps\maven-1.0.2
\lib\endorsed\xml-apis-1.0.b2.jar;c:\j2sdk1.4.2_05\jre\lib\rt.jar;c:\j2sdk1.4.2_05\jre\lib\i18n.jar;c:\j2sdk1.4.2_05\jre\lib\sunrs
asign.jar;c:\j2sdk1.4.2_05\jre\lib\jsse.jar;c:\j2sdk1.4.2_05\jre\lib\jce.jar;c:\j2sdk1.4.2_05\jre\lib\charsets.jar;c:\j2sdk1.4.2_0
5\jre\classes;c:\j2sdk1.4.2_05\jre\lib\ext\dnsns.jar;c:\j2sdk1.4.2_05\jre\lib\ext\ldapsec.jar;c:\j2sdk1.4.2_05\jre\lib\ext\localed
ata.jar;c:\j2sdk1.4.2_05\jre\lib\ext\sunjce_provider.jar;C:\Documents and 
Settings\Vincent Massol\.maven\repository\aspectj\jars\a
spectjrt-1.2.jar;C:\Documents and Settings\Vincent 
Massol\.maven\repository\aspectj\jars\aspectjrt-1.2.1.jar;C:\dev\maven\maven-1\
plugins\trunk\aspectj\src\plugin-test\target\classes

C:\dev\maven\maven-1\plugins\trunk\aspectj\src\plugin-test\src\aspect\org\apache\maven\aspectj\Sample.aj:25
 [warning] no match for
 this type name: org.apache.maven.aspectj.AspectTest 
[Xlint:invalidAbsoluteTypeName]
execution(public * org.apache.maven.aspectj.AspectTest.someMethod());
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[warning] couldn't find aspectjrt.jar on classpath, checked: 
c:\apps\maven-1.0.2\lib\endorsed\xerces-2.4.0.jar;c:\apps\maven-1.0.2
\lib\endorsed\xml-apis-1.0.b2.jar;c:\j2sdk1.4.2_05\jre\lib\rt.jar;c:\j2sdk1.4.2_05\jre\lib\i18n.jar;c:\j2sdk1.4.2_05\jre\lib\sunrs
asign.jar;c:\j2sdk1.4.2_05\jre\lib\jsse.jar;c:\j2sdk1.4.2_05\jre\lib\jce.jar;c:\j2sdk1.4.2_05\jre\lib\charsets.jar;c:\j2sdk1.4.2_0
5\jre\classes;c:\j2sdk1.4.2_05\jre\lib\ext\dnsns.jar;c:\j2sdk1.4.2_05\jre\lib\ext\ldapsec.jar;c:\j2sdk1.4.2_05\jre\lib\ext\localed
ata.jar;c:\j2sdk1.4.2_05\jre\lib\ext\sunjce_provider.jar;C:\Documents and 
Settings\Vincent Massol\.maven\repository\aspectj\jars\a
spectjrt-1.2.jar;C:\Documents and Settings\Vincent 
Massol\.maven\repository\junit\jars\junit-3.8.1.jar;C:\Documents and Settings\V
incent 
Massol\.maven\repository\aspectj\jars\aspectjrt-1.2.1.jar;C:\dev\maven\maven-1\plugins\trunk\aspectj\src\plugin-test\target
\classes

The 1st and last warnings are ok (they are due to our aspectjrt.jar being named 
with a version and it seems the "aspectjrt.jar" name is hardcoded in AspectJ's 
code. Real ugly!). I'm concerned about the warning about "no match for...". Any 
idea?

Thanks

> Add messageHolderClass property
> -------------------------------
>
>          Key: MPASPECTJ-19
>          URL: http://jira.codehaus.org/browse/MPASPECTJ-19
>      Project: maven-aspectj-plugin
>         Type: Improvement
>     Versions: 3.2
>     Reporter: Matt Smith
>  Attachments: MPASPECTJ-19-2.txt, MPASPECTJ-19.txt
>
>
> from the iajc ant task documentation:
> Specify a class to use as the message holder for the compile process. The 
> entry must be a fully-qualified name of a class resolveable from the task 
> classpath complying with the org.aspectj.bridge.IMessageHolder interface and 
> having a public no-argument constructor.
> Adding the ability to use a messageHolderClass requires two changes:
>   1)  add maven.aspectj.messageHolderClass property and associated attribute 
> messageHolderClass
>   2)  add maven.dependency.classpath to the ant task def
> Please see the attached file

-- 
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, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to