You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by GitBox <gi...@apache.org> on 2022/09/24 16:02:20 UTC

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

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

   @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
    - for META-INF manifests, do you think we could keep the `</cxf.osgi.export>`, `<cxf.osgi.import>` and others? I would certainly not make deployable (no Jakarta support for OSGi) but would save us some time when we would need to bring this back
   
   WDYT? In any case, LGTM!


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