Uploaded image for project: 'Maven'
  1. Maven
  2. MNG-3464

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

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 2.0.9
    • 2.0.9
    • Command Line, Toolchains
    • None

    Description

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

      Attachments

        Activity

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

          jdcasey John Dennis 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.

          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.

          jdcasey John Dennis 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.

          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.

          jdcasey John Dennis 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.
          mkleint 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?

          mkleint 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?

          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.

          jdcasey John Dennis 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.
          mkleint Milos Kleint added a comment - http://svn.apache.org/viewvc?rev=637648&view=rev
          mkleint Milos Kleint added a comment - and http://svn.apache.org/viewvc?rev=637657&view=rev

          People

            mkleint Milos Kleint
            mkleint Milos Kleint
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: