[ 
https://issues.apache.org/jira/browse/SLING-1482?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carsten Ziegeler resolved SLING-1482.
-------------------------------------

    Resolution: Fixed

I've tested this fix - it seems to work as advertised

> Commons Log should not export OSGi Configuration Admin package but have a 
> dynamic dependency
> --------------------------------------------------------------------------------------------
>
>                 Key: SLING-1482
>                 URL: https://issues.apache.org/jira/browse/SLING-1482
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>    Affects Versions: Commons Log 2.0.6
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Commons Log 2.0.8
>
>
> Currently the Commons Log module exports the OSGi Configuration Admin package 
> to be able to register ManagedService[Factory] services. This is probably 
> very bad style but has been done to have the Log mechanism configurable as 
> soon as the Configuration Admin service is registered.
> There is a better solution to this problem though:
>   * The OSGi Configuration Admin package is imported using 
> DynamicImport-Package. This allows for
>        dynamic wiring only when the API is used.
>   * The ManagedService[Factory] services are registered as ServiceFactory 
> services. This means,
>        that the actual Configuration Admin API (ManagedService[Factory] 
> interfaces and ConfigurationException)
>        need only be wired, when these services are actually accessed -- which 
> in turn is the case only
>        when the ConfigurationAdmin service is registered and starts working.
> Thus we solve the exact problem: Commons Log can be active and will be 
> Configuration Admin configurable as soon as the ConfigurationAdmin service is 
> active.

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