Fix Logging in threadsafe components

Shouldn't the component use a monitor instead of logging in the cases where the output is useful on a per-build basis?

If it's relavent to the API of the application yes, if you want debug output or general tracing then this falls outside the monitor. Though this is a general problem that would be solved using a more direct use of log4j and it's NDC facilities.

log4j has been unchanged (=dead?) for 4 years.
It's creator created slf4j with logback (AKA log4j 2), similar license, much better code, still maintained, but due to politics it never got the name "log4j 2".

Maven should use its own logging facade which then is backed by logback (which supports NDC). Then if Geki creates something even better down the road we can swap it under the covers without any issue.

See Spring Framework for the devastation depending on a specific logging mechanism can do.

FYI, SLF4J supports MDC but not NDC. There are counter arguments for making SLF4J an optional dependency: see http://slf4j.org/faq.html#optional_dependency

However, whether you choose to depend on SLF4J or go through your own facade, I am fairly certain that neither approach will be future proof.

I am a bit unsure of exactly which application use-case is being addressed in this issue;

For the current concurrency effort (MNG-3004) my current favourite strategy is replacing the LoggerManager with a ConcurrentLoggerManager that
provides a MavenProject-aware proxy for the existing logger subsystem. Such a proxy would typically be able to collect all logger output for
the project to a single outputstream, no matter how this intertwines with other builds.

A rough draft of this logic can be seen in LifecycleThreadedBuilder line 164 & 167, where the MavenProject is associated with the current thread (Code is commented out because the current implementation is not good enough). This association changes logger output, and the setThisModuleComplete releases the output.

I fully sense that there may be some other use cases, possibly related to embedding. But it seems like a solution that captures the "logical stream" that constitutes the output from a single build should be expandable to these different cases.

Stuart is taking care of this as part of moving over to Guice. We just need to inject a threadsafe logger. There's no point in doing this in Plexus and in Guice.

since the switch to Guice happened, is there anything still left to do here, or was it taken care of as suggested in the last comment?

I'm not really sure we need to do anything more about this issue, the 3.x logging seems to work fairly well in parallel.