You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2013/06/01 21:45:24 UTC

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

Author: hboutemy
Date: Sat Jun  1 19:45:24 2013
New Revision: 1488568

URL: http://svn.apache.org/r1488568
Log:
separate groupId and artifact columns to improve readability

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=1488568&r1=1488567&r2=1488568&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 Sat Jun  1 19:45:24 2013
@@ -213,7 +213,10 @@ public class DistCheckSiteMojo extends A
         sink.table();
         sink.tableRow();
         sink.tableHeaderCell();
-        sink.rawText( "groupId:artifactId (from conf file)" );
+        sink.rawText( "groupId" );
+        sink.tableHeaderCell_();
+        sink.tableHeaderCell();
+        sink.rawText( "artifactId" );
         sink.tableHeaderCell_();
         sink.tableHeaderCell();
         sink.rawText( "LATEST" );
@@ -236,7 +239,11 @@ public class DistCheckSiteMojo extends A
         {
             sink.tableRow();
             sink.tableCell();
-            sink.rawText( csr.getConfigurationLine().getGroupId() + ":" + csr.getConfigurationLine().getArtifactId() );
+            sink.rawText( csr.getConfigurationLine().getGroupId() );
+            sink.tableCell_();
+
+            sink.tableCell();
+            sink.rawText( csr.getConfigurationLine().getArtifactId() );
             sink.tableCell_();
 
             sink.tableCell();
@@ -336,7 +343,7 @@ public class DistCheckSiteMojo extends A
     @Override
     void checkArtifact( ConfigurationLineInfo configLine, String repoBaseUrl ) throws MojoExecutionException
     {
-        try (BufferedReader input = new BufferedReader( new InputStreamReader( new URL( configLine.getMetadataFileURL( repoBaseUrl ) ).openStream() ) ))
+        try ( BufferedReader input = new BufferedReader( new InputStreamReader( new URL( configLine.getMetadataFileURL( repoBaseUrl ) ).openStream() ) ) )
         {
             JAXBContext context = JAXBContext.newInstance( MavenMetadata.class );
             Unmarshaller unmarshaller = context.createUnmarshaller();

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=1488568&r1=1488567&r2=1488568&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 Sat Jun  1 19:45:24 2013
@@ -121,7 +121,10 @@ public class DistCheckSourceReleaseMojo 
         sink.table();
         sink.tableRow();
         sink.tableHeaderCell();
-        sink.rawText( "groupId:artifactId (from conf file)" );
+        sink.rawText( "groupId" );
+        sink.tableHeaderCell_();
+        sink.tableHeaderCell();
+        sink.rawText( "artifactId" );
         sink.tableHeaderCell_();
         sink.tableHeaderCell();
         sink.rawText( "LATEST" );
@@ -133,11 +136,14 @@ public class DistCheckSourceReleaseMojo 
         sink.rawText( "Dist" );
         sink.tableHeaderCell_();
         sink.tableRow_();
+
         for ( DistCheckSourceRelease csr : results )
         {
             sink.tableRow();
             sink.tableCell();
-            sink.rawText( csr.getConfigurationLine().getGroupId() + ":" );
+            sink.rawText( csr.getConfigurationLine().getGroupId() );
+            sink.tableCell_();
+            sink.tableCell();
             sink.rawText( csr.getConfigurationLine().getArtifactId() );
             sink.tableCell_();
             sink.tableCell();