You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Herve Boutemy (JIRA)" <ji...@codehaus.org> on 2014/06/08 11:02:10 UTC

[jira] (MJAVADOC-394) javadoc is not found properly by default under Oracle's JDK 7+ on Mac OSX

     [ https://jira.codehaus.org/browse/MJAVADOC-394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Herve Boutemy closed MJAVADOC-394.
----------------------------------

       Resolution: Fixed
    Fix Version/s: 2.9.2
         Assignee: Herve Boutemy

patch applied in [r1601204|http://svn.apache.org/r1601204]
thank you

> javadoc is not found properly by default under Oracle's JDK 7+ on Mac OSX
> -------------------------------------------------------------------------
>
>                 Key: MJAVADOC-394
>                 URL: https://jira.codehaus.org/browse/MJAVADOC-394
>             Project: Maven Javadoc Plugin
>          Issue Type: Bug
>    Affects Versions: 2.9.1
>         Environment: Mac OSX, JDK 1.7+
>            Reporter: Laird Nelson
>            Assignee: Herve Boutemy
>             Fix For: 2.9.2
>
>         Attachments: AbstractJavadocMojo.java.patch
>
>
> The logic to detect where the {{javadoc}} script is located is not correct for Oracle's JVM 1.7 and higher on Mac OSX.
> The logic inside of {{AbstractJavadocMojo}} currently special-cases all JVMs running on OSX (line 3534):
> {code:title=AbstractJavadocMojo.java}
>         else if ( SystemUtils.IS_OS_MAC_OSX )
>         {
>             javadocExe = new File( SystemUtils.getJavaHome() + File.separator + "bin", javadocCommand );
>         }
> {code}
> But as of JDK 1.7 as distributed by Oracle, the default "else" block should apply here (line 3538):
> {code:title=AbstractJavadocMojo.java}
>         else
>         {
>             javadocExe =
>                 new File( SystemUtils.getJavaHome() + File.separator + ".." + File.separator + "bin", javadocCommand );
>         }
> {code}
> The solution might be to modify line 3534 as follows (or perhaps also check for Oracle's vendor string as well--anyway, you get the idea):
> {code:title=AbstractJavadocMojo.java}
>         else if ( SystemUtils.IS_OS_MAC_OSX && SystemUtils.JAVA_VERSION_FLOAT < 1.7f )
>         {
>             javadocExe = new File( SystemUtils.getJavaHome() + File.separator + "bin", javadocCommand );
>         }
> {code}
> Patch forthcoming.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)