Author: niallp
Date: Thu May 22 23:14:00 2008
New Revision: 659440
URL: http://svn.apache.org/viewvc?rev=659440&view=rev
Log:
COMMONSSITE-30 - fix the checkstyle configuration to work for multi-module and
review/update <reporting> plugin version numbers for the poms being changed.
- pom's need to use ${basedir} for configuration files as the multi-module
profile is run from commons-parent, so if not specified it can't find those
files
- the checkstyle configuration file needs to use ${checkstyle.header.file} for
the headerFile property and <headerFile> element added to the plugin config in
the pom.xml
- fix m1 checkstyle config to reflect the above where appropriate (specify
header file property)
Modified:
commons/proper/jexl/branches/2.0/pom.xml
Modified: commons/proper/jexl/branches/2.0/pom.xml
URL:
http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/pom.xml?rev=659440&r1=659439&r2=659440&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0/pom.xml (original)
+++ commons/proper/jexl/branches/2.0/pom.xml Thu May 22 23:14:00 2008
@@ -169,6 +169,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-changes-plugin</artifactId>
+ <version>2.0</version>
<configuration>
<xmlPath>${basedir}/xdocs/changes.xml</xmlPath>
<issueLinkTemplate>%URL%/%ISSUE%</issueLinkTemplate>
@@ -184,22 +185,23 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
+ <version>2.1</version>
<configuration>
- <configLocation>src/conf/checkstyle.xml</configLocation>
+ <configLocation>${basedir}/src/conf/checkstyle.xml</configLocation>
<excludes>org/apache/commons/jexl/parser/*.java</excludes>
- <headerFile>src/conf/header.txt</headerFile>
+ <headerFile>${basedir}/src/conf/header.txt</headerFile>
<enableRulesSummary>false</enableRulesSummary>
</configuration>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>cobertura-maven-plugin</artifactId>
- <version>2.1</version>
+ <version>2.2</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-pmd-plugin</artifactId>
- <version>2.2</version>
+ <version>2.3</version>
<configuration>
<excludes>
<exclude>org/apache/commons/jexl/parser/*.java</exclude>
