Hi Justin, It’s added in this PR[1] which is trying to improve our LICENSE file. A bunch of submodules were also added and we are aware we should only include those bundled in source release. I have verified all license in the top level LICENSE file are included in source release except this MKL BLAS license. It was added by mistake since there was some confusion on MKL-DNN (Apache 2.0), MKLML(Intel simplified license) and MKL (intel simplified license).
There are two related options in MXNet build flags, USE_BLAS and USE_MKLDNN, USE_BLAS has option to use full MKL BLAS, users has to install separately and it's not included in MXNet release. USE_MKLDNN=1 will use MKLDNN(included in MXNet source release). MKLDNN will donwload MKLML with intel simplified license during build time only. As per discussion here, the Intel simplified license should be removed from LICENSE file.[2] [1] https://github.com/apache/incubator-mxnet/pull/13808/files#diff-9879d6db96fd29134fc802214163b95aR535 [2]https://github.com/apache/incubator-mxnet/blob/master/make/config.mk#L111 On Tue, Jul 16, 2019 at 2:45 AM Justin Mclean <jus...@classsoftware.com> wrote: > Hi, > > > For [1], I think we need to remove it. For [2] I have raised on legal > via JIRA[3]. > > Thanks. It would be nice to know why this happened, and depending on that > answer there may be more work to do to get the LICENSE into shape. I’m > guessing that teh project may of added all dependancies to the LICENSE file > when you only need to add what is bundled in the source release. > > > Can we resolve these issues during the next release? > > Sure but’s that’s not up to me. > > Thanks, > Justin > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
