You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by gi...@apache.org on 2018/06/08 06:05:14 UTC

ant-ivyde git commit: IvyDE Jenkins expects that Checkstyle failures are ignored

Repository: ant-ivyde
Updated Branches:
  refs/heads/master f4b72144e -> e1ed2617f


IvyDE Jenkins expects that Checkstyle failures are ignored

Project: http://git-wip-us.apache.org/repos/asf/ant-ivyde/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant-ivyde/commit/e1ed2617
Tree: http://git-wip-us.apache.org/repos/asf/ant-ivyde/tree/e1ed2617
Diff: http://git-wip-us.apache.org/repos/asf/ant-ivyde/diff/e1ed2617

Branch: refs/heads/master
Commit: e1ed2617faa341d7d066026655039a9e3d153b8e
Parents: f4b7214
Author: Gintas Grigelionis <gi...@apache.org>
Authored: Fri Jun 8 08:04:50 2018 +0200
Committer: Gintas Grigelionis <gi...@apache.org>
Committed: Fri Jun 8 08:04:50 2018 +0200

----------------------------------------------------------------------
 build.properties | 3 +++
 build.xml        | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ant-ivyde/blob/e1ed2617/build.properties
----------------------------------------------------------------------
diff --git a/build.properties b/build.properties
index 822535c..771a3a9 100644
--- a/build.properties
+++ b/build.properties
@@ -22,6 +22,9 @@ version.qualifier=beta1
 # For deeper debugging of the build system, set it to true
 eclipse.consoleLog=false
 
+# Comment out to fail builds if Checkstyle fails
+ignore.checkstyle=
+
 # URL of the mirror to use when trying to install Ivy from an official release
 mirror.url=http://archive.apache.org/dist/
 

http://git-wip-us.apache.org/repos/asf/ant-ivyde/blob/e1ed2617/build.xml
----------------------------------------------------------------------
diff --git a/build.xml b/build.xml
index 98ecd09..7c3961c 100644
--- a/build.xml
+++ b/build.xml
@@ -533,7 +533,7 @@ You have to specify the Ivy to install with one of the following property:
 
     <target name="checkstyle" depends="checkstyle-internal"
             description="checks Ivy codebase according to ${checkstyle.src.dir}/checkstyle-config">
-        <fail if="checkstyle.failed"
+        <fail if="checkstyle.failed" unless="ignore.checkstyle"
               message="Checkstyle has errors. See report in ${checkstyle.report.dir}"/>
     </target>