Author: hboutemy
Date: Wed Nov 12 23:53:49 2014
New Revision: 1639199
URL: http://svn.apache.org/r1639199
Log:
updated parent pom and fixed errors reported by Checkstyle
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/pom.xml
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckIndexPageMojo.java
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ListPluginsPrerequisitesMojo.java
Modified: maven/sandbox/trunk/dist-tools/dist-tool-plugin/pom.xml
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/pom.xml?rev=1639199&r1=1639198&r2=1639199&view=diff
==============================================================================
--- maven/sandbox/trunk/dist-tools/dist-tool-plugin/pom.xml (original)
+++ maven/sandbox/trunk/dist-tools/dist-tool-plugin/pom.xml Wed Nov 12 23:53:49
2014
@@ -17,7 +17,7 @@
<parent>
<groupId>org.apache.maven</groupId>
<artifactId>maven-parent</artifactId>
- <version>24</version>
+ <version>25</version>
<relativePath>../../../pom/maven/pom.xml</relativePath>
</parent>
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckIndexPageMojo.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckIndexPageMojo.java?rev=1639199&r1=1639198&r2=1639199&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckIndexPageMojo.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckIndexPageMojo.java
Wed Nov 12 23:53:49 2014
@@ -204,6 +204,7 @@ public class DistCheckIndexPageMojo
Date d1 = df.parse( date1 );
Date d2 = df.parse( date2 );
+ @SuppressWarnings( "checkstyle:magicnumber" )
long daysDifference = ( d1.getTime() - d2.getTime() ) / ( 24 * 60
* 60 * 1000 );
return Math.abs( daysDifference ) < 7; // ok for 7 days difference
}
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java?rev=1639199&r1=1639198&r2=1639199&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java
Wed Nov 12 23:53:49 2014
@@ -40,7 +40,7 @@ public class GetPrerequisites
/**
* Currently hard code should be somehow extracted from the configuration
file....
*/
- public String[] pluginNames = {
+ public static final String[] PLUGIN_NAMES = {
"maven-acr-plugin",
"maven-ant-plugin",
"maven-antrun-plugin",
@@ -92,7 +92,7 @@ public class GetPrerequisites
"maven-war-plugin",
};
- public String BASEURL = "http://maven.eu.apache.org/plugins/";;
+ private static final String BASEURL = "http://maven.apache.org/plugins/";;
public String getPluginInfoUrl( String pluginName )
{
@@ -120,10 +120,10 @@ public class GetPrerequisites
}
Element tableInfo = select.get( 1 );
- Elements elementsByAttribute_a =
tableInfo.getElementsByAttributeValue( "class", "a" );
- Elements elementsByAttribute_b =
tableInfo.getElementsByAttributeValue( "class", "b" );
- String mavenVersion = elementsByAttribute_a.first().text();
- String jdkVersion = elementsByAttribute_b.first().text();
+ Elements elementsByAttributeA = tableInfo.getElementsByAttributeValue(
"class", "a" );
+ Elements elementsByAttributeB = tableInfo.getElementsByAttributeValue(
"class", "b" );
+ String mavenVersion = elementsByAttributeA.first().text();
+ String jdkVersion = elementsByAttributeB.first().text();
//FIXME: Sometimes it happens that the indexes are swapped (I don't
know why...I have to find out why...)
if ( mavenVersion.startsWith( "JDK" ) )
@@ -156,7 +156,7 @@ public class GetPrerequisites
{
List<PluginPrerequisites> result = new
ArrayList<PluginPrerequisites>();
- for ( String pluginName : pluginNames )
+ for ( String pluginName : PLUGIN_NAMES )
{
try
{
Modified:
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ListPluginsPrerequisitesMojo.java
URL:
http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ListPluginsPrerequisitesMojo.java?rev=1639199&r1=1639198&r2=1639199&view=diff
==============================================================================
---
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ListPluginsPrerequisitesMojo.java
(original)
+++
maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/ListPluginsPrerequisitesMojo.java
Wed Nov 12 23:53:49 2014
@@ -103,22 +103,22 @@ public class ListPluginsPrerequisitesMoj
Collections.<ArtifactVersion>sort( sortedVersion );
- for ( ArtifactVersion mavenVersion : sortedVersion)
+ for ( ArtifactVersion mavenVersion : sortedVersion )
{
List<PluginPrerequisites> pluginsPrerequisites =
groupedPrequisites.get( mavenVersion );
sink.tableRow();
sink.tableHeaderCell();
sink.rawText( "Maven Version Prerequisite " + mavenVersion + " ("
+ pluginsPrerequisites.size() + " / "
- + prerequisites.pluginNames.length + ")" );
+ + prerequisites.PLUGIN_NAMES.length + ")" );
sink.tableHeaderCell_();
sink.tableHeaderCell();
- sink.rawText( "Maven Version");
+ sink.rawText( "Maven Version" );
sink.tableHeaderCell_();
sink.tableHeaderCell();
- sink.rawText( "JDK Version");
+ sink.rawText( "JDK Version" );
sink.tableHeaderCell_();
sink.tableRow_();
