On 2/21/10 6:37 PM, Clement Escoffier wrote:
Hi,
I've looked into this release, and I've a couple of comments:
- first the NOTICE file does not follow the format generally used for Felix
projects which means that automatic or semi-automatic checking tools failed.
- then, the embedded EDU.oswego.cs.dl.util.concurrent package should also be
cited in the license. The work 'partially' comes from Sun which means that some
parts was made directly at oswego. It would be nice to cite them under the same
license term.
We are still straightening this out, but we are told that the NOTICE
file should only include references to licenses that require NOTICE. So,
if we are going to follow that rule from now on, then I guess Oswego
doesn't require it, but the Sun portion does.
We had looked into automating this at one time (Felix!), but I think it
wasn't completely successful. At any rate, we should likely start
following this approach for future releases, but we never did decide if
were were going to have a separate acknowledgments file for other
dependencies or just don't mention them at all.
-> richard
- despite it's obvious the 'Licensed under the Apache License 2.0.' is missing
for the software developed at the ASF.
I'm not sure these issues are blocking the release, but it would be useful to
fix them in the next versions.
Regards,
Clement
On 18.02.2010 09:08, Carsten Ziegeler wrote:
Hi,
We solved 6 issues in this release:
https://issues.apache.org/jira/browse/FELIX/fixforversion/12314393
Staging repository:
https://repository.apache.org/content/repositories/orgapachefelix-008/
You can use this UNIX script to download the release and verify the
signatures:
http://svn.apache.org/repos/asf/felix/trunk/check_staged_release.sh
Usage:
sh check_staged_release.sh 008 /tmp/felix-staging
Please vote to approve this release:
[ ] +1 Approve the release
[ ] -1 Veto the release (please provide specific comments)
This vote will be open for 72 hours.
Carsten
