mattrpav commented on pull request #37: URL: https://github.com/apache/openjpa/pull/37#issuecomment-700732205
@rmannibucau what do you mean by "wrong in terms of OSGi metadata"? There are only two classes in OpenJPA that make use of JMS, and they both use the API signatures that are compatible with both JMS 1.1 and 2.0. If we do not change the OSGi import, OpenJPA users would not be able to use current versions of IBM MQ or Tibco EMS either. I believe the OSGi import in this PR is correct, saying "OpenJPA will support using v1.1 up to (but not including 3.0)". This is the same OSGi import setting that other Apache projects have also adopted. ref: https://github.com/apache/openjpa/search?q=javax.jms ---------------------------------------------------------------- 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: us...@infra.apache.org
