[ 
https://issues.apache.org/jira/browse/MNG-7724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17697946#comment-17697946
 ] 

Elliotte Rusty Harold commented on MNG-7724:
--------------------------------------------

The code in question is a bit of a mess. I think 
UnsupportedSlf4jBindingConfiguration.activate should be a no-op and the 
warning, if any, belongs in Slf4jConfigurationFactory.getConfigurationFactory. 
However my gut is this shouldn't even be a warning. I'd accept no warning at 
all here, or just maybe a debug level warning. Info is too high. 




> Don't log warnings when runtime is not broken (slf4j integrations)
> ------------------------------------------------------------------
>
>                 Key: MNG-7724
>                 URL: https://issues.apache.org/jira/browse/MNG-7724
>             Project: Maven
>          Issue Type: Improvement
>    Affects Versions: 4.0.0-alpha-4
>            Reporter: Romain Manni-Bucau
>            Priority: Minor
>
> As of now if we change the SLF4J bindings maven will issue warnings because a 
> few features are disabled but due to the nature of switching these bindings 
> it is highly likely it is intended and therefore the warning are misleading 
> more than helping.
> The solution can be to log a warning if a configured factory fails to load 
> but just log info when it succeeds.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to