Martin Grotzke wrote:
> Hi Paul,
>
> On Mon, 2008-04-21 at 08:59 +0200, Paul Sandoz wrote:
>> Hi Martin,
>>
>> Great!
>>
>> How do you think we should integrate this into the into trunk (the
>> code merge is the easiest bit :-) ) for building, testing and
>> constructing the distribution package?
>>
>> Currently we use "ant dist", i am guessing this task would need to
>> defer to maven for the spring building/packaging.
> I'd say that the jersey-spring package will be built separately from the
> jersey core with a dependency on this, therefore the jersey-package
> should not need to be changed.
>
> I will update the ant build.xml in the jersey-spring module so that
> there's an `ant dist` and `ant -Dprofile=spring20 test`...
>
I think just using Maven is OK. The most important thing is to make sure
we can add this as a Hudson task so it will get built/tested/packaged
every time we make changes to jersey or to jersey spring.
> What has to be clarified is the project structure, currently the trunk
> contains these directories:
>
> commons
> \- spring
> jersey
> \- <everything related directly to jersey like src, build, dist etc.>
> repo
> www
>
> To the question is if we replace commons/spring with the new spring
> package or if we create a jersey-spring package at the root level.
> IMHO a container for contrib modules is useful, so we could use the
> commons folder or rename it to contrib or s.th. else.
>
Renaming "common" to "contribs" is fine my be and we can delete
"commons/spring".
We can mandate that everything in "contribs" should use maven.
> Before doing the merge we should also make sure that classes are at the
> correct place (like e.g. ResourceContext, the package of the
> SpringServlet, the groupId and artifactId of the jersey-spring module
> etc.). Can you check these things, Paul?
>
OK. It would be good if other "mavenintes" could have a quick look at
the maven stuff.
The location of ResourceContext is fine.
If we are going to keep a similar naming scheme then i recommend using
the following for the Servlet:
com/sun/ws/rest/spi/spring/container/servlet/
and the following for the annotations:
com/sun/ws/rest/api/spring/
Paul.
--
| ? + ? = To question
----------------\
Paul Sandoz
x38109
+33-4-76188109