Would it be nicer to specify the classpath once rather than as both an attribute and an element? In my branch I have this:

      <classpath>
        <dirset dir="${module.out.dir}"/>
        <path refid="classpath"/>
      </classpath>