You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/05/23 21:15:59 UTC
svn commit: r1485830 - /commons/proper/csv/trunk/pom.xml
Author: sebb
Date: Thu May 23 19:15:59 2013
New Revision: 1485830
URL: http://svn.apache.org/r1485830
Log:
Allow checkstyle to be run interactively (not part of site build)
Modified:
commons/proper/csv/trunk/pom.xml
Modified: commons/proper/csv/trunk/pom.xml
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/pom.xml?rev=1485830&r1=1485829&r2=1485830&view=diff
==============================================================================
--- commons/proper/csv/trunk/pom.xml (original)
+++ commons/proper/csv/trunk/pom.xml Thu May 23 19:15:59 2013
@@ -123,6 +123,11 @@ CSV files of various types.
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<commons.encoding>UTF-8</commons.encoding>
+ <!--
+ Use 2.9.1 instead of 2.10; 2.10 seems to scan the 'target' dir
+ and wants license headers in .properties to be the header for Java files.
+ -->
+ <checkstyle.version>2.9.1</checkstyle.version>
<checkstyle.header.file>${basedir}/LICENSE-header.txt</checkstyle.header.file>
</properties>
@@ -147,22 +152,31 @@ CSV files of various types.
</excludes>
</configuration>
</plugin>
+ <!-- Allow checkstyle to be run interactively; keep in sync with report config below -->
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-checkstyle-plugin</artifactId>
+ <version>${checkstyle.version}</version>
+ <configuration>
+ <configLocation>${basedir}/checkstyle.xml</configLocation>
+ <enableRulesSummary>false</enableRulesSummary>
+ <headerLocation>${checkstyle.header.file}</headerLocation>
+ </configuration>
+ </plugin>
</plugins>
</build>
<reporting>
<plugins>
+ <!-- Keep in sync with build config above -->
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
- <!-- Use 2.9.1 instead of 2.10; 2.10 seems to scan the 'target'
- dir and wants license headers in .properties to be the header for
- Java files. -->
- <version>2.9.1</version>
+ <version>${checkstyle.version}</version>
<configuration>
<configLocation>${basedir}/checkstyle.xml</configLocation>
<enableRulesSummary>false</enableRulesSummary>
- <headerLocation>${basedir}/LICENSE-header.txt</headerLocation>
+ <headerLocation>${checkstyle.header.file}</headerLocation>
</configuration>
<!-- We need to specify reportSets because 2.9.1 creates two reports -->
<reportSets>
@@ -198,7 +212,7 @@ CSV files of various types.
</tags>
</configuration>
</plugin>
- <!--
+ <!--
NOTE: this requires Maven 3.0.x; when used with Maven 2.2.1, the following error is seen:
Embedded error: Error rendering Maven report: org.dom4j.DocumentFactory cannot be cast to org.dom4j.DocumentFactory
-->