svn commit: r1495601 - in /maven/sandbox/trunk/dist-tools/dist-tool-plugin: src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java test.sh

[sk...@apache.org]

Author: skygo
Date: Fri Jun 21 21:14:22 2013
New Revision: 1495601

URL: http://svn.apache.org/r1495601
Log:
reduce a bit error log
remove os specific file :p 

Removed:
    maven/sandbox/trunk/dist-tools/dist-tool-plugin/test.sh
Modified:
    maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java
    maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java

Modified: maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java?rev=1495601&r1=1495600&r2=1495601&view=diff
==============================================================================
--- maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java (original)
+++ maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSiteMojo.java Fri Jun 21 21:14:22 2013
@@ -433,8 +433,7 @@ public class DistCheckSiteMojo extends A
         catch ( Exception ex )
         {
             //continue for  other artifact
-            getLog().error( ex.getMessage() );
-            getLog().error( ex );
+            getLog().error( ex.getMessage() + configLine.getArtifactId() );
         }
 
     }
@@ -458,7 +457,8 @@ public class DistCheckSiteMojo extends A
             
             // central
             checkSite( configLine.getVersionnedPomFileURL( 
-                    repoBaseUrl, metadata.getVersioning().getLatest() ), configLine, metadata.getVersioning().getLatest() );
+                    repoBaseUrl, metadata.getVersioning().getLatest() ), 
+                    configLine, metadata.getVersioning().getLatest() );
 
         }
         catch ( IOException | XmlPullParserException ex )

Modified: maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java?rev=1495601&r1=1495600&r2=1495601&view=diff
==============================================================================
--- maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java (original)
+++ maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistCheckSourceReleaseMojo.java Fri Jun 21 21:14:22 2013
@@ -102,7 +102,7 @@ public class DistCheckSourceReleaseMojo
             older = checkRepos;
         }
     }
-    private List<DistCheckSourceRelease> results = new LinkedList<>();
+    private final List<DistCheckSourceRelease> results = new LinkedList<>();
 
     private void reportLine( Sink sink, DistCheckSourceRelease csr )
     {
@@ -363,9 +363,12 @@ public class DistCheckSourceReleaseMojo
 
         if ( !retrievedFile.isEmpty() )
         {
+            // write the following output in red so it's more readable in jenkins console
+            getLog().error( "Older version than " + version + " for "
+                    + configLine.getArtifactId() + " still available in " + repourl );
             for ( String sourceItem : retrievedFile )
             {
-                getLog().error( "Older than " + version + " version >> " + sourceItem + " << in " + repourl );
+                getLog().error( " > " + sourceItem + " <" );
             }
         }
 
@@ -393,9 +396,10 @@ public class DistCheckSourceReleaseMojo
 
         if ( !expectedFile.isEmpty() )
         {
+            getLog().error( "Missing archive for " + configLine.getArtifactId() + " in " + repourl );
             for ( String sourceItem : expectedFile )
             {
-                getLog().error( "Missing archive >> " + sourceItem + " << in " + repourl );
+                getLog().error( " > " + sourceItem + " <" );
             }
         }
 
@@ -419,12 +423,14 @@ public class DistCheckSourceReleaseMojo
             Collections.sort( metadata.getVersioning().getVersions(), Collections.reverseOrder() );
             getLog().debug( metadata.getVersioning().getVersions() + " version(s) detected " + repoBaseUrl );
             configLine.addMetadata( metadata );
-            DistCheckSourceRelease result = new DistCheckSourceRelease( configLine, metadata.getVersioning().getLatest() );
+            DistCheckSourceRelease result = new DistCheckSourceRelease( configLine, 
+                    metadata.getVersioning().getLatest() );
             results.add( result );
             // central
             result.setMissingCentralSourceRelease(
                     checkRepos( configLine.getVersionnedFolderURL(
-                    repoBaseUrl, metadata.getVersioning().getLatest() ), configLine, metadata.getVersioning().getLatest() ) );
+                    repoBaseUrl, metadata.getVersioning().getLatest() ), 
+                    configLine, metadata.getVersioning().getLatest() ) );
             //dist
             result.setMissingDistSourceRelease(
                     checkRepos( configLine.getDist(), configLine, metadata.getVersioning().getLatest() ) );