dev@glassfish.java.net

Re: POM, OSGi, and Packaging Changes

From: Jane Young <jane.young_at_oracle.com>
Date: Tue, 04 Oct 2011 05:58:34 -0700

Thanks for the explanation.
The pom changes look fine.

On 10/4/11 5:48 AM, Jason Lee wrote:
> All the tests we have require a running server, so we have, for a long
> time, skipped the tests by default, only enabling them via a profile,
> which, in theory, we use under Hudson. This same configuration was
> under rest-service AND rest-client, but I moved it up to rest-parent,
> so as to reduce the duplication. It is possible, I guess, that
> rest-client may have some unit tests that can run without a server.
> I'm sad to report, though, that rest-client has no tests at all at
> this point, but the plan is to fix that lack at some point in the near
> future, and then run said tests along with the rest-service tests,
> which brings us back to the running server. :)
>
> On 10/4/11 7:26 AM, Jane Young wrote:
>> Question: why are you skipping unit-tests in nucleus/admin/rest?
>>
>>
>> On 10/3/11 3:07 PM, Jason Lee wrote:
>>> Great! I'll wait for Jane, etc to comment on the POM changes.
>>>
>>> Thanks.
>>>
>>> On 10/3/11 3:30 PM, Snjezana Sevo-Zenzerovic wrote:
>>>> Looks fine to me...
>>>>
>>>>
>>>> ----- Original Message -----
>>>> From: jason.d.lee_at_oracle.com
>>>> To: dev_at_glassfish.java.net
>>>> Sent: Monday, October 3, 2011 1:15:45 PM GMT -08:00 US/Canada Pacific
>>>> Subject: POM, OSGi, and Packaging Changes
>>>>
>>>> Attached is a diff of some proposed POM and osgi.bundle changes
>>>> that I'd
>>>> like to make to support REST client generation code. In a nutshell,
>>>> I've
>>>> moved some classes from rest-service to rest-client, which will make
>>>> exposing them to users a little easier/lighter. I then added a
>>>> dependency on rest-client to rest-service to reintroduce the moved
>>>> classes, and changed the packager to include rest-client. The
>>>> rest-client osgi.bundle was changes to export the correct package.
>>>>
>>>> Please let me know if I need to make any more changes, or if this
>>>> is OK
>>>> to check in.
>>>>
>>>> Thanks.
>>>>
>>>
>>>
>>
>
>