[jira] Commented: (MJAVADOC-162) javadocExecutable unusable

["Wendy Smoak (JIRA)" <ji...@codehaus.org>]

    [ http://jira.codehaus.org/browse/MJAVADOC-162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_126243 ] 

Wendy Smoak commented on MJAVADOC-162:
--------------------------------------

Vincent, you marked this for 2.4 back in January.  Are you planning to work on it, or should we bump it to 2.5?  I'd like to stage 2.4 this weekend.

> javadocExecutable unusable
> --------------------------
>
>                 Key: MJAVADOC-162
>                 URL: http://jira.codehaus.org/browse/MJAVADOC-162
>             Project: Maven 2.x Javadoc Plugin
>          Issue Type: Bug
>    Affects Versions: 2.3
>         Environment: Windows XP and non-Windows
>            Reporter: Greg Thompson
>             Fix For: 2.4
>
>
> AbstractJavadocMojo tries to be smart by seeing if the file indicated by javadocExecutable exists, but this is actually quite problematic.  If you put the following in your config:
> <javadocExecutable>blahblahblah${file.separator}javadoc</javadocExecutable>
> then you'll get an error on Windows since the file is actually javadoc.exe, which is a pain since it's perfectly acceptable to omit the .exe when executing a command.
> If you put .exe in the config, then it won't work on non-Windows platforms since the files doesn't have .exe.
> Forcing users to add hackery to their POMs to add .exe in some cases and leave it off in others is onerous.
> Methinks it's much preferable to simply use the path provided by the config.  If it doesn't exist, let CommandLineUtils.executeCommandLine (or something else) throw an exception.

-- 
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