Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15738 )

Change subject: mini-cluster: exclude libpcre for license check
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15738/1/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py
File build-support/mini-cluster/relocate_binaries_for_mini_cluster.py:

http://gerrit.cloudera.org:8080/#/c/15738/1/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py@54
PS1, Line 54: # Exclude libraries that are (L)GPL-licensed and libraries that 
are not
            : # portable across Linux kernel versions.
> Is libpcre a beast of a different nature?
libpcre is BSD licensed, which is GPL-licensed compatible 
(https://en.wikipedia.org/wiki/License_compatibility)? (Moreover as libpcre is 
a transitive dependency introduced by libselinux in this case) Do you think I 
need to call it out that it is BSD licensed?



--
To view, visit http://gerrit.cloudera.org:8080/15738
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0fc69eaf57025b1ce2f7857808e60218dc4f2771
Gerrit-Change-Number: 15738
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 15 Apr 2020 22:30:43 +0000
Gerrit-HasComments: Yes

Reply via email to