gnodet commented on pull request #619:
URL: https://github.com/apache/maven/pull/619#issuecomment-977836932


   > `-Dorg.slf4j.simpleLogger.log.Sisu=trace` (as system property, e.g. via 
MAVEN_OPTS) didn't work? If this does work (or something similar) I wouldn't 
add a new flag just for simplifying the edge case.
   
   It does I suppose.  
   The problem wasn't really in enabling the logging, rather that, at the time 
I could understand that I had to raise the logging, I already had fully 
analysed / understood what the problem was and how to fix it, i.e. find that 
the problem comes from the injection, debugging the Sisu code, understand where 
the problem is located and find out I could have raised the log level.  
   I expected the `-X` option to show any problem, but it did not, hence my 
original proposal to raise `-X` to display `TRACE` level.  After this solution, 
I'd favour fixing https://github.com/eclipse/sisu.inject/issues/36, then this 
PR if none of the previous solution is acceptable. 


-- 
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