Thanks Justin! I have reviewed this PR and made a few more changes. The final changes to the LICENSE file can be summarized as follows - 1. I have reverted commit 8930d96 (PR #9484) 2. Revisited some comments from previous release which are now relevant and made appropriate changes - based on points 8-11 and 13 to 19 in this wiki, section E 3. Ran a fresh search for dependencies with a separate license and added the missing ones to the LICENSE file. 4. Reviewed the Apache Policy and confirmed to the best of my understanding the LICENSE file complies with the guidelines. Please let me know if anyone sees any other major issues in this file. Thanks, Meghna Baijal On Mon, Feb 5, 2018 at 9:19 PM, Justin Mclean wrote: > Hi, > > > I have created a PR to revert the LICENSE file to the previous version. I > > have also updated this file with a few packages that were missing here. > > Link to PR - https://github.com/apache/incubator-mxnet/pull/9701 > > > > It would be great if you could review this PR to suggest any other > > necessary changes. > > Please follow the process at [1], what important is that the PPMC knows > how to do this on their own. > > I would guess that a little more needs to be done than just reverting the > changes as wouldn’t that mean the feedback from the previous release would > of of been reverted as well? > > Double check that: > - all 3rd party bundled software is mentioned in license (it's optional if > you want to put ALv2 in there) > - that only bundled things are mentioned and that dependancies are not > - the full text of 3rd party licenses is included or pointed to from the > LICENSE file > > If you miss a couple that’s OK, as every thing has a permissive license, > we can fix that in a later release. > > Thanks, > Justin > > 1. http://www.apache.org/dev/licensing-howto.html > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org > For additional commands, e-mail: general-help@incubator.apache.org > >