+1 binding

Verified
- Release files in correct location
- Release files have the word incubating in their name
- Digital signature and hashes correct
- DISCLAIMER file exist
- LICENSE and NOTICE files exists
- LICENSE and NOTICE text correct
- Un-included software dependencies are not mentioned in LICENSE or NOTICE
- License information is not mentioned in NOTICE
- All source files have ASF headers
- Contents of the release match with what's tagged in version control
- No unexpected binary files in the release

Non verified
- NOTICE year not correct, unfortunate it was updated when the release was
cut again
Is there any 3rd party code contained inside the release? If so:
- Was not able to compile.  Stopped after 10 min, is
https://mxnet.incubator.apache.org/versions/master/install/build_from_source.html
right place to look?  Instructions could be clearer

Good work fixing the licensing.

Mike

On Tue, Feb 26, 2019 at 9:34 PM Justin Mclean <jus...@classsoftware.com>
wrote:

> HI,.
>
> +1 (binding)
>
> I didn’t have time to check all of the licenses so there may be some work
> that’s still needed there, but it looks fairly comprehensive.
>
> I checked:
> - incubating in name
> - DISCLAIMER exists
> - LICENSE looks OK
> - NOTICE need year updating
> - No unexpected binary files
> - All ASF files have ASF headers
> - I didn’t compile from source and assume others have
>
> Thanks,
> Justin
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Reply via email to