Hi,

+1 binding

But would be good clarify licenses of patches and fix up the slightly confusing 
section in LICENSE.

I checked:
- incubating in name
- signature and hashes good
- DISCLAIMER exists
- NOTICE is good
- LICENSE I think is missing a boost [1] and again I think the end section is 
confusing. Boost is MIT licensed and is in the source not just the binary [1]
- all source files have Apache headers
- no unexpected binary files
- can compile from source

There are a number of patches in [2], it’s not clear how they are licensed.

Thanks,
Justin

1. apache-kudu-incubating-0.9.1/thirdparty/boost_uuid/boost/*
2. thirdparty/patches/*
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to