Actually, perhaps you could just add a second target that creates the HTML output, or define a boolean ANT property that I could set in my local properties that I could set to generate.
On Fri, Dec 20, 2013 at 1:21 PM, Glenn Adams <gl...@skynav.com> wrote: > I always use the HTML version, so please re-enable. > > > On Tue, Dec 17, 2013 at 10:20 AM, Vincent Hennebert > <vhenneb...@gmail.com>wrote: > >> Hi, >> >> I have personally never used the HTML output of Checkstyle as I find it >> of very little use. It lists each and every file it checked along with >> the number of errors in that file. That makes it virtually impossible to >> find those files that have a non-zero number of errors, as they are lost >> in an ‘ocean’ of non-erroneous files. >> >> As I wanted to have Gump run Checkstyle, I needed to simplify the task >> and the easiest was to remove the reference to the XSLT stylesheet that >> creates the HTML out of the XML output. >> >> If anybody feels that they would miss that feature, please let me know >> and I’ll try and re-enable it in a separate target. >> >> Thanks, >> Vincent >> >> >> On 17/12/13 18:11, vhennebert wrote: >> >>> Author: vhennebert >>> Date: Tue Dec 17 17:11:56 2013 >>> New Revision: 1551634 >>> >>> URL: http://svn.apache.org/r1551634 >>> Log: >>> Have Gump run Checkstyle on the source code >>> Removed HTML output for easier integration with Gump >>> >>> Modified: >>> xmlgraphics/fop/trunk/build.xml >>> >>> Modified: xmlgraphics/fop/trunk/build.xml >>> URL: http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/build. >>> xml?rev=1551634&r1=1551633&r2=1551634&view=diff >>> ============================================================ >>> ================== >>> --- xmlgraphics/fop/trunk/build.xml (original) >>> +++ xmlgraphics/fop/trunk/build.xml Tue Dec 17 17:11:56 2013 >>> @@ -1042,10 +1042,8 @@ NOTE: >>> <!-- Checkstyle >>> --> >>> <!-- =================================================================== >>> --> >>> <property name="checkstyle.location" >>> value="${lib-tools}/checkstyle-5.5-all.jar" >>> /> >>> - <property name="checkstyle.noframes.xslt" >>> value="${basedir}/checkstyle-noframes.xsl" /> >>> <property name="checkstyle.config" value="${basedir}/checkstyle-5.5.xml" >>> /> >>> <path id="checkstyle-classpath"> >>> - <path refid="libs-build-classpath"/> >>> <pathelement location="${checkstyle.location}"/> >>> </path> >>> <condition property="checkstyle.avail"> >>> @@ -1053,14 +1051,12 @@ NOTE: >>> <available classname="com.puppycrawl.tools.checkstyle. >>> CheckStyleTask"> >>> <classpath refid="checkstyle-classpath"/> >>> </available> >>> - <available file="${checkstyle.noframes.xslt}"/> >>> <available file="${checkstyle.config}"/> >>> </and> >>> </condition> >>> <target name="checkstyle-avail" unless="checkstyle.avail"> >>> <echo message="Checkstyle support NOT present. Please download it >>> from http://checkstyle.sf.net/ and"/> >>> <echo message="... please provide ${checkstyle.location}"/> >>> - <echo message="... please provide ${checkstyle.noframes.xslt}"/> >>> <echo message="... please provide ${checkstyle.config}"/> >>> </target> >>> <target name="checkstyle" depends="package, checkstyle-avail" >>> if="checkstyle.avail" description="Runs Checkstyle for a code quality >>> report"> >>> @@ -1077,7 +1073,6 @@ NOTE: >>> <fileset dir="${test.dir}" includes="**/*.java"/> >>> <formatter type="xml" toFile="${build.dir}/report_ >>> checkstyle.xml"/> >>> </checkstyle> >>> - <xslt in="${build.dir}/report_checkstyle.xml" >>> out="${build.dir}/report_checkstyle.html" style="${checkstyle.noframes. >>> xslt}"/> >>> </target> >>> <!-- =================================================================== >>> --> >>> <!-- PMD >>> --> >>> @@ -1508,7 +1503,7 @@ NOTE: >>> <!-- Special target for Gump >>> --> >>> <!-- =================================================================== >>> --> >>> <target name="gump" depends="package,transcoder-pkg"/> >>> - <target name="gump-test" depends="junit-all"> >>> + <target name="gump-test" depends="junit-all,checkstyle"> >>> <fail> >>> <condition> >>> <or> >>> >> >