...
- PR to fix the top level LICENSE file - This one has been merged but a review from mentors/PMC would still be helpful
- PR to update the license_header.py file and add apache licenses where missing - Merged.
PR to add license for dockerfiles - Merged
PR to fix License headers in specific files - Part 1 - Merged
PR for some fixes based on Apache RAT failures - Part 2 - Merged
- PR for some fixes based on Apache RAT failures - Part 3 - Merged
- PR for some fixes based on Apache RAT failures - Part 4 - Merged
- PR for adding headers to the perl-package
- PR for adding headers to the docs folder
D. Open Issues/Questions/Doubts/Concerns (Help Requested!)
...
Name of File/Folder Excluded from RAT Check | Reason for Ignoring | Concerns | |
---|---|---|---|
1 | These File Types: *.xml ; *.css ; *.txt; *.md ; \..* ; *.ipynb ; *.html ; *.js ; *.json ; *.svg; *.config; *.names; *.csv | ||
2 | Submodules - 3rdparty/* ; dmlc-core/* ; mshadow/* ; dmlc-core/* ; dlpack/* ; nnvm/* ; ps-lite/* | None | |
3 | R-package | Not a part of MXNet Release | None |
4 | Ignore all Dockerfiles - docker/*; Dockerfile* ; docker_multiarch/* | Dockerfiles cant have license | |
5 | perl-package/* | I am not entirely sure about licensing here. (Also see issue 12 in the last table below) | I have created a PR - https://github.com/apache/incubator-mxnet/pull/9554 But there are still some files here which dont have the license (*.i files and some more) |
6 | contrib/* | I am not entirely sure about licensing here. | Is ASF header ok? |
7 | __init__.py files | These files contain no text | None |
8 | docs/_static/* | A header might affect website. Added the | ASF header to the scripts here - Only one script here without a license - docs/_static/mxnet-theme/theme.conf |
9 | This file - src/operator/nn/pool.h | It was decided that this file should not have an Apache License and it was removed here - PR 9170 | None. But can be verified again. |
10 | This file - src/operator/special_functions-inl.h | It was decided that this file should not have an Apache License and it was removed here - PR 9170 | None. But can be verified again. |
11 | example/rcnn/rcnn/cython/* | This is licensed under MIT but RAT doesnt pick that. ASF should not be added | Should MIT license text be added explicitly? |
12 | This Dataset - example/gluon/tree_lstm/dataset.cPickle | This is a dataset | None. But can be verified again. |
13 | This file - tools/coreml/pip_package/README.rst | This is a README | None. But can be verified again. |
...