Maven Javadoc Plugin
  1. Maven Javadoc Plugin
  2. MJAVADOC-334

Patch to make <additionalDependencies> easier and fix typo on field name

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.1
    • Fix Version/s: 2.9
    • Labels:
      None
    • Flags:
      Patch

      Description

      This patch corrects the spelling of <additionalDependencies> and adds the wrapper class org.apache.maven.plugin.javadoc.AdditionalDependency. This allows us to declare <additionalDependency> without also requiring us to use the implementation attribute to manually specify org.apache.maven.model.Dependency.

      1. maven-javadoc-plugin.diff
        3 kB
        Ben Speakmon
      2. MJAVADOC-334.patch
        4 kB
        Joseph Walton

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          270d 23h 37m 1 Olivier Lamy (*$^¨%`£) 30/Jul/12 17:36
          Mark Thomas made changes -
          Workflow jira [ 12960160 ] Default workflow, editable Closed status [ 12997039 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 00:11:46 UTC 2015 [ 1428279106587 ]
          Mark Thomas made changes -
          Flags Patch [ 10430 ]
          Patch Submitted Yes [ 10763 ]
          Mark Thomas made changes -
          Workflow jira [ 12722566 ] Default workflow, editable Closed status [ 12762340 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 11:56:47 UTC 2015 [ 1428235007093 ]
          Olivier Lamy (*$^¨%`£) made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Assignee Olivier Lamy [ olamy ]
          Resolution Fixed [ 1 ]
          Fix Version/s 2.9 [ 17249 ]
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          Patch applied.
          Thanks!

          Show
          Olivier Lamy (*$^¨%`£) added a comment - Patch applied. Thanks!
          Olivier Lamy (*$^¨%`£) made changes -
          Summary Patch to make <additionalDependencies> easier Patch to make <additionalDependencies> easier and fix typo on field name
          Hide
          Ben Speakmon added a comment -

          We had it on 2.1.0 and 2.2.1. (Internal issues make moving to 3.x impossible for the time being.)

          Show
          Ben Speakmon added a comment - We had it on 2.1.0 and 2.2.1. (Internal issues make moving to 3.x impossible for the time being.)
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          @Ben which maven version are you using to have issues ??

          Show
          Olivier Lamy (*$^¨%`£) added a comment - @Ben which maven version are you using to have issues ??
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          @Joseph sure but not for maven 2.x users

          Show
          Olivier Lamy (*$^¨%`£) added a comment - @Joseph sure but not for maven 2.x users
          Joseph Walton made changes -
          Attachment MJAVADOC-334.patch [ 60628 ]
          Hide
          Joseph Walton added a comment -

          Adding a new class doesn't seem to be necessary now that config parsing considers the generic type of collections (SISU-60?), so I'm attaching a patch without that change, but which fixes the spelling for the mojo and for its integration tests.

          Show
          Joseph Walton added a comment - Adding a new class doesn't seem to be necessary now that config parsing considers the generic type of collections ( SISU-60 ?), so I'm attaching a patch without that change, but which fixes the spelling for the mojo and for its integration tests.
          Ben Speakmon made changes -
          Field Original Value New Value
          Link This issue relates to MJAVADOC-322 [ MJAVADOC-322 ]
          Ben Speakmon created issue -

            People

            • Assignee:
              Olivier Lamy (*$^¨%`£)
              Reporter:
              Ben Speakmon
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development