kou commented on code in PR #38660: URL: https://github.com/apache/arrow/pull/38660#discussion_r1525651207
########## LICENSE.txt: ########## @@ -2252,3 +2252,13 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. + +-------------------------------------------------------------------------------- + +3rdparty dependency mathworks/libmexclass is redistributed as a dynamically +linked shared library in certain binary distributions, like the MATLAB +distribution. + +Copyright: 2022-2024 The MathWorks, Inc. All rights reserved. +Homepage: https://github.com/mathworks/libmexclass +License: 3-clause BSD Review Comment: Ah, we should recheck existing entries such as the zlib's one in `LICENSE.txt`... > Also, we noted the following instructions in https://infra.apache.org/licensing-howto.html#permissive-deps: It seems that this description is for "bundled" dependency not "non-bundled" dependency. https://infra.apache.org/licensing-howto.html#step-by-step > For any [bundled](https://infra.apache.org/licensing-howto.html#bundled-vs-non-bundled) dependency, consider whether LICENSE and/or NOTICE need to be modified. Do not modify LICENSE or NOTICE for non-bundled dependencies. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
