Maven 2 & 3

Reactor projects should be included in dependency resolution

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.7
  • Fix Version/s: 2.1.0
  • Component/s: Dependencies
  • Labels:
    None
  • Complexity:
    Intermediate
  • Testcase included:
    yes
  • Number of attachments :
    1

Description

Dependency resolution for mojos with @requiresDependencyResolution should include the projects within the reactor. See attached it0122 for an example (numbered as such due to MNG-2994's it0121).

Issue Links

Activity

Hide
Brett Porter added a comment -

will review patch for 2.0.7 since it has a test and all

Show
Brett Porter added a comment - will review patch for 2.0.7 since it has a test and all
Hide
Brett Porter added a comment -

my bad, it's just the test - no fix yet.

Show
Brett Porter added a comment - my bad, it's just the test - no fix yet.
Hide
Brett Porter added a comment -

patch applied for test

Show
Brett Porter added a comment - patch applied for test
Hide
Samuel Le Berrigaud added a comment -

This sounds like a critical issue to me. It is impossible to work with multi-module projects if dependencies resolution goes to the repositories instead of the reactor. An good example of this is working with the release plugin (version 2.0-beta-7 IIRC):

1. I do a clean checkout on a build server,
2. I run mvn release:prepare

And as release:prepare relies on the @requiresDependencyResolution, it fails because the SNAPSHOT is not installed locally.

Show
Samuel Le Berrigaud added a comment - This sounds like a critical issue to me. It is impossible to work with multi-module projects if dependencies resolution goes to the repositories instead of the reactor. An good example of this is working with the release plugin (version 2.0-beta-7 IIRC): 1. I do a clean checkout on a build server, 2. I run mvn release:prepare And as release:prepare relies on the @requiresDependencyResolution, it fails because the SNAPSHOT is not installed locally.
Hide
Brett Porter added a comment -

this constantly proves a nuisance - worth reviewing

Show
Brett Porter added a comment - this constantly proves a nuisance - worth reviewing
Hide
John Casey added a comment -

Consolidating to 2.1.0-M1 so we can then rename to 2.1.0. We can weed out any issues we want to push to a later release from this set once we've done the consolidation.

Show
John Casey added a comment - Consolidating to 2.1.0-M1 so we can then rename to 2.1.0. We can weed out any issues we want to push to a later release from this set once we've done the consolidation.
Hide
Zac Thompson added a comment - - edited

I still see this behaviour in 2.1.0. (Samuel's example above)

... update: actually, I think this is because of a test dependency, which is posted as a different issue. My apologies!

Show
Zac Thompson added a comment - - edited I still see this behaviour in 2.1.0. (Samuel's example above) ... update: actually, I think this is because of a test dependency, which is posted as a different issue. My apologies!

People

Vote (0)
Watch (7)

Dates

  • Created:
    Updated:
    Resolved: