slice2java ant task not re-running

[blair]

I have a build.xml file that works fine the first time you run it, but after running an "ant clean" task that deletes the subdirectory that slice2java creates, then successive runs of ant do not rebuild things:

$ ant
Buildfile: build.xml

slice2java:
[slice2java] skipping vnp_subversion.ice

Right now, I need to touch the slice file for it to always regenerate the *.java files.

The top portion of our slice file:

#ifndef VNP_ICE
#define VNP_ICE

[["java:package:com.imageworks.vnp"]]
module SubversionIce
{
    /** Generic exception. */
    exception VnpIceException {
        string message;
    };

and a portion of our build.xml.

<project name="VnpSubversionServer" default="slice2java">

    <property name="ice_home.dir" value="/usr/lib/Ice-3.1.1"/>

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

    <property name="generated.dir"
              value="${src.dir}/com/imageworks/vnp/SubversionIce"/>

    <taskdef name="slice2java"
             classpath="${ice_home.dir}/ant"
             classname="Slice2JavaTask"/>

    <target name="slice2java">
        <!-- We seem to always need to run slice2java otherwise it
             will not create the *.java files if they have been
             deleted. -->
        <slice2java outputdir="${src.dir}">
            <fileset dir="slice" includes="vnp_subversion.ice"/>
        </slice2java>
    </target>

    <target name="clean">
        <delete file=".depend"/>
        <delete dir="${generated.dir}"/>
        <delete dir="${java.build.dir}"/>
        <delete file="${vnp_svn.jar}"/>
        <delete dir="${scala.build.dir}"/>
    </target>

</project>

Any ideas on how to make slice2java rerun if the files it generates are gone?

Regards,
Blair

[mes]

Hi Blair,

The Slice2Java ant task uses a dependency file to determine whether a Slice file needs to be recompiled. By default this file is named .depend, or ${outputdir}/.depend if you define the outputdir attribute. In your ant project, this file would be called ${src.dir}/.depend.

As you've seen, removing the generated files does not prompt the Slice2Java task to execute the translator. However, if you remove the dependency file as part of your clean target, the Slice2Java task will recompile all of your Slice files.

Hope that helps,
- Mark

[blair]

Hi Mark,

Thanks, that did the trick.

Regards,
Blair

[jrray]

I ran into a problem with the ant dependency feature after I enabled checksums. If a built detects that only some of the slice files need to be rebuilt, my checksum class is remade from only the out of date slice files, and the class is then missing entries.

I made this patch to change the dependency tracking into an all-or-nothing rebuild, if the checksum feature is enabled.

--- ant/Slice2JavaTask.java     (revision 153092)
+++ ant/Slice2JavaTask.java     (revision 153093)
@@ -118,6 +118,7 @@
         // last updated or a slice file it depends on changed).
         //
         java.util.Vector buildList = new java.util.Vector();
+        java.util.Vector skipList = new java.util.Vector();
         java.util.Iterator p = _fileSets.iterator();
         while(p.hasNext())
         {
@@ -137,7 +138,27 @@
                 }
                 else
                 {
-                    log("skipping " + files[i]);
+                    skipList.addElement(slice);
+                }
+            }
+
+            if(   _checksum != null && _checksum.length() > 0
+               && !buildList.isEmpty())
+            {
+                //
+                // If generating a checksum, all slice files must be compiled
+                // if any are out of date so that the checksum class contains
+                // all the entities from all the slice files.
+                //
+                buildList.addAll(skipList);
+            }
+            else
+            {
+                java.util.Iterator i = skipList.iterator();
+                while(i.hasNext())
+                {
+                    File file = (File)i.next();
+                    log("skipping " + file.getName());
                 }
             }
         }