Maven 2 & 3

Maven Embedder incorrectly resolves properties in multiprojects and incorrectly resolves dependency management for POM dependencies

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Cannot Reproduce
  • Affects Version/s: 3.0
  • Fix Version/s: None
  • Component/s: Embedding
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

Maven Embedder incorrectly resolves properties in multiprojects and incorrectly resolves dependency management for POM dependencies

This was first noticed using IntelliJ embedder - see JIRA's are

The first issue prevents maven embedder from parsing a child POM that uses a parent POM to set the version of an artifact, if the parent POM using properties to set the version.

The second issue shows even if you set the version explicitly the parent POM's dependency management section does not cascade in embedder (but it does in maven) correctly for POM type dependencies.

I have written a unit test (attached) to embedder to test for this. But am unsure how to fix.

Activity

Hide
Ben Gidley added a comment -

Further experimentation has shown the properties issue only effect items whose type is NOT jar.

Show
Ben Gidley added a comment - Further experimentation has shown the properties issue only effect items whose type is NOT jar.
Hide
Milos Kleint added a comment -

which version of maven-embedder is IDEA using? I doubt it's 2.0.8, as the embedder code in 2.0.x branch is rather unusable. It's more likely a 2.1-SNAPSHOT of sorts.

Show
Milos Kleint added a comment - which version of maven-embedder is IDEA using? I doubt it's 2.0.8, as the embedder code in 2.0.x branch is rather unusable. It's more likely a 2.1-SNAPSHOT of sorts.
Hide
Anton Makeev added a comment -

Yes, we use 2.1-SNAPSHOT. Currently, the bundled version is of 2008-02-12.

Show
Anton Makeev added a comment - Yes, we use 2.1-SNAPSHOT. Currently, the bundled version is of 2008-02-12.
Hide
Jason van Zyl added a comment -

I believe this has been fixed. Reopen if you find the issue still persists.

Show
Jason van Zyl added a comment - I believe this has been fixed. Reopen if you find the issue still persists.

People

Vote (1)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: