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

[v1.8.x] Update LICENSE for all submodules. #19440

Merged
merged 6 commits into from
Oct 30, 2020
Merged

[v1.8.x] Update LICENSE for all submodules. #19440

merged 6 commits into from
Oct 30, 2020

Conversation

josephevans
Copy link
Contributor

@josephevans josephevans commented Oct 28, 2020 •
edited
Loading

Description

This updates the LICENSE and NOTICE file to include the latest data from all submodules.

Part of #19427.

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

@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: [windows-cpu, sanity, centos-cpu, unix-cpu, windows-gpu, miscellaneous, unix-gpu, clang, edge, centos-gpu, website]

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.

@lanking520 lanking520 added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Oct 28, 2020
@josephevans josephevans changed the title Update LICENSE for TVM and 3rdparty modules. [v1.8.x] Update LICENSE for TVM and 3rdparty modules. Oct 28, 2020
@josephevans
Copy link
Contributor Author

Ensured all items were correctly listed in our LICENSE file based on tvm's LICENSE file:

https://github.com/apache/incubator-tvm/blob/9bd2c7b44208ed992061f8c2688e1137357f1db1/LICENSE#L209-L240

@josephevans josephevans changed the title [v1.8.x] Update LICENSE for TVM and 3rdparty modules. [v1.8.x] Update LICENSE for all submodules. Oct 28, 2020
@josephevans josephevans mentioned this pull request Oct 29, 2020
Closed
@lanking520 lanking520 added pr-awaiting-review PR is waiting for code review pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-testing PR is reviewed and waiting CI build and test pr-awaiting-review PR is waiting for code review labels Oct 29, 2020
sandeep-krishnamurthy
sandeep-krishnamurthy approved these changes Oct 29, 2020
View reviewed changes
Copy link
Contributor

@sandeep-krishnamurthy sandeep-krishnamurthy left a comment

Choose a reason for hiding this comment

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

Thank you for this.

@lanking520 lanking520 added pr-work-in-progress PR is still work in progress and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 29, 2020
@sandeep-krishnamurthy
Copy link
Contributor

@mxnet-bot run ci [windows-gpu]

@mxnet-bot
Copy link

Jenkins CI successfully triggered : [windows-gpu]

@lanking520 lanking520 added pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-work-in-progress PR is still work in progress labels Oct 29, 2020
@lanking520 lanking520 added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 29, 2020
leezu
leezu reviewed Oct 29, 2020
View reviewed changes
LICENSE Outdated Show resolved Hide resolved
@lanking520 lanking520 added pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-merge Review and CI is complete. Ready to Merge labels Oct 29, 2020
@lanking520 lanking520 added pr-work-in-progress PR is still work in progress and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 29, 2020
@lanking520 lanking520 added pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-work-in-progress PR is still work in progress labels Oct 29, 2020
@mseth10 mseth10 mentioned this pull request Oct 29, 2020
Closed
@lanking520 lanking520 added pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress labels Oct 29, 2020
@waytrue17 waytrue17 mentioned this pull request Oct 29, 2020
Closed
1 task
@lanking520 lanking520 added pr-awaiting-review PR is waiting for code review and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 30, 2020
@samskalicky samskalicky merged commit 6d5d8b9 into apache:v1.8.x Oct 30, 2020
@josephevans josephevans deleted the license_fixes_v1.8.x branch October 30, 2020 01:36
@josephevans josephevans mentioned this pull request Oct 30, 2020
Closed
1 task
leezu pushed a commit to leezu/mxnet that referenced this pull request Nov 4, 2020
@josephevans @leezu
[v1.8.x] Update LICENSE for all submodules. (apache#19440)
06056da 
* Include content in TVM's NOTICE file in main NOTICE.

* Add licenses for all 3rdparty modules in tvm, and remove old submodule HalideIR.

* Fix path - license is in source code.

* Incorporate other license PRs into this one for simplicity, and move OpenMP into new category.

* Update reference to license.

Co-authored-by: Joe Evans <[email protected]>
leezu pushed a commit to leezu/mxnet that referenced this pull request Nov 4, 2020
@josephevans @leezu
[v1.8.x] Update LICENSE for all submodules. (apache#19440)
f8d970d 
* Include content in TVM's NOTICE file in main NOTICE.

* Add licenses for all 3rdparty modules in tvm, and remove old submodule HalideIR.

* Fix path - license is in source code.

* Incorporate other license PRs into this one for simplicity, and move OpenMP into new category.

* Update reference to license.

Co-authored-by: Joe Evans <[email protected]>
@leezu leezu mentioned this pull request Nov 4, 2020
Merged
leezu pushed a commit that referenced this pull request Nov 5, 2020
@josephevans @leezu
[v1.8.x] Update LICENSE for all submodules. (#19440)
f78042f 
* Include content in TVM's NOTICE file in main NOTICE.

* Add licenses for all 3rdparty modules in tvm, and remove old submodule HalideIR.

* Fix path - license is in source code.

* Incorporate other license PRs into this one for simplicity, and move OpenMP into new category.

* Update reference to license.

Co-authored-by: Joe Evans <[email protected]>
leezu added a commit that referenced this pull request Nov 5, 2020
@leezu
Update LICENSE (#19478)
c0c4cb5 
This ports #19443 and #19440 from the v1.8.x branch.

Further, the LICENSE file is cleaned up to avoid listing unnecessary information that can easily get out of sync with the actually included files (copyright headers).

See https://infra.apache.org/licensing-howto.html for more information
vidyaravipati pushed a commit to vidyaravipati/incubator-mxnet that referenced this pull request Nov 11, 2020
@leezu
Update LICENSE (apache#19478)
b420d5e 
This ports apache#19443 and apache#19440 from the v1.8.x branch.

Further, the LICENSE file is cleaned up to avoid listing unnecessary information that can easily get out of sync with the actually included files (copyright headers).

See https://infra.apache.org/licensing-howto.html for more information
josephevans added a commit to josephevans/mxnet that referenced this pull request Dec 8, 2020
@josephevans
[v1.8.x] Update LICENSE for all submodules. (apache#19440)
0d43948 
* Include content in TVM's NOTICE file in main NOTICE.

* Add licenses for all 3rdparty modules in tvm, and remove old submodule HalideIR.

* Fix path - license is in source code.

* Incorporate other license PRs into this one for simplicity, and move OpenMP into new category.

* Update reference to license.

Co-authored-by: Joe Evans <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@leezu leezu leezu left review comments

@samskalicky samskalicky samskalicky left review comments

@sandeep-krishnamurthy sandeep-krishnamurthy sandeep-krishnamurthy approved these changes

@mseth10 mseth10 mseth10 approved these changes

Assignees
No one assigned
Labels
pr-awaiting-review PR is waiting for code review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@josephevans @mxnet-bot @sandeep-krishnamurthy @leezu @samskalicky @mseth10 @lanking520