[GitHub] [cxf] jimma commented on pull request #999: Remove osgi, blueprint and karaf from CXF

[GitBox <gi...@apache.org>]

jimma commented on PR #999:
URL: https://github.com/apache/cxf/pull/999#issuecomment-1257373341

   Thanks for the review @reta 
   
   > @jimma massive one, thanks a lot for doing it, a couple of questions / suggestion please:
   > 
   > * for commented code snippets, could we mark them all with some unique label, like `// TODO: [OSGi+Jakarta] ...`, it will help us to identify all places in code which needs to be revisited
   
   Good idea. I already updated the comment. 
   
   > * for META-INF manifests, do you think we could keep the `</cxf.osgi.export>`, `<cxf.osgi.import>` and others? It would certainly not make deployable (no Jakarta support for OSGi) but would save us some time when we would need to bring this back
   
   From these META-INF manifests, we'll have to review and modify them when osgi jakarta is ready. For example, there is no need to add the jakarta.jws package and it has been already included in jakarta.xml.ws since 4.0.0(Jakarta EE10). There will be probably some changes in other package name . These osgi import/export menifests should be reviewed when we brings the osgi code back.   
   ```
           <cxf.osgi.import>
               jakarta.jws*;version="${cxf.osgi.jws.version}",
               jakarta.xml.ws*;version="${cxf.osgi.jakarta.xml.ws.version}",
               jakarta.xml.bind*;version="${cxf.osgi.jakarta.bind.version}"
           </cxf.osgi.import>
   ```
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org