incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: Help reviewing PhotArk podling release
Date Tue, 01 Sep 2009 22:57:13 GMT
On 31/08/2009, Luciano Resende <luckbr1975@gmail.com> wrote:
> Thanks for all the reviews and feedback for the PhotArk RC2. I have
>  fixed all the severe and blocking issues and have produced a new RC...
>
>  Please follow the link below for the new RC details :
>
>  http://www.mail-archive.com/photark-dev@incubator.apache.org/msg00145.html

For the benefit of other reviewers, here are the details:

<quote>
The artifacts are available for review at:
http://people.apache.org/~lresende/photark/M1-incubating-RC3/

This includes the signed binary and source distributions, the RAT report,
and the Maven staging repository.

The release tag is available at :
https://svn.apache.org/repos/asf/incubator/photark/tags/M1-incubating-RC3/
</quote>

Sigs and hashes OK. However no SHA hashes - only MD5 - are provided.
tgz and zip archives agree with each other. Source archives agree with SVN tag.

SVN has one incorrect setting:
svn pd svn:executable photark-webapp/src/main/webapp/prev_on.gif
[not a blocker]

The binary NOTICE file has a lot of paragraphs which start:

"This product includes/uses ..."

The NOTICE file should only include required 3rd party notices; code
that is used (but not included) should not be mentioned in the NOTICE
file, so the /uses part and any non-included 3rd party dependencies
need to be removed.

There are no NOTICE and LICENSE files in the META-INF directory in the
war file; ideally these should be present - in this case they will be
the same as the ones in the binary archive.

There's some inconsistency between the NOTICE files.

The top-level NOTICE file in the binary archive says:

Copyright (c) 2008 - 2009

whereas the NOTICE file in the photoark-assets-M1-incubating.jar
META-INF directory says:

Copyright (c) 2005 - 2008 The Apache Software Foundation

The parent pom says the inception year is 2008, so the latter file looks wrong.
Seems to be an error in the NOTICE skeleton files.

>  Thanks
>
>
>  --
>
> Luciano Resende
>  http://people.apache.org/~lresende
>  http://lresende.blogspot.com/
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>  For additional commands, e-mail: general-help@incubator.apache.org
>
>

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


Mime
View raw message