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> >> >