[
https://issues.apache.org/jira/browse/MNG-7532?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17793606#comment-17793606
]
ASF GitHub Bot commented on MNG-7532:
-------------------------------------
cstamas commented on PR #793:
URL: https://github.com/apache/maven/pull/793#issuecomment-1842559579
This thread is full of misconceptions. Slf4j API worked and works. I bet
99.999 of our users use logger just for that, for logging. What Romain is
talking about is setting up your own logger implementation and so on.
First, BEFORE we do anything about this, can we see some numbers (like Mojos
"just logging" vs "setting up their own loggers as Romain explains")? As I feel
we have the situation of "it would be nice to do it", but the actual benefit
would be beneficial only for 1-2 mojo coders?
Take away simplicity and availability of SLF4J API _only due someone
somewhere fiddles with logging implementations_? Would be like forcing
left-handed scissors on everyone, because there are left handed people out
there (I am one of them).
As always, and since eons, if you want something super-duper-uber-smart
stuff, go for it, just isolate it in your own classloader, no problem here.
For me this PR is bicycle shedding at it's best.
> Revert MNG-6931 deprecation since list shows no consensus on that
> -----------------------------------------------------------------
>
> Key: MNG-7532
> URL: https://issues.apache.org/jira/browse/MNG-7532
> Project: Maven
> Issue Type: Task
> Reporter: Romain Manni-Bucau
> Priority: Major
>
> There are several threads on the dev list asking for the drop of slf4j and at
> least keeping a logging abstraction for not internal dev (= core can use
> slf4j but not mojo/extensions).
> Work is being done to abstract plugin api so let's keep the
> deprecation/replacement of our Log API in this track and keep it the official
> way for now.
> one of the multiple refs:
> https://www.mail-archive.com/[email protected]/msg123452.html
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
