Hi,

+1 binding

I checked:
- release files include incubating in the name
- signatures and hashes good
- DISCLAIMER exists
- LICENSE good (although no need to list Apache licensed software)
- NOTICE has an issue (see below)
- All source files have Apache headers
- No unexpected binaries in source release
- Don’t compile as it requires later version of java than I have installed.

Bundled with the software (according to the license file)  is Activiti which is 
Apache licensed and has a notice file [1] (although most of this content 
shouldn't actually be in license). As per [2] this needs to be looked at and 
parts added to the Tinkerpop NOTICE file.  However I can’t even find the two 
files mentioned in LICENSE so it may be that nothing needs to be done and those 
lines removed from LICENSE?

As noted in the previous incubator release [3] can the binary NOTICE/LICENSE 
files please be fixed before the next release. Your mentors should be able to 
help with this.

Thanks,
Justin

1. https://github.com/Activiti/Activiti/blob/master/distro/src/notice.txt
2. http://www.apache.org/dev/licensing-howto.html#alv2-dep
3. 
https://mail-archives.apache.org/mod_mbox/incubator-general/201507.mbox/%3c2b93b23f-63f9-4bf0-9079-aa63ff688...@classsoftware.com%3E


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

Reply via email to