You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2004/03/02 13:56:12 UTC
cvs commit: maven-plugins/checkstyle plugin.jelly
brett 2004/03/02 04:56:12
Modified: checkstyle plugin.jelly
Log:
fix error reporting
Revision Changes Path
1.21 +5 -14 maven-plugins/checkstyle/plugin.jelly
Index: plugin.jelly
===================================================================
RCS file: /home/cvs/maven-plugins/checkstyle/plugin.jelly,v
retrieving revision 1.20
retrieving revision 1.21
diff -u -r1.20 -r1.21
--- plugin.jelly 7 Feb 2004 23:26:22 -0000 1.20
+++ plugin.jelly 2 Mar 2004 12:56:12 -0000 1.21
@@ -76,14 +76,9 @@
<j:if test="${sourcesPresent == 'true'}">
- <j:catch var="exception">
- <attainGoal name="checkstyle:run"/>
- </j:catch>
-
- <j:if test="${(exception != null)}">
- <j:choose>
- <j:when test="${exception.indexOf('unable to load header file') != '-1'}">
- <ant:echo>
+ <util:file var="headerFile" name="${maven.checkstyle.header.file}"/>
+ <j:if test="${!headerFile.exists()}">
+ <ant:echo>
=================================================================
= W A R N I N G =
=================================================================
@@ -95,14 +90,10 @@
= A temporary solution is to create an empty LICENSE.txt file. =
= Though, we recommend that you edit this License. =
=================================================================
- </ant:echo>
- </j:when>
- <j:otherwise>
- <ant:fail message="${exception}"/>
- </j:otherwise>
- </j:choose>
+ </ant:echo>
</j:if>
+ <attainGoal name="checkstyle:run"/>
</j:if>
</goal>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org