You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Chuck Daniels (JIRA)" <ji...@codehaus.org> on 2007/01/31 23:26:44 UTC

[jira] Created: (MPASPECTJ-28) aspectj:test-compile goal should compile test classes to maven.test.dest by default

aspectj:test-compile goal should compile test classes to maven.test.dest by default
-----------------------------------------------------------------------------------

                 Key: MPASPECTJ-28
                 URL: http://jira.codehaus.org/browse/MPASPECTJ-28
             Project: maven-aspectj-plugin
          Issue Type: Improvement
    Affects Versions: 4.0
         Environment: All
            Reporter: Chuck Daniels
            Priority: Minor


aspectj:test-compile compiles test classes to maven.build.dest by default (since that is the default value for maven.aspectj.dest). Of course, setting maven.aspectj.dest=${maven.test.dest} will cause test classes to be compiled to the right place. Unfortunately, this then causes the main classes to also be compiled there, instead of to maven.build.dest. The aspectj:test-classes goal should probably use a separate property, such as maven.aspectj.test.dest (defaulted to ${maven.test.dest}), for compiling test classes to.

This can cause problems since a project's resulting artifact (jar file) can be built incorrectly. If maven.aspectj.dest is set to maven.build.dest, then the jar file will contain test classes (if aspectj:test-compile is part of the build process). If maven.aspectj.dest is set to maven.test.dest, then the jar file will contain classes compiled by the standard compiler, not the aspectj compiler, since the aspectj compiler will put both main and test classes in maven.test.dest.

-- 
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