AsbtractMojo should look for LoggerManager plexus component

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major 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

is related to

Improvement - An improvement or enhancement to an existing feature or task. MNG-2570 Maven needs to support multiple logging levels

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. MNG-3183 Allow a user to specify logging to a text file

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
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.
Hide
Permalink
Benjamin Bentmann added a comment -

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

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

People

  • Assignee:
    Benjamin Bentmann
    Reporter:
    Robert Egan
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: