[ 
https://issues.apache.org/jira/browse/SLING-1739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12907709#action_12907709
 ] 

Justin Edelson commented on SLING-1739:
---------------------------------------

-1 on exporting spi and helpers. These packages are for implementers of SLF4J, 
not API consumers. The only exports from commons log should be logging APIs. If 
we decide to expose an SPI, that should be done in a Sling package and then 
mapped internally to the bundle to the SPI of whatever implementation we happen 
to be using at the time. 

> Update Commons Log exported SLF4J version
> -----------------------------------------
>
>                 Key: SLING-1739
>                 URL: https://issues.apache.org/jira/browse/SLING-1739
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>    Affects Versions: Commons Log 2.0.6
>            Reporter: Jason Rose
>            Assignee: Felix Meschberger
>            Priority: Minor
>             Fix For: Commons Log 2.0.8
>
>         Attachments: commons-log.patch
>
>
> Commons Log is currently exporting version 1.5.2 of SLF4j.  This is a very 
> old version of the library.  I'm currently using Commons Logging as a 
> replacement for PAX Logging because of classloading issues with PAX in JBoss, 
> and I needed to make a couple changes to facilitate that.
> Updating the dependency and exported versions of SLF4j fixes most of the 
> issues I have with trying to exclusively use Commons Log.  In addition to 
> updating the version, I've moved the spi and helpers packages into the 
> exported packages instead of private packages.  This makes it mirror PAX more 
> closely and handles all the logging requirements of 3rd party bundles in my 
> application.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to