hstaudacher / osgi-jax-rs-connector

An OSGi - JAX-RS 2.0 Connector, software repository available on the link below
http://hstaudacher.github.io/osgi-jax-rs-connector
Other
191 stars 98 forks source link

Test 4.3.0 RC1 #89

Closed hstaudacher closed 9 years ago

hstaudacher commented 9 years ago

The 4.3.0 RC1 is ready to be tested including the new ApplicationConfiguration API (see #88).

Maven: https://oss.sonatype.org/content/repositories/snapshots/ ziped p2 repo: https://dl.dropboxusercontent.com/u/5808972/com.eclipsesource.jaxrs.repository-4.3.0-RC1.zip

@kaikreuzer can you test it with openHAB? @stefanseifert can you test the maven stuff? :) @BryanHunt bnd repo also included in the zip archive ;) @larsp can you test the new version with Swookie? @isole Can you test the ApplicationConfiguration stuff?

Will publish the release next monday if no-one vetoes or requests more time. Thanks guys!

isole commented 9 years ago

ApplicationConfiguration is looking good

kaikreuzer commented 9 years ago

Seems to work well with openHAB. My only concern is that the jersey-all bundle has again grown in size from 5448789 to 6157007 byes. For embedded use this is an incredibly fat bundle and I would very much wish to shrink it as much as possible.

hstaudacher commented 9 years ago

Thanks for testing. @kaikreuzer one idea:

What about creating an additional jersey-min bundle and deliver it together with the all bundle. So it will be up to the user which one he want's to use. But this would require your help. The all bundle has a separate maven build. It takes all dependencies into the bundle during the build. See https://github.com/hstaudacher/osgi-jax-rs-connector/blob/master/build/com.eclipsesource.jaxrs.jersey.all/pom.xml#L117

So, can you just duplicate this build and reduce the dependencies to the level you need and open a pull request? I would integrate it and include it in the release.

kaikreuzer commented 9 years ago

A jersey-min bundle sounds like a good idea! @kolkoo already stripped out all client stuff from Jersey and he will create a PR for you. Size is then around 4MB.

kaikreuzer commented 9 years ago

There we go: https://github.com/hstaudacher/osgi-jax-rs-connector/pull/91

hstaudacher commented 9 years ago

Thanks. Will integrate over the weekend

BryanHunt commented 9 years ago

I did a very basic test with Bndtools, and it is working.

hstaudacher commented 9 years ago

Thanks @BryanHunt

@kaikreuzer I have integrated the jersey-min bundle in the build. Here is the RC2 repository with the min bundle included. Can you give it a try? https://dl.dropboxusercontent.com/u/5808972/com.eclipsesource.jaxrs.repository-4.3.0-RC2.zip

larsp commented 9 years ago

Hi, if you change the dependencies also to the SNAPSHOTS, such as:

...
<groupId>com.eclipsesource.jaxrs</groupId>
<artifactId>jersey-all</artifactId>
<version>2.17</version>
</dependency>
...

to 2.17-SNAPSHOT I can continue testing. And it's "swookiee" :grin:

hstaudacher commented 9 years ago

@larsp I don't get it. Why is this change required?

larsp commented 9 years ago

Because there is no 2.17 but only a 2.17-SNAPSHOT in the repository: https://oss.sonatype.org/content/repositories/staging/com/eclipsesource/jaxrs/jersey-all/ and you are referencing a release version which is not there yet, I assume.

hstaudacher commented 9 years ago

Oh, I see ;) It's currently uploading and should be available in a few minutes.

larsp commented 9 years ago

Strange, the https://oss.sonatype.org/content/repositories/staging/com/eclipsesource/jaxrs/publisher/4.3-SNAPSHOT/publisher-4.3-20150322.133633-6.jar only contains html?

larsp commented 9 years ago

If you like you can also provide a regular staging release build instead of a snapshot. Just without releasing. In case it is easier?

hstaudacher commented 9 years ago

@larsp just uploaded a new version

larsp commented 9 years ago

@hstaudacher same, no bytecode in publisher-4.3-20150322.145840-9.jar instead the html I would expect in a -javadoc.jar just with a Manifest.

hstaudacher commented 9 years ago

@larsp damn. I messed up. Found the problem and it's fixed now. A valid version is online. Sorry for this

larsp commented 9 years ago

Works, great! :shipit:

hstaudacher commented 9 years ago

Alright. Testing done. Will release 4.3 shortly

hstaudacher commented 9 years ago

Thanks you all for your effort. It's very appreciated...

hstaudacher commented 9 years ago

4.3 is published -> https://github.com/hstaudacher/osgi-jax-rs-connector/releases/tag/4.3.0

Thank you all for the big help!