on 10/21/01 8:40 PM, "Pier Fumagalli" <[EMAIL PROTECTED]> wrote:

> Jon Stevens at [EMAIL PROTECTED] wrote:
> 
>> <http://importscrubber.sourceforge.net/ant.html>
>> 
>> org.apache.tools.ant.taskdefs.optional.importscrubber.ImportScrubberTask
>> 
>> Given that this isn't an official Jakarta project, shouldn't the tool choose
>> another namespace?
> 
> Indeed...
> 
>   Pier
> 

That said, I just tried running importscrubber against Scarab's code and it
totally f*cked it up. I highly recommend not using it or at least being very
careful with it.

Tom, feel free to try yourself...I added the following to Scarab's
build.xml:

    <!-- ==================================================================
-->
    <!-- Tool to create proper import statements
-->
    <!-- ==================================================================
-->
    <target name="scrub" depends="om-peer,prepare">
        <taskdef name="scrub"
          
classname="org.apache.tools.ant.taskdefs.optional.importscrubber.ImportScrub
berTask"/>

        <property name="tmp.dir" value="tmp"/>

        <delete dir="${tmp.dir}" quiet="true"/>

        <copy todir="${tmp.dir}/org">
          <fileset dir="${build.src.scarab}/org"/>
        </copy>

        <javac srcdir="${tmp.dir}"
            destdir="${tmp.dir}"
            excludes="**/package.html,torque/**"
            debug="true">

          <classpath refid="classpath"/>

          <classpath>
            <fileset dir="${build.project.webinf.lib}">
              <include name="**/torque*.jar"/>
            </fileset>
          </classpath>
        </javac>
        
        <scrub root="${tmp.dir}" format="nobreaks" recurse="true"/>

        <delete>
          <fileset dir="${tmp.dir}" includes="**/*.class"/>
        </delete>

        <copy todir="${src.java.dir.scarab}/org" overwrite="true">
          <fileset dir="${tmp.dir}/org"/>
        </copy>

    </target>

-jon


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to