-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[License] move cma_api to 3rdparty. separate BSD 2-clause and 3-clause #4410
Conversation
LICENSE
Outdated
@@ -214,18 +214,22 @@ Apache Software Foundation License 2.0 | |||
3rdparty/dmlc-core | |||
|
|||
|
|||
BSD License | |||
BSD 2-clause License | |||
----------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make sure the underscore covers all the above line
|
||
BSD 3-clause License | ||
----------- | ||
|
||
3rdparty/dmlc-core/include/dmlc/concurrentqueue.h |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add 3rdparty/dmlc-core/include/dmlc/blockingconcurrentqueue.h
c.f. https://github.com/apache/incubator-mxnet/blob/master/LICENSE#L337
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NOTE: we need to add a license file for blockingconcurrentqueue.h, as part of the file is under zlib, and others under BSD-2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should have a section
BSD 2-clause license + zlib license
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pre-approved, can merged as long as the comments are fixed
LICENSE
Outdated
----------- | ||
|
||
3rdparty/picojson | ||
|
||
|
||
BSD 3-clause License |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I double checked, and find that the concurrentqueue.h is under BSD-2 (instead of 3)
@tqchen please double check. |
Thanks @yzhliu I will send a followup PR to refine the blockingconcurrentqueue license |
apache#4410) * [License] move cma_api to 3rdparty. separate BSD 2-clause and 3-clause * add zlib license for blockingconcurrentqueue.h
apache#4410) * [License] move cma_api to 3rdparty. separate BSD 2-clause and 3-clause * add zlib license for blockingconcurrentqueue.h
apache#4410) * [License] move cma_api to 3rdparty. separate BSD 2-clause and 3-clause * add zlib license for blockingconcurrentqueue.h
@tqchen @tmoreau89