incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: [VOTE] Release Hama version 0.2-incubating RC2
Date Thu, 07 Apr 2011 15:01:02 GMT
On 7 April 2011 13:49, Edward J. Yoon <edwardyoon@apache.org> wrote:
> Please review this again if you have time -
> http://svn.apache.org/repos/asf/incubator/hama/branches/0.2/
>
>  - added "DISCLAIMER.txt" file

+1

The Hama website also needs to include the disclaimer (and apache
should be spelt Apache), and should include the Incubator logo with
link back to the incubator site.

>  - added license header to "conf/log4j.properties".
> ("src/test/log4j.properties" is deleted.)
>  - updated "NOTICE" and "LICENSE" files.

NOTICE file is for *required* notices only.

In particular, other ASF projects should not be listed, though I
suppose if their NOTICE files have required attributions they should
be copied here?

In the case of UmlGraph, it appears that the required notice is as follows:

This product includes UmlGraph http://www.umlgraph.org/
Copyright (c) 2002-2007, Diomidis Spinellis

It's not clear whether the other two 3rd party licenses require
attribution or not, but in any case the license should only be
mentioned in LICENSE.txt

The LICENSE.txt file looks OK, except that the GroboUtils section
refers to "The above copyright notice" - which does not exist.
Is this a copy/paste error? or is it an error in the original
document? It would be useful to include the license URL in LICENSE.txt
(same for the other two)

>  - deleted unused dependencies. ("findbugs" and "xmlenc-0.52.jar")

+1

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


Mime
View raw message