Maven 2 & 3

maven-toolchains missing from final binary.. need to update the assembly

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 2.0.9
  • Fix Version/s: 2.0.9
  • Component/s: Command Line
  • Labels:
    None
  • Number of attachments :
    0

Description

the newly added toolchains components are not packaged in the final binary fo 2.0.9

Activity

Hide
John Casey added a comment -

done, just needed to be added to the dependencyManagement in the maven top-level POM and the dependencies of the maven-core POM.

Show
John Casey added a comment - done, just needed to be added to the dependencyManagement in the maven top-level POM and the dependencies of the maven-core POM.
Hide
John Casey added a comment -

adds a cyclic dependency when added to maven-core...I didn't realize this. I need to move the assembly stuff out to a new module.

Show
John Casey added a comment - adds a cyclic dependency when added to maven-core...I didn't realize this. I need to move the assembly stuff out to a new module.
Hide
John Casey added a comment -

Milos, I have a working fix for this, but Brian mentioned you already had something you thought would work. If you have any trouble, let me know.

Show
John Casey added a comment - Milos, I have a working fix for this, but Brian mentioned you already had something you thought would work. If you have any trouble, let me know.
Hide
Milos Kleint added a comment -

i have split the assembly into a separate module, it depends on core and toolchain. it seems to work. Is your fix along those lines?

Show
Milos Kleint added a comment - i have split the assembly into a separate module, it depends on core and toolchain. it seems to work. Is your fix along those lines?
Hide
John Casey added a comment -

Mine does the same thing, though there are also changes that need to be done for the integration-test executor, and the src/bin, src/conf, and NOTICE.txt, etc. files need to be moved as well.

Show
John Casey added a comment - Mine does the same thing, though there are also changes that need to be done for the integration-test executor, and the src/bin, src/conf, and NOTICE.txt, etc. files need to be moved as well.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: