Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[WIP] [v1.9.x] license updates #20598

Merged
merged 18 commits into from
Sep 29, 2021

Conversation

josephevans
Copy link
Contributor

Description

Updates per feedback from RC7 release.

@mxnet-bot
Copy link

Hey @josephevans , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [edge, miscellaneous, centos-cpu, unix-cpu, website, unix-gpu, windows-cpu, windows-gpu, centos-gpu, sanity, clang]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@mseth10 mseth10 added the pr-work-in-progress PR is still work in progress label Sep 21, 2021
barry-jin added a commit to barry-jin/incubator-mxnet that referenced this pull request Sep 28, 2021
barry-jin added a commit to barry-jin/incubator-mxnet that referenced this pull request Sep 28, 2021
@barry-jin
Copy link
Contributor

Should we also add the license checker workflow into v1.9.x and v1.x branch. I'm trying to create one in master branch.

@josephevans
Copy link
Contributor Author

Should we also add the license checker workflow into v1.9.x and v1.x branch. I'm trying to create one in master branch.

Great idea, I've added the workflow to this PR.

Copy link
Contributor

@barry-jin barry-jin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@waytrue17 waytrue17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@josephevans josephevans merged commit 7b2c0c8 into apache:v1.9.x Sep 29, 2021
@josephevans josephevans deleted the v1.9.x_license_updates_3 branch September 29, 2021 22:42
josephevans added a commit to josephevans/mxnet that referenced this pull request Sep 29, 2021
Fix the first 3 items in apache#20616

* Remove ASF2.0 license and properly list under 3-clause BSD files in LICENSE for np_einsum files.

* Fix license category for deformable_convolution files.

* Update list of Apache-2.0 licensed files.

* Add files to LICENSE that fall under Apache 2.0 license.

* Add include symlinks and other ASF2 files to LICENSE.

* Remove extra line.

* Add numpy einsum files (licensed under bsd 3-clause) to rat-excludes.

* Mention include/nnvm in LICENSE file.

* Remove files from LICENSE with clear ASF-2.0 license headers, added by accident based on skywalking-eyes faulty results.

* Move header guards to below license header section, so skywalking-eyes tool recognizes ASF-2.0 header.

* Fix formatting with license header.

* Fix license section for FindJeMalloc.cmake, move protoc-gen-mypy.py to Apache-2.0 licensed files.

* Clearly indicate caffe subdirectory's licensing model in LICENSE.

* Add skywalking-eyes config to repo.

* Add skywalking-eyes config (.licenserv.yaml) to rat-excludes.

* Add skywalking-eyes license checker into Github workflow.

* Remove caffe subdir from LICENSE and license-check config whitelist, since it is no longer in the repo.

* Remove duplicate entries in LICENSE - leave in Caffe license section.
josephevans added a commit that referenced this pull request Sep 30, 2021
* [v1.9.x] license updates (#20598)

Fix the first 3 items in #20616

* Remove ASF2.0 license and properly list under 3-clause BSD files in LICENSE for np_einsum files.

* Fix license category for deformable_convolution files.

* Update list of Apache-2.0 licensed files.

* Add files to LICENSE that fall under Apache 2.0 license.

* Add include symlinks and other ASF2 files to LICENSE.

* Remove extra line.

* Add numpy einsum files (licensed under bsd 3-clause) to rat-excludes.

* Mention include/nnvm in LICENSE file.

* Remove files from LICENSE with clear ASF-2.0 license headers, added by accident based on skywalking-eyes faulty results.

* Move header guards to below license header section, so skywalking-eyes tool recognizes ASF-2.0 header.

* Fix formatting with license header.

* Fix license section for FindJeMalloc.cmake, move protoc-gen-mypy.py to Apache-2.0 licensed files.

* Clearly indicate caffe subdirectory's licensing model in LICENSE.

* Add skywalking-eyes config to repo.

* Add skywalking-eyes config (.licenserv.yaml) to rat-excludes.

* Add skywalking-eyes license checker into Github workflow.

* Remove caffe subdir from LICENSE and license-check config whitelist, since it is no longer in the repo.

* Remove duplicate entries in LICENSE - leave in Caffe license section.

* Remove extra characters accidentally added to Apache license header.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Licenses pr-work-in-progress PR is still work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants