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

Minor patches needed to run Maven3 with the replacement Guice-Plexus container

    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.0-beta-1
    • 3.0-beta-2
    • General
    • None
    • Patch

    Description

      The attached patch contains two minor changes that are a pre-req for running Maven3 with the new Guice-Plexus container:

      1) Fix incorrect role setting in LifecyclePluginAnalyzerImpl (test also updated to lookup the role)

      2) Rework ConcurrentBuildLogger to use JDK classes for the multimap instead of google-collections

      The last change is because while the Plexus container requires google-collections, the new Guice-Plexus container has no such dependency. Adding google-collections (~600k) to the distribution just for this single use of a multimap doesn't seem to be worthwhile imho.

      Attachments

        1. MAVEN_3_CORE_PATCHES_FOR_GUICE.txt
          4 kB
          Stuart McCulloch

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. MNG-4749 Move from Plexus to Guice as IoC container

          • Major - Major loss of function.
          • Closed

          Activity

            People

              bentmann Benjamin Bentmann
              mcculls Stuart McCulloch
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: