You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by ca...@apache.org on 2004/06/17 20:07:50 UTC

cvs commit: maven-plugins/aspectj plugin.jelly

carlos      2004/06/17 11:07:50

  Modified:    aspectj/xdocs changes.xml
               aspectj  plugin.jelly
  Log:
  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
  
  Revision  Changes    Path
  1.15      +5 -0      maven-plugins/aspectj/xdocs/changes.xml
  
  Index: changes.xml
  ===================================================================
  RCS file: /home/cvs/maven-plugins/aspectj/xdocs/changes.xml,v
  retrieving revision 1.14
  retrieving revision 1.15
  diff -u -r1.14 -r1.15
  --- changes.xml	3 Jun 2004 11:53:16 -0000	1.14
  +++ changes.xml	17 Jun 2004 18:07:49 -0000	1.15
  @@ -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>
  
  
  
  1.13      +6 -1      maven-plugins/aspectj/plugin.jelly
  
  Index: plugin.jelly
  ===================================================================
  RCS file: /home/cvs/maven-plugins/aspectj/plugin.jelly,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- plugin.jelly	3 Jun 2004 11:49:45 -0000	1.12
  +++ plugin.jelly	17 Jun 2004 18:07:50 -0000	1.13
  @@ -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}"
  @@ -106,6 +105,12 @@
             verbose="${maven.aspectj.verbose}"
             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"/>
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org