Hi,

Looks much better than the last release candidate, however still +0 (binding) 
due to LICENSE issues, the items brought up for the last RC have not been 
addressed and possible GPL dependancies.

All like like there may be license dependancy issues, but I’m not familiar 
enough with sbt and the project to comment. This "sbt dependencyLicenseInfo | 
grep GNU” shows several GPL dependancies (may be duplicated?). It may be that 
these items are dual licensed with a license that is comparable with the Apache 
license?

I also notice that you have release downloads on your web site. [3] Please make 
it clear that these are not Apache releases. I assume this is a work in 
progress but you might also want to look at the branding requirements at the 
same time and fix those as well. [4]

I checked:
- incubating in release name
- signatures and hashes good
- LICENSE is better, but still missing bundled items brought up last RC
- there a few files with incorrect Apache headers [2] (also brought up last RC)
- can compile from source

LICENSE is still missing, in general anything that is bundled needs to be added 
to LICENSE. [1]
- pygments CSS
- normalize (in bootstrap)
- polypill (in moderniser)

One issue here is that you’re not following the terms of the bundled licenses 
(also brought up last RC) Just mentioning the license in LICENSE is not enough, 
the full text of the license needs to be included somewhere, usually this is in 
the header, but if it’s not there you need to include it. I see you have added 
some files to the licenses directory but having "Copyright (c) <year> 
<copyright holders>” in them doesn’t really tell the whole story does it :-)

Also What happen to the binary files that were in the the last RC? Has the 
creative commons licensing issue been resolved?

Thanks,
Justin

1. http://www.apache.org/dev/licensing-howto.html#guiding-principle
2. 
gearpump-0.8.1-RC4-incubating/services/dashboard/views/cluster/workers/worker/worker.js
3. http://gearpump.apache.org/downloads.html
4. http://incubator.apache.org/guides/branding.html
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to