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. Sounds good to me, will update the comment. > BTW, did you figure out why it has surfaced only now? If it's a transitive > dependency for libselinux, why didn't we see it before? I am not 100% sure, but looking at the transitive dependencies chain: libcurl.so.4 => /root/kudu/thirdparty/installed/uninstrumented/lib/libcurl.so.4 libk5crypto.so.3 => /lib64/libk5crypto.so.3 libkrb5support.so.0 => /lib64/libkrb5support.so.0 libselinux.so.1 => /lib64/libselinux.so.1 libpcre.so.1 => /lib64/libpcre.so.1 It looks like is initiated by libcurl which we recently make it a run-time dependency? -- 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 23:59:56 +0000 Gerrit-HasComments: Yes
