Maven 2 & 3

Plugin dependencies should be considered when the reactor creates the build order list

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.4
  • Fix Version/s: 2.1.0
  • Component/s: Reactor and workspace
  • Labels:
    None
  • Complexity:
    Intermediate
  • Patch Submitted:
    Yes
  • Number of attachments :
    2

Description

In one project, I am using the checkstyle plugin with a plugin dependency on a build-tools module that produces a jar. Maven doesn't recognize that this build-tools module must be built before the module using the checkstyle plugin which has this dependency.

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. MNG-4110 cyclic reference with 2.1.0 that doesn't matter with 2.0.10

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

Bug - A problem which impairs or prevents the functions of the product. MNG-4074 cyclic reference with 2.1.0-RC1 that doesn't occur with 2.0.10

  • 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
Tibor Varga added a comment -

I took a look to see if I could fix that problem and came up with a simple patch (attached).

Could a Maven developer please check the patch and comment?

Show
Tibor Varga added a comment - I took a look to see if I could fix that problem and came up with a simple patch (attached). Could a Maven developer please check the patch and comment?
Hide
Permalink
Tibor Varga added a comment - - edited

My previous patch only worked if the plugin declaring the dependency was itself a project in the reactor.

The new patch (#1, I believe) should work with any plugin. Please disregard the first patch.

Sorry about the confusion... Please check and comment.

Show
Tibor Varga added a comment - - edited My previous patch only worked if the plugin declaring the dependency was itself a project in the reactor. The new patch (#1, I believe) should work with any plugin. Please disregard the first patch. Sorry about the confusion... Please check and comment.
Hide
Permalink
Brett Porter added a comment -

applied to Maven 2.1.x branch - will close once documentation of the change has been added

Show
Brett Porter added a comment - applied to Maven 2.1.x branch - will close once documentation of the change has been added

People

  • Assignee:
    Brett Porter
    Reporter:
    Vincent Massol
Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: