Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.5
    • Labels:
      None
    • Environment:
      Windows XP and non-Windows

      Description

      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.

        Activity

        Greg Thompson created issue -
        Siveton Vincent made changes -
        Field Original Value New Value
        Fix Version/s 2.4 [ 13630 ]
        Wendy Smoak made changes -
        Fix Version/s 2.4 [ 13630 ]
        Benjamin Bentmann made changes -
        Attachment javadoc-executable.patch [ 33066 ]
        Benjamin Bentmann made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Assignee Benjamin Bentmann [ bentmann ]
        Fix Version/s 2.5 [ 14120 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 11:56:47 UTC 2015 [ 1428235007093 ]
        Mark Thomas made changes -
        Workflow jira [ 12722399 ] Default workflow, editable Closed status [ 12762615 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 00:11:46 UTC 2015 [ 1428279106587 ]
        Mark Thomas made changes -
        Workflow jira [ 12959923 ] Default workflow, editable Closed status [ 12996823 ]

          People

          • Assignee:
            Benjamin Bentmann
            Reporter:
            Greg Thompson
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development