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

AsbtractMojo should look for LoggerManager plexus component

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.9
    • Component/s: Logging
    • Labels:
      None

      Description

      AbstractMojo currently defines it's own Log interface, hard coded to use System.out. This really restricts the logging capabilities of all plugins. It would be nice if it attempted to look up the plexus role "org.codehaus.plexus.logging.LoggerManager" first and only used System.out if that failed.

      Doing this would also go a long way towards resolving MNG-2570 and MNG-3305.

        Issue Links

          Activity

          Hide
          bentmann Benjamin Bentmann added a comment -

          Looking at the DefaultPluginManager, the mojo logger is already delegating to a plexus logger since ages.

          Show
          bentmann Benjamin Bentmann added a comment - Looking at the DefaultPluginManager , the mojo logger is already delegating to a plexus logger since ages.
          Hide
          brettporter Brett Porter added a comment -

          I'm not sure coupling to plexus is a good idea (even if most other things are). The embedder already does handle directing this to the plexus logger, not system.out specifically.

          Show
          brettporter Brett Porter added a comment - I'm not sure coupling to plexus is a good idea (even if most other things are). The embedder already does handle directing this to the plexus logger, not system.out specifically.

            People

            • Assignee:
              bentmann Benjamin Bentmann
              Reporter:
              regan Robert Egan
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development