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 Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.1
    • Fix Version/s: 2.9
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      2

      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

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

          Patch applied.
          Thanks!

          Show
          Olivier Lamy added a comment - Patch applied. Thanks!

            People

            • Assignee:
              Olivier Lamy
              Reporter:
              Ben Speakmon
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: