-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
Hi Jan,
Thank you! The problem was not the JVM memory parameter but how to specify the source packages without having to pass in a full list of the packages to be javadoc'd. I must have tried a dozen different ways to do it, none of which Ant or javadoc would accept. I'm pretty sure I tried several different combinations of org.*, but I guess I was determined to pass it to <javadoc> in an attribute for the source files/packages that doesn't like wildcards. I finally gave up and just called javadoc with <java> because that took me less time to figure out. :)
With that fixed, the subpackages target is no longer necessary, so that can be removed and the remainder consolidated to the derbydocs target. Below is an updated patch which consolidates the targets and fixes a couple of other little nits.
I'd like to check this in. I guess now would be a good time to discuss whether Derby should follow the 'commit then review' or the 'review then commit' style for applying patches.
Thanks again, Andrew
Index: tools/ant/properties/dirs.properties =================================================================== --- tools/ant/properties/dirs.properties (revision 37102) +++ tools/ant/properties/dirs.properties (working copy) @@ -8,6 +8,7 @@ src.dir=${basedir} javasrc.dir=${src.dir}/java javatools.dir=${src.dir}/tools/java +javadoctools.dir=${src.dir}/tools/javadoc javacom.dir=${javasrc.dir}/${com.dir} functest.dir=${javacom.dir}/functionTests locales.dir=${javacom.dir}/locales @@ -17,7 +18,7 @@ class.dir=classes out.dir=${basedir}/${class.dir} outcom.dir=${out.dir}/${com.dir} -out.javadocs.dir=${basedir}/html +out.javadoc.dir=${basedir}/javadoc out.testdir=${basedir}/testout_${jdk} loc.dir=${out.dir}/org/apache/derby/loc drdaloc.dir=${out.dir}/org/apache/derby/loc/drda @@ -33,10 +34,10 @@ oscom.dir=com/ihost/cs ossrc.dir=${basedir}/java
-derby.engine.src.dir=${derbysrc.dir}/engine/
-derby.drda.src.dir=${derbysrc.dir}/drda/
-derby.tools.src.dir=${derbysrc.dir}/tools/
-derby.build.src.dir=${derbysrc.dir}/build/
+derby.engine.src.dir=${derbysrc.dir}/engine
+derby.drda.src.dir=${derbysrc.dir}/drda
+derby.tools.src.dir=${derbysrc.dir}/tools
+derby.build.src.dir=${derbysrc.dir}/build derby.engine.dir=${derby.engine.src.dir}/${derby.dir}
derby.drda.dir=${derby.drda.src.dir}/${derby.dir}
Index: build.xml
===================================================================
--- build.xml (revision 37102)
+++ build.xml (working copy)
@@ -354,79 +354,94 @@<target name="publishedapi">
- <delete dir="${basedir}/javadoc/publishedapi"/>
- <delete dir="${basedir}/javadoc/sourcedir"/>
- <mkdir dir="${basedir}/javadoc/publishedapi"/>
- <mkdir dir="${basedir}/javadoc/sourcedir"/>
+ <delete dir="${out.javadoc.dir}/publishedapi"/>
+ <delete dir="${out.javadoc.dir}/sourcedir"/>
+ <mkdir dir="${out.javadoc.dir}/publishedapi"/>
+ <mkdir dir="${out.javadoc.dir}/sourcedir"/> <!-- drda files -->
- <mkdir dir="${basedir}/javadoc/sourcedir"/>
- <copy todir="${basedir}/javadoc/sourcedir">
+ <mkdir dir="${out.javadoc.dir}/sourcedir"/>
+ <copy todir="${out.javadoc.dir}/sourcedir">
<fileset dir="${derby.drda.src.dir}"
includesfile="${javadoctools.dir}/publishedapi.ant"/>
</copy> <!-- engine files -->
- <copy todir="${basedir}/javadoc/sourcedir">
+ <copy todir="${out.javadoc.dir}/sourcedir">
<fileset dir="${derby.engine.src.dir}"
includesfile="${javadoctools.dir}/publishedapi.ant"/>
</copy> <!-- tools files -->
- <copy todir="${basedir}/javadoc/sourcedir">
+ <copy todir="${out.javadoc.dir}/sourcedir">
<fileset dir="${derby.tools.src.dir}"
includesfile="${javadoctools.dir}/publishedapi.ant"/>
</copy>- <javadoc sourcepath="${basedir}/javadoc/sourcedir/"
+ <javadoc sourcepath="${out.javadoc.dir}/sourcedir/"
classpath="${compile.classpath};${osgi};${servlet22};${out.dir}"
breakiterator="true"
- destdir="${basedir}/javadoc/publishedapi"
+ destdir="${out.javadoc.dir}/publishedapi"
packagenames="org.apache.derby.authentication,org.apache.derby.drda,org. apache.derby.jdbc.io,org.apache.derby.jdbc,org.apache.derby.tools"
Windowtitle="Apache Derby ${major}.${minor} API Documentation"
Doctitle="Apache Derby ${major}.${minor} API Documentation"
bottom="Apache Derby ${major}.${minor} API Documentation - <i>Copyright &copy; IBM Corp. 1997,2004 All Rights Reserved.</i>"/>
- <delete dir="${basedir}/javadoc/sourcedir"/>
+ <delete dir="${out.javadoc.dir}/sourcedir"/></target>
<target name="derbydocs">
- <delete dir="${basedir}/javadoc/engine"/>
- <mkdir dir="${basedir}/javadoc/engine"/>
+ <delete dir="${out.javadoc.dir}/engine"/>
+ <mkdir dir="${out.javadoc.dir}/engine"/>- <antcall target="javadocsubpackages">
- <param name="javadoc.sourcepath" value="${derby.engine.src.dir};${derby.drda.src.dir};${derby.tools.src.d ir}"/>
- <param name="javadoc.classpath" value="${basedir}/ classes;${java14compile.classpath};${jars.extras};${osgi}"/>
- <param name="javadoc.destdir" value="${basedir}/javadoc/engine"/>
- <param name="javadoc.subpackages" value="org.apache.derby"/>
- <param name="javadoc.doctitle" value="Apache Derby V${major}.${minor} Engine Documentation"/>
- <param name="javadoc.windowtitle" value="Apache Derby V${major}.${minor} Engine Documentation"/>
- <param name="javadoc.bottom" value="Apache Derby V${major}.${minor} Engine Documentation - <i>Copyright &copy; IBM Corp. 1997,2004 All Rights Reserved.</i>"/>
- </antcall>
+ <javadoc
+ additionalparam="-J-mx320M"
+ packagenames="org.*"
+ access="private"
+ breakiterator="yes"
+ sourcepath="${derby.engine.src.dir};${derby.drda.src.dir};${derby.tools. src.dir}"
+ classpath="${basedir}/ classes;${java14compile.classpath};${jars.extras};${osgi}"
+ destdir="${out.javadoc.dir}/engine"
+ windowtitle="Apache Derby V${major}.${minor} Engine Documentation"
+ doctitle="Apache Derby V${major}.${minor} Engine Documentation"
+ bottom="Apache Derby V${major}.${minor} Engine Documentation - <i>Copyright &copy; IBM Corp. 1997,2004 All Rights Reserved.</i>"
+ source="1.4">
+ <taglet name="org.apache.derbyBuild.javadoc.FormatIdTaglet"
+path="${basedir}/classes"/>
+ <taglet name="org.apache.derbyBuild.javadoc.PurposeTaglet"
+path="${basedir}/classes"/>
+ <taglet name="org.apache.derbyBuild.javadoc.UpgradeTaglet"
+path="${basedir}/classes"/>
+ <taglet name="org.apache.derbyBuild.javadoc.DiskLayoutTaglet"
+path="${basedir}/classes"/>
+ <taglet name="org.apache.derbyBuild.javadoc.EndFormatTaglet"
+path="${basedir}/classes"/>
+ </javadoc>
+
</target>
<target name="toolsdocs">
- <delete dir="${basedir}/javadoc/tools"/>
- <mkdir dir="${basedir}/javadoc/tools"/>
+ <delete dir="${out.javadoc.dir}/tools"/>
+ <mkdir dir="${out.javadoc.dir}/tools"/> <antcall target="dojjdocs">
<param name="jjtodoc" value="${derby.tools.dir}/impl/tools/ij/ij.jj"/>
- <param name="jjdoctarget" value="${basedir}/javadoc/tools/ij.html"/>
+ <param name="jjdoctarget" value="${out.javadoc.dir}/tools/ij.html"/>
</antcall></target>
<target name="grammardocs">
- <delete dir="${basedir}/javadoc/language"/>
- <mkdir dir="${basedir}/javadoc/language"/>
+ <delete dir="${out.javadoc.dir}/language"/>
+ <mkdir dir="${out.javadoc.dir}/language"/><antcall target="dojjdocs">
<param name="jjtodoc" value="${derby.engine.dir}/impl/sql/compile/sqlgrammar.jj"/>
- <param name="jjdoctarget" value="${basedir}/javadoc/language/sqlgrammar.html"/>
+ <param name="jjdoctarget" value="${out.javadoc.dir}/language/sqlgrammar.html"/>
</antcall>
</target> @@ -442,42 +457,9 @@
</target>
- <target name="javadocsubpackages">
- <java classname="com.sun.tools.javadoc.Main" fork="yes">
- <classpath path="${java.home}/../lib/tools.jar;${out.dir}"/>
- <jvmarg value="-mx320M"/>
- <arg value="-subpackages"/>
- <arg value="org"/>
- <arg value="-private"/>
- <arg value="-breakiterator"/>
- <arg value="-sourcepath"/>
- <arg value="${javadoc.sourcepath}"/>
- <arg value="-classpath"/>
- <arg value="${javadoc.classpath}"/>
- <arg value="-d"/>
- <arg value="${javadoc.destdir}"/>
- <arg value="-windowtitle"/>
- <arg value="${javadoc.windowtitle}"/>
- <arg value="-doctitle"/>
- <arg value="${javadoc.doctitle}"/>
- <arg value="-bottom"/>
- <arg value="${javadoc.bottom}"/>
- <arg value="-taglet"/>
- <arg value="org.apache.derbyBuild.javadoc.FormatIdTaglet"/>
- <arg value="-taglet"/>
- <arg value="org.apache.derbyBuild.javadoc.PurposeTaglet"/>
- <arg value="-taglet"/>
- <arg value="org.apache.derbyBuild.javadoc.UpgradeTaglet"/>
- <arg value="-taglet"/>
- <arg value="org.apache.derbyBuild.javadoc.DiskLayoutTaglet"/>
- <arg value="-taglet"/>
- <arg value="org.apache.derbyBuild.javadoc.EndFormatTaglet"/>
- </java>
- </target>
-
<target name="cleandocs"> - <delete dir="${basedir}/javadoc"/>
+ <delete dir="${out.javadoc.dir}"/></target> -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (Darwin)
iD8DBQFBLt5HDfB0XauCH7wRAvKJAKCKcMJRrB7FCXoCizRDBjbP5Y4r1QCfflUL cPqdrzsgEreDK7Sv+mZUo1U= =HOoF -----END PGP SIGNATURE-----
