You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Thomas Watson (JIRA)" <ji...@apache.org> on 2019/02/15 20:29:00 UTC

[jira] [Resolved] (FELIX-6056) SCR exports promises

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

Thomas Watson resolved FELIX-6056.
----------------------------------
       Resolution: Fixed
         Assignee: Thomas Watson
    Fix Version/s: scr-2.1.16

> SCR exports promises
> --------------------
>
>                 Key: FELIX-6056
>                 URL: https://issues.apache.org/jira/browse/FELIX-6056
>             Project: Felix
>          Issue Type: Bug
>          Components: Declarative Services (SCR)
>            Reporter: Peter Kriens
>            Assignee: Thomas Watson
>            Priority: Major
>             Fix For: scr-2.1.16
>
>
> Apache Felix SCR exports promises 1.0 as a convenience. However, since this package contains the implementation it is a +provider+. Originally, the import was correct for a provider, [1,1.1). however, last July it was increased to [1,2). This therefore against the semantic versioning specification that prescribes a minor import range when providing the implementation of an API.
> Exporting an package like Promises is not a good idea anyway. It is only a convenience, promises have nothing to do with SCR in itself and that will always back fire in the end. Even exporting it correctly causes unnecessary version constraints since the order of resolving cannot be controlled. So if you have Promises 1.1 it depends on the order of resolving if SCR uses 1.0 or 1.1 and it creates nasty class spaces.
> It is inconvenient to make SCR depend on Promises. Personally I would not have used promises or would have included the package privately. (No API of SCR depends on Promises) Since SCR is such a crucial base component I strongly recommend to not export it from SCR. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)