[ 
http://jira.codehaus.org/browse/MJAVADOC-211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=143855#action_143855
 ] 

Vincent Siveton commented on MJAVADOC-211:
------------------------------------------

This was already catched in the AbstractJavadocMojo#executeReport( Locale 
unusedLocale ) 

{noformat}
...
        File javadocOutputDirectory = new File( getOutputDirectory() );
        if ( javadocOutputDirectory.exists() && 
!javadocOutputDirectory.isDirectory() )
        {
            throw new MavenReportException( "IOException: " + 
getOutputDirectory() + " is not a directory." );
        }
        if ( javadocOutputDirectory.exists() && 
!javadocOutputDirectory.canWrite() )
        {
            throw new MavenReportException( "IOException: " + 
getOutputDirectory() + " is not writable." );
        }
{noformat}

So, if the output was not created the javadoc plugin should failed. 

Anyway, your commit is a safe one. Thanks!

> create the report output directory if it doesn't exist
> ------------------------------------------------------
>
>                 Key: MJAVADOC-211
>                 URL: http://jira.codehaus.org/browse/MJAVADOC-211
>             Project: Maven 2.x Javadoc Plugin
>          Issue Type: Bug
>    Affects Versions: 2.4
>            Reporter: Brett Porter
>            Assignee: Brett Porter
>             Fix For: 2.5
>
>
> currently, the javadoc report relies on maven to have created the reporting 
> base directory first. This need not be the case.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to