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

[v1.9.x] Make sure files with 2 licenses are listed properly in LICENSE. #20492

Merged
merged 2 commits into from
Aug 9, 2021
Merged

[v1.9.x] Make sure files with 2 licenses are listed properly in LICENSE. #20492

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
287529f
Make sure files with 2 licenses are listed properly in LICENSE.
josephevans Aug 6, 2021
830b2d0
Remove ASF copyright line.
josephevans Aug 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 4 additions & 4 deletions LICENSE
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,6 @@
3rdparty/onnx-tensorrt/third_party/onnx
3rdparty/intgemm
3rdparty/tvm/3rdparty/compiler-rt/builtin_fp16.h
src/operator/nn/layer_norm.cc

=======================================================================================
3-clause BSD license
Expand All @@ -263,9 +262,6 @@
3rdparty/googletest/googletest
cmake/upstream/FindCUDAToolkit.cmake
cmake/upstream/select_compute_arch.cmake
src/operator/numpy/np_einsum_op-inl.h
src/operator/numpy/np_einsum_path_op-inl.h
src/operator/numpy/np_einsum_op.cc
src/operator/contrib/erfinv-inl.h

=======================================================================================
Expand Down Expand Up @@ -310,12 +306,16 @@
python/mxnet/onnx/mx2onnx/_export_onnx.py
python/mxnet/onnx/mx2onnx/_op_translations/_op_translations_opset12.py
python/mxnet/onnx/mx2onnx/_op_translations/_op_translations_opset13.py
src/operator/numpy/np_einsum_op-inl.h
Copy link
Contributor

Choose a reason for hiding this comment

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

should we list functions of BSD clause.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Based on the history of the file, it seems as if it was heavily modified, so I think we are correct to release it under ASF 2.0 license and don't need to call out specific functions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We also don't call out specific functions in other files in this LICENSE category.

src/operator/numpy/np_einsum_path_op-inl.h
src/operator/numpy/np_einsum_op.cc

=======================================================================================
Apache-2.0 license + MIT License
=======================================================================================

3rdparty/onnx-tensorrt/third_party/onnx/tools/protoc-gen-mypy.py (Copy of the referenced AL2 License available at top of current file)
src/operator/nn/layer_norm.cc (function LayerNormCPUKernel is adapated from MIT-licensed code)

=======================================================================================
Apache-2.0 license + Boost Software License, Version 1.0
Expand Down