Bug 62300 - -Xreweavable option is ignored by the AjcTask
Summary: -Xreweavable option is ignored by the AjcTask
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: 1.2   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Jim Hugunin CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-05-14 12:17 EDT by nmreid2001 CLA
Modified: 2004-05-15 09:19 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description nmreid2001 CLA 2004-05-14 12:17:28 EDT
When using the iajc task from within ant, we tried to specify X="reweavable" to
produce reweavable class files:

      <iajc verbose="${verbose}"
            debug="${debug}"
            deprecation="${deprecation}"
            destdir="${build.classes.dir}"
            nowarn="true"
            maxmem="512m"
            X="reweavable">
         <injars>
            <pathelement path="${build.dir}/classes.jar"/>
         </injars>
         <sourceroots>
            <!-- aspects -->
            <pathelement path="${src.aspects.dir}"/>
            <pathelement path="${src.debugaspects.dir}"/>
         </sourceroots>
         <classpath>
            <path refid="compile.main.class.path"/>
         </classpath>
      </iajc>

When the ant task runs this is the output on the command line:

ignored: -Xreweavable at file:/xxxxx/build.xml

A quick scan through the AjcTask source would seem to point to the fact that the
reweavable option has not been added to the VALID_XOPTIONS list (which may or
may not control this behaviour) whereas the other new X option "lazyTjp" has
been added.

Tested in 1.2rc2.
Comment 1 Andrew Clement CLA 2004-05-15 07:17:58 EDT
Have you tried this alternative?

<iajc verbose="${verbose}"
            debug="${debug}"
            deprecation="${deprecation}"
            destdir="${build.classes.dir}"
            nowarn="true"
            maxmem="512m"
            Xreweavable="true">
Comment 2 nmreid2001 CLA 2004-05-15 09:19:38 EDT
Xreweavable="true" seems to work fine (no errors or warnings) at any rate... I
was under the impression from the documentation that the X options were
specified using a comma separated list in the ant task.