PR:MPASPECTJ-10
CVS: ---------------------------------------------------------------------- CVS: PR: CVS: If this change addresses a PR in the problem report tracking CVS: database, then enter the PR number(s) here. CVS: Obtained from: CVS: If this change has been taken from another system, such as NCSA, CVS: then name the system in this line, otherwise delete it. CVS: Submitted by: CVS: If this code has been contributed to Apache by someone else; i.e., CVS: they sent us a patch or a new module, then include their name/email CVS: address here. If this is your work then delete this line. CVS: Reviewed by: CVS: If we are doing pre-commit code reviews and someone else has CVS: reviewed your changes, include their name(s) here. CVS: If you have not had it reviewed then delete this line. Use maven.aspectj.maxmem only when maven.aspectj.fork=true to avoid aspectj warnings git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@115487 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
@@ -97,7 +97,6 @@
|
||||
|
||||
<ant:iajc
|
||||
fork="${maven.aspectj.fork}"
|
||||
maxmem="${maven.aspectj.maxmem}"
|
||||
incremental="${maven.aspectj.incremental}"
|
||||
destDir="${maven.build.dest}"
|
||||
sourceRootCopyFilter="${maven.aspectj.sourceRootCopyFilter}"
|
||||
@@ -107,6 +106,12 @@
|
||||
source="${maven.aspectj.source}"
|
||||
time="${maven.aspectj.time}">
|
||||
|
||||
<j:if test="${context.getVariable('maven.aspectj.fork') == true}">
|
||||
<j:if test="${context.getVariable('maven.aspectj.maxmem') != null}">
|
||||
<ant:setProperty name="maxmem" value="${maven.aspectj.maxmem}" />
|
||||
</j:if>
|
||||
</j:if>
|
||||
|
||||
<ant:sourceroots>
|
||||
<ant:path refid="maven.compile.src.set"/>
|
||||
<j:if test="${aspectSourcesPresent == 'true'}">
|
||||
|
||||
@@ -28,6 +28,11 @@
|
||||
</properties>
|
||||
<body>
|
||||
<release version="3.1-SNAPSHOT" date="in CVS">
|
||||
<action dev="carlos" type="fix" issue="MPASPECTJ-10">
|
||||
Use <code>maven.aspectj.maxmem</code>
|
||||
only when <code>maven.aspectj.fork=true</code>
|
||||
to avoid aspectj warnings
|
||||
</action>
|
||||
<action dev="vmassol" type="update">
|
||||
Now using AspectJ 1.2 release.
|
||||
</action>
|
||||
|
||||
Reference in New Issue
Block a user