You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/01/06 15:59:56 UTC

svn commit: r609325 - /maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java

Author: vsiveton
Date: Sun Jan  6 06:59:53 2008
New Revision: 609325

URL: http://svn.apache.org/viewvc?rev=609325&view=rev
Log:
MJAVADOC-164: Javadoc 1.4 fails due to missing directory in linkoffline option

o verify that the location of the linkoffline exists

Modified:
    maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java

Modified: maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java?rev=609325&r1=609324&r2=609325&view=diff
==============================================================================
--- maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java (original)
+++ maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java Sun Jan  6 06:59:53 2008
@@ -2490,10 +2490,17 @@
 
                 if ( p.getUrl() != null )
                 {
-                    OfflineLink ol = new OfflineLink();
-                    ol.setUrl( p.getUrl() + "/apidocs" );
-                    ol.setLocation( p.getBasedir().getAbsolutePath() + "/" + javadocDirRelative );
-                    offlineLinksList.add( ol );
+                    String url = p.getUrl() + "/apidocs";
+                    File location = new File( p.getBasedir(), javadocDirRelative );
+
+                    if ( location.exists() )
+                    {
+                        OfflineLink ol = new OfflineLink();
+                        ol.setUrl( url );
+                        ol.setLocation( location.getAbsolutePath() );
+
+                        offlineLinksList.add( ol );
+                    }
                 }
             }
         }