wink-integration-test/wink-server-integration-test/wink-jaxrs-optional-providers/pom.xml
became
wink-itests/wink-itest/wink-itest-extra-providers/pom.xml
in trunk.

I'm assuming that we'll have to re-spin from trunk to a new 0.1 tag so
I'm making changes there.  I always thought that SVN tags shouldn't be
worked on, but if we're not going to re-spin I will be glad to merge
changes over.

On Fri, Aug 7, 2009 at 3:15 PM, Nicholas Gallardo<[email protected]> wrote:
>
> Kevan, a few responses to some of the issues.
>
>> wink-integration-test/wink-server-integration-test/wink-jaxrs-optional-providers/pom.xml,
>
> Bryant fixed this before it was moved to "wink-itest" to shorten the name.
>
>> I assume that we have run the release binaries through the TCK?
>
> Michael Elman ran the TCK on the tag.  Everything was good.
>
> There are JIRAs opened for the other issues you mentioned.
>
> -N
>
>
>
> ----- Original Message ----
> From: Kevan Miller <[email protected]>
> To: [email protected]
> Sent: Friday, August 7, 2009 2:12:35 PM
> Subject: Re: [VOTE] Release Wink 0.1
>
>
> On Aug 5, 2009, at 4:01 AM, Snitkovsky, Martin wrote:
>
>> Folks,
>>
>> I'm calling a vote to release Wink 0.1
>>
>> The Maven staging area is at:
>> https://repository.apache.org/content/repositories/wink-staging-014/
>>
>> The distributions are in:
>> https://repository.apache.org/content/repositories/wink-staging-014/org/apache/wink/apache-wink/0.1-incubating/
>>
>> This release is tagged at:
>> https://svn.apache.org/repos/asf/incubator/wink/tags/wink-0.1-incubating/
>>
>> The vote will be open here for at least 72 hours.
>>
>> Please consider this call to vote as my +1
>
>
> As noted by Sebb, the top-level svn src directory needs to include the 
> LICENSE and NOTICE files. IMO, it should also include the DISCLAIMER and  
> README/RELEASE-NOTES.
>
> The SVN tag contains src/doc/release_notes.txt. However, the src distribution 
> does not. The src distribution does have license and notice files in 
> top-level, directory, though...
>
> The file, 
> wink-integration-test/wink-server-integration-test/wink-jaxrs-optional-providers/pom.xml,
>  does not have an Apache source license header. It should.
>
> Otherwise, the source looks good and build worked well for me.
>
> I note that release_notes.txt states that "Apache Wink is TCK compliant 
> implementation of the JAX-RS v1.0 specification." I assume that we have run 
> the release binaries through the TCK?
>
> The binary distribution contains two source jars:
>
> lib/jaxb-api-2.1-sources.jar
> lib/jaxb-impl-2.1.4-sources.jar
>
> You cannot distribute these sources.
>
> Finally, have the licenses of the included binaries (lib/*.jar) been properly 
> reviewed and documented, if required? Certainly the CDDL license for the JAXB 
> jars is missing. I can only guess that there are others...
>
> Here's my -1 until the above issues have been resolved.
>
> --kevan
>
>
>
>



-- 

- Bryant Luk

Reply via email to