You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by xa...@apache.org on 2008/03/02 18:55:30 UTC
svn commit: r632800 - /ant/ivy/core/trunk/build.xml
Author: xavier
Date: Sun Mar 2 09:55:27 2008
New Revision: 632800
URL: http://svn.apache.org/viewvc?rev=632800&view=rev
Log:
make checkstyle error fail the build now that all checkstyle errors have been fixed
Modified:
ant/ivy/core/trunk/build.xml
Modified: ant/ivy/core/trunk/build.xml
URL: http://svn.apache.org/viewvc/ant/ivy/core/trunk/build.xml?rev=632800&r1=632799&r2=632800&view=diff
==============================================================================
--- ant/ivy/core/trunk/build.xml (original)
+++ ant/ivy/core/trunk/build.xml Sun Mar 2 09:55:27 2008
@@ -377,9 +377,11 @@
<target name="emma" depends="jar" unless="skip.test">
<ivy:cachepath organisation="emma" module="emma" revision="2.0.5312"
- inline="true" conf="default" pathid="emma.classpath" />
+ inline="true" conf="default" pathid="emma.classpath"
+ log="download-only" />
<ivy:cachepath organisation="emma" module="emma_ant" revision="2.0.5312"
- inline="true" conf="default" pathid="emma.ant.classpath" transitive="false" />
+ inline="true" conf="default" pathid="emma.ant.classpath" transitive="false"
+ log="download-only" />
<taskdef resource="emma_ant.properties">
<classpath refid="emma.classpath" />
<classpath refid="emma.ant.classpath" />
@@ -534,11 +536,16 @@
<target name="checkstyle" depends="jar" description="checks Ivy codebase according to ${checkstyle.src.dir}/checkstyle-config">
<ivy:cachepath organisation="checkstyle" module="checkstyle" revision="4.3"
- inline="true" conf="default" pathid="checkstyle.classpath" transitive="true" />
+ inline="true" conf="default" pathid="checkstyle.classpath" transitive="true"
+ log="download-only"/>
<taskdef resource="checkstyletask.properties" classpathref="checkstyle.classpath" />
<mkdir dir="${checkstyle.report.dir}" />
- <checkstyle config="${checkstyle.src.dir}/checkstyle-config" failOnViolation="false">
+ <checkstyle config="${checkstyle.src.dir}/checkstyle-config"
+ failOnViolation="false" failureProperty="checkstyle.failed">
+ <classpath>
+ <path refid="run.classpath" />
+ </classpath>
<formatter type="xml" toFile="${checkstyle.report.dir}/checkstyle.xml" />
<fileset dir="${src.dir}">
<include name="**/*.java" />
@@ -547,6 +554,8 @@
<include name="**/*.java" />
</fileset>
</checkstyle>
+ <fail if="checkstyle.failed"
+ message="Checkstyle has errors. See report in ${checkstyle.report.dir}" />
</target>
<target name="checkstyle-report" depends="checkstyle">