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 2014/10/18 12:43:41 UTC

svn commit: r1632749 - in /maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools: DistPrerequisiteMojo.java GetPrerequisites.java

Author: hboutemy
Date: Sat Oct 18 10:43:40 2014
New Revision: 1632749

URL: http://svn.apache.org/r1632749
Log:
removed extra-/ in calculated url

Modified:
    maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistPrerequisiteMojo.java
    maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/GetPrerequisites.java

Modified: maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistPrerequisiteMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistPrerequisiteMojo.java?rev=1632749&r1=1632748&r2=1632749&view=diff
==============================================================================
--- maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistPrerequisiteMojo.java (original)
+++ maven/sandbox/trunk/dist-tools/dist-tool-plugin/src/main/java/org/apache/maven/dist/tools/DistPrerequisiteMojo.java Sat Oct 18 10:43:40 2014
@@ -127,7 +127,9 @@ public class DistPrerequisiteMojo
             {
                 sink.tableRow();
                 sink.tableCell();
+                sink.link( prerequisites.getPluginInfoUrl( mavenJDKInformation.getPluginName() ) );
                 sink.text( mavenJDKInformation.getPluginName() );
+                sink.link_();
                 sink.text( " " );
                 sink.text( mavenJDKInformation.getPluginVersion() );
                 sink.tableCell_();

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=1632749&r1=1632748&r2=1632749&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 Sat Oct 18 10:43:40 2014
@@ -93,10 +93,15 @@ public class GetPrerequisites
 
     public String BASEURL = "http://maven.eu.apache.org/plugins/";
 
-    public MavenJDKInformation getMavenJdkInformation( String baseURL, String pluginName )
+    public String getPluginInfoUrl( String pluginName )
+    {
+        return BASEURL + pluginName + "/plugin-info.html";
+    }
+
+    public MavenJDKInformation getMavenJdkInformation( String pluginName )
         throws IOException
     {
-        String url = baseURL + "/" + pluginName + "/plugin-info.html";
+        String url = getPluginInfoUrl( pluginName );
 
         Document doc = Jsoup.connect( url ).get();
 
@@ -150,7 +155,7 @@ public class GetPrerequisites
         {
             try
             {
-                result.add( getMavenJdkInformation( BASEURL, pluginName ) );
+                result.add( getMavenJdkInformation( pluginName ) );
             }
             catch ( IOException e )
             {