You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@johnzon.apache.org by GitBox <gi...@apache.org> on 2020/07/16 08:47:15 UTC

[GitHub] [johnzon] amarkevich opened a new pull request #65: Switch to Jakarta JSON Processing API

amarkevich opened a new pull request #65:
URL: https://github.com/apache/johnzon/pull/65


   geronimo-json_1.1_spec dependency cause extra manifest entry:
   
   Require-Capability: osgi.extender;filter:="(osgi.extender=osgi.service
    loader.registrar)",**osgi.contract;osgi.contract=JavaJSONP;filter:="(&(
    osgi.contract=JavaJSONP)(version=1.1.0))"**,osgi.ee;filter:="(&(osgi.ee
    =JavaSE)(version=1.8))"


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

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



[GitHub] [johnzon] amarkevich commented on pull request #65: Switch to Jakarta JSON Processing API

Posted by GitBox <gi...@apache.org>.
amarkevich commented on pull request #65:
URL: https://github.com/apache/johnzon/pull/65#issuecomment-659431421


   Ok, got it


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

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



[GitHub] [johnzon] rmannibucau commented on pull request #65: Switch to Jakarta JSON Processing API

Posted by GitBox <gi...@apache.org>.
rmannibucau commented on pull request #65:
URL: https://github.com/apache/johnzon/pull/65#issuecomment-659261578


   Hi @amarkevich , we deliver jakarta bundles already and must keep delivering javax bundles (using classifier=jakarta for geronimo and johnzon). The contract is also desired - was a fix of a recent version to enable bnd to properly package distros.
   What is the env which does not work for you?
   
   Rergarding the patch itself, distribution and maven plugin do not impact OSGi deployments normally so change should be a noop for that regard. So only root pom diff changes something and it is in scope provided anyway so it is still about the application to bring an OSGi compliant spec jar which provides the expected contracts I think.


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

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



[GitHub] [johnzon] amarkevich commented on pull request #65: Switch to Jakarta JSON Processing API

Posted by GitBox <gi...@apache.org>.
amarkevich commented on pull request #65:
URL: https://github.com/apache/johnzon/pull/65#issuecomment-659300185


   In OSGi env the
   mvn:org.apache.geronimo.specs/geronimo-json_1.1_spec/${geronimo.json.version}
   required to satisfy mentioned contract; unable to use the only
   mvn:jakarta.json/jakarta.json-api/${json.api.version}
   


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

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



[GitHub] [johnzon] rmannibucau commented on pull request #65: Switch to Jakarta JSON Processing API

Posted by GitBox <gi...@apache.org>.
rmannibucau commented on pull request #65:
URL: https://github.com/apache/johnzon/pull/65#issuecomment-659310326


   @amarkevich hmm, geronimo-json is a replacement of jakarta.json-api and you must not have both. Do you use geronimo-json_1.1_spec/1.4 ? If it is about jsonb you can use geronimo-jsonb_1.0_spec/1.3 (with geronimo-json_1.1 since jakarta releases are not OSGi complete). These ones works well in Karaf and equinox standalone (bnd executable goal).
   
   For some background, we contacted eclipse to ask them to make microprofile and jakarta spec jars OSGi compliant (with contracts in particular) but it is unlikely to happen soon so we kept geronimo as main OSGi artifacts.


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

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



[GitHub] [johnzon] amarkevich closed pull request #65: Switch to Jakarta JSON Processing API

Posted by GitBox <gi...@apache.org>.
amarkevich closed pull request #65:
URL: https://github.com/apache/johnzon/pull/65


   


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

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