dev@glassfish.java.net

Re: Jerome, Sahoo: please review Jersey 1.1.0-ea integration patch

From: Snjezana Sevo-Zenzerovic <Snjezana.Sevo-Zenzerovic_at_Sun.COM>
Date: Mon, 04 May 2009 13:20:58 -0700

Jakub,

one note/question - I see that you changed IPS package dependencies to
include both glassfish-nucleus and metro packages. Do you still require
only jaxb from metro? If that's the case, you should define dependency
on glassfish-common package since jaxb is there now, and you don't need
separate dependency on glassfish-nucleus since glassfish-common depends
on it.

Thanks,

Snjezana

Jakub Podlesak wrote:

>Hi Jerome, Sahoo,
>
>Could you please review the attached patch,
>which should integrate Jersey version 1.1.0-ea
>into the 3.0-Preview branch and the main trunk.
>
>Thanks,
>
>~Jakub
>
>
>------------------------------------------------------------------------
>
>Index: pom.xml
>===================================================================
>--- pom.xml (revision 26906)
>+++ pom.xml (working copy)
>@@ -155,7 +155,7 @@
> <antlr.version>2.7.6</antlr.version>
> <mq.version>4.4-005</mq.version>
> <webbeans.version>1.0.0.PREVIEW1</webbeans.version>
>- <jersey.version>1.0.1</jersey.version>
>+ <jersey.version>1.1.0-ea</jersey.version>
> <jbi.version>1.0</jbi.version>
> <wsdl4j.version>1.6.2</wsdl4j.version>
> <gfprobe-provider-client.version>3.0.0-b001</gfprobe-provider-client.version>
>Index: packager/jersey/src/main/resources/pkg_proto.py
>===================================================================
>--- packager/jersey/src/main/resources/pkg_proto.py (revision 26906)
>+++ packager/jersey/src/main/resources/pkg_proto.py (working copy)
>@@ -35,21 +35,30 @@
> # only if the new code is made subject to such option by the copyright
> # holder.
> #
>+
> import imp
>
> conf = imp.load_source("pkg_conf", "../pkg_conf.py")
>
> pkg = {
> "name" : "jersey",
>- "version" : "1.0.1,0-0.1",
>- "attributes" : {
>- "pkg.summary" : "Jersey RESTful Web services for GlassFish",
>- "pkg.description" : "Jersey is the open source (under dual CDDL+GPL license) JAX-RS (JSR 311) Reference Implementation for building RESTful Web services. But, it is also more than the Reference Implementation. Jersey provides additional APIs and extension points (SPIs) so that developers may extend Jersey to suite their needs.",
>- "info.classification" : "Web Services",
>- },
>- "dirtrees" : [ "jersey", "glassfish" ],
>- "depends" : { "pkg:/metro_at_1.4" : {"type" : "require" }},
>- "licenses" : {
>- "../../../../CDDL+GPL.txt" : {"license" : "CDDL and GPL v2 with classpath exception"},
>- },
>+ "version" : "1.1.0,0-1.0",
>+ "attributes" : { "pkg.summary" : "Jersey Core, RESTful Web services for GlassFish",
>+ "pkg.description" :
>+"Jersey core runtime libraries including some 3rd party dependencies. \
>+ Documentation and examples of Jersey were moved to a separate package, Jersey Examples And Documentation. \
>+ If you are about to upgrade Jersey, you might want to install that new package as well, otherwise Jersey examples and javadocs will get deleted. \
>+ Also please note, that from 1.0.3 version on, Jersey docs and examples get installed into <as_home>/glassfish/jersey directory instead of just <as_home>/jersey. \
>+ Jersey is the open source (under dual CDDL+GPL license)\
>+ JAX-RS (JSR 311) Reference Implementation for building RESTful Web services. \
>+ But, it is also more than the Reference Implementation. \
>+ Jersey provides additional APIs and extension points (SPIs) \
>+ so that developers may extend Jersey to suite their needs.",
>+ "info.classification" : "Web Services" },
>+ "dirtrees" : [ "glassfish"],
>+ "depends" : {
>+ "pkg:/glassfish-nucleus_at_3.0" : {"type" : "require" }
>+ ,"pkg:/metro_at_1.4" : {"type" : "require" }
>+ },
>+ "licenses" : { "../../../../CDDL+GPL.txt" : {"license" : "CDDL and GPL v2 with classpath exception"}}
> }
>Index: packager/jersey/pom.xml
>===================================================================
>--- packager/jersey/pom.xml (revision 26906)
>+++ packager/jersey/pom.xml (working copy)
>@@ -88,15 +88,12 @@
>
>
> <dependencies>
>- <!--
>- modules to be included in this bundle - temporary solution
>-until we sort out 1.0.2 non-snapshot dependencies
>- -->
> <dependency>
>- <groupId>org.glassfish.packager.temp</groupId>
>- <artifactId>jersey</artifactId>
>+ <groupId>com.sun.jersey.glassfish.v3</groupId>
>+ <artifactId>jersey-gfv3-core</artifactId>
> <version>${jersey.version}</version>
> <type>zip</type>
>+ <classifier>project</classifier>
> </dependency>
> </dependencies>
>
>@@ -134,9 +131,17 @@
> </profile>
> </profiles>
>
>+ <repositories>
>+ <repository>
>+ <id>maven2-repository.dev.java.net</id>
>+ <name>Java.net Repository for Maven</name>
>+ <url>http://download.java.net/maven/2/</url>
>+ </repository>
>+ </repositories>
>+
> <pluginRepositories>
> <pluginRepository>
>- <id>maven2-repository.dev.java.net</id>
>+ <id>maven2-plugin-repository.dev.java.net</id>
> <name>Java.net Repository for Maven</name>
> <url>http://download.java.net/maven/2/</url>
> </pluginRepository>
>Index: packager/jersey/build.xml
>===================================================================
>--- packager/jersey/build.xml (revision 26906)
>+++ packager/jersey/build.xml (working copy)
>@@ -25,7 +25,7 @@
> <target name="create.package">
> <echo message="Staging files"/>
>
>- <unzip src="target/tmp/jersey.zip"
>+ <unzip src="target/tmp/jersey-gfv3-core-project.zip"
> dest="target/stage/${install.dir.name}">
> </unzip>
>
>@@ -34,7 +34,7 @@
> <antcall target="pack.compression" />
> </if>
>
>- <zip file="target/jersey.zip" basedir="target/stage">
>+ <zip file="target/jersey.zip" basedir="target/stage" excludes="**/jersey-core_proto.py, **/LICENSE.txt">
> </zip>
>
> <attachArtifact file="target/jersey.zip"/>
>
>
>
>------------------------------------------------------------------------
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>
>