rmannibucau commented on issue #1028:
URL: 
https://github.com/apache/maven-compiler-plugin/issues/1028#issuecomment-3836041412

   > I think that this is a description of pull request 
https://github.com/apache/maven-compiler-plugin/pull/1027.
   
   yes just created it to ensure there is a ticket, sory if it was not clear
   
   > I don't think that this is a duplication of the compiler messages. This is 
a summary. 
   
   The fact is that you have twice the same info high level, I can see some 
value in both but not at the same time, therefore the enum to enable to select 
your preferred style - don't think everyone will want the same.
   
   > but it a task for logging configuration
   
   I don't like the fact to make a ton of logger for that purpose and I prefer 
to keep loggers 1-1 with classes to keep things simple and understandable while 
we do use simple logger (fork).
   
   > System.getLogger("org.apache.maven.compiler.summary")
   
   default impl is JUL and we don't use JUL at all so it would be very not that 
common for end users.
   it also has a very negative effect: now plugins have a broken logging 
configuration when they do rely on JUL configuration so I'm very mixed about it 
for now (to be transparent was a pain before already with LogManager)
   we did decide to stick to slf4j as backend and have their own API and I 
think we shouldn't extend it too much if we don't want to break/limit plugin 
writers more.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to