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

Oliver Lietz commented on SLING-4537:
-------------------------------------

I'm not happy with the current implementation in trunk and there are also still 
parts which should be cleaned up (in trunk and also in whiteboard). With the 
refactoring of Content Loader an increased API version is required (see 
baseline output).

Servlets Post require only a minor internal change regarding import providers 
({{public boolean isIgnoredImportProvider(String extension)}} -> {{public 
Set<String> getIgnoreImportProviders()}} in {{ImportOperation}} line 190).

I suggest rolling back to a version being more 2.1 than 2.2 in trunk and 
finishing refactoring of Content Loader in whiteboard before messing up further.

I think we do have APIs which are not backwards compatible elsewhere and force 
depending bundles to update, but I would like to hear other opinions.

> Provide a backwards compatible API in o.a.s.jcr.contentloader
> -------------------------------------------------------------
>
>                 Key: SLING-4537
>                 URL: https://issues.apache.org/jira/browse/SLING-4537
>             Project: Sling
>          Issue Type: Bug
>          Components: JCR
>            Reporter: Radu Cotescu
>            Assignee: Radu Cotescu
>             Fix For: JCR ContentLoader 2.2.0
>
>
> Changes proposed in SLING-3534 introduced a major API change which is not 
> really needed. Instead we should increase the API version from 0.1 to 0.2 
> (instead of 1.0.0) in order to allow upgrades of the 
> {{org.apache.sling.jcr.contentloader}} bundle independent of upgrades of the 
> {{org.apache.sling.servlets.post}}:
> {noformat}
> org.osgi.framework.BundleException: Unresolved constraint in bundle 
> org.apache.sling.jcr.jackrabbit.usermanager [55]: Unable to resolve 55.0: 
> missing requirement [55.0] osgi.wiring.package; 
> (&(osgi.wiring.package=org.apache.sling.servlets.post)(version>=2.1.0)(!(version>=3.0.0)))
>  [caused by: Unable to resolve 112.0: missing requirement [112.0] 
> osgi.wiring.package; 
> (&(osgi.wiring.package=org.apache.sling.jcr.contentloader)(version>=0.1.0)(!(version>=1.0.0)))]
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to