Maven 2.x Javadoc Plugin

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. File
    maven-javadoc-plugin.diff
    02/Nov/11 4:59 PM
    3 kB
    Ben Speakmon
  2. Text File
    MJAVADOC-334.patch
    17/Jul/12 10:56 PM
    4 kB
    Joseph Walton

Issue Links

relates to

New Feature - A new feature of the product, which has yet to be developed. MJAVADOC-322 Capabilty to add additionnal dependencies when running javadoc

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
Olivier Lamy added a comment -

Patch applied.
Thanks!

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

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Ben Speakmon
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: