Hi Sebb, 

1. As you noticed, sources/javadoc jars are missing default implementation and 
specification entries. 
   As good Apache citizens, we use apache-6.pom for all common plugin 
configuration.
   For maven-source-plugin, apache apache-6.pom uses old version 2.0.4 that has 
this issue http://jira.codehaus.org/browse/MSOURCES-42 
   For maven-javadoc-plugin, apache apache-6.pom does not define default 
implementation and specification entries. 
So I went ahead, and made required changes in our main pom. 

*Question* - how do we fix these issues in next version of apache-[X].pom ? 

2. You suggested to add a compiler source and target settings to manifest 
(these are proprietary headers, right?). Correct me if I am wrong, but we 
already have "Build-Jdk" header that provides same information.

3. You mentioned that NOTICE and LICENSE files are missing in individual jars - 
can you be more specific?  


Thanks for your comments.

Martin


-----Original Message-----
From: sebb [mailto:seb...@gmail.com] 
Sent: Thursday, August 06, 2009 6:32 PM
To: general@incubator.apache.org
Cc: wink-...@incubator.apache.org
Subject: Re: [VOTE] Releasing Wink 0.1

On 06/08/2009, sebb <seb...@gmail.com> wrote:
> On 06/08/2009, Snitkovsky, Martin <martin.snitkov...@hp.com> wrote:
>  > The Wink community has started the voting to release Wink 0.1.
>  >  The voting mail thread can be viewed at 
> http://mail-archives.apache.org/mod_mbox/incubator-wink-dev/200908.mbox/<2c60aa1b234ac84d8486dc6ba56275fc508a3ca...@gvw1163exb.americas.hpqcorp.net>
>  >
>  >  We are sending this e-mail as a heads up so we can get additional 
> feedback on the artifacts from the community.
>  >
>  >  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/
>
>
> No point in having .asc.md5 and .asc.sha1 files; these should be
>  deleted before release.
>
>  The signing key does not seem to have been uploaded to any pgp servers.
>  [Key id 0x5E67E108]
>
>  Also, could not find the KEYS file.
>
>  It's not possible to check the sigs.
>
>  The hashes are OK.
>

Some of the jars have manifests which include the following:

Specification-Title: Wink WebDav
Specification-Version: 0.1-incubating
Specification-Vendor: The Apache Software Foundation
Implementation-Title: Wink WebDav
Implementation-Version: 0.1-incubating
Implementation-Vendor-Id: org.apache.wink
Implementation-Vendor: The Apache Software Foundation

This is good as it helps to identify the jar in isolation.
However, not all of the jars have this information.
It would also be good to include the compiler source and target settings.

I'd like to see the NOTICE and LICENSE files also in the individual
jars (in the META-INF directory). At present they only seem to be in
the source jars, which don't have the full manifest info as above.

I got one test failure:

Failed tests:
  testLockingExample(org.apache.wink.example.locking.LockingTest)

Tests run: 1, Failures: 1, Errors: 0, Skipped: 0

I'll raise a JIRA and attach the Surefire output files.

>  >  This release is tagged at:
>  >  https://svn.apache.org/repos/asf/incubator/wink/tags/wink-0.1-incubating/
>
>
> The LICENSE and NOTICE files need to be in the top-level directory.
>
>  I had problems checking out the SVN tree because some of the path/file
>  names are extremely long (up to 198 characters), for example:
>
>  
> wink-integration-test/wink-server-integration-test/wink-jaxrs-test-providers/src/test/java/org/apache/wink/jaxrs/test/providers/exceptionmappers/nullconditions/JAXRSExceptionsNullConditionsTest.java
>
>  In the end I had to check out the tree into a top-level directory with
>  a short name.
>  This is on Win/XP.
>
>  I also had problems with the archives for the same reason.
>
>  I think this is going to cause problems for other users too.
>
>  There are quite a lot of files in SVN that are not in the source
>  archive, for example:
>
>  release_notes.txt
>  Apache_Wink_User_Guide.docx/pdf
>
>  src/assembly/dist.xml
>  etc.
>
>  Missing AL header:
>  
> wink-integration-test/wink-server-integration-test/wink-jaxrs-optional-providers/pom.xml
>
>  As a separate issue, there are a lot of missing SVN properties (mainly
>  eol-type:native).
>  I'll file a bug report for that.
>
>
>  >  Thanks,
>  >  Martin
>  >
>  >  ---------------------------------------------------------------------
>  >  To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
>  >  For additional commands, e-mail: general-h...@incubator.apache.org
>  >
>  >
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to