You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2021/03/03 16:31:07 UTC
[kudu-CR] [mini-cluster] Fix mini-cluster license check when built on CentOS 8
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17148
Change subject: [mini-cluster] Fix mini-cluster license check when built on CentOS 8
......................................................................
[mini-cluster] Fix mini-cluster license check when built on CentOS 8
When building the mini-cluser on CentOS 8 the Cyrus SASL
libraries also link in the libpcre2 library which is BSD licensed.
This patch adds that libary to the license file to fix the check
and allow the build to complete.
Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
---
M build-support/mini-cluster/LICENSE-BINARY-JAR-LINUX.txt
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/17148/1
--
To view, visit http://gerrit.cloudera.org:8080/17148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Gerrit-Change-Number: 17148
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR] [mini-cluster] Fix mini-cluster license check when built on CentOS 8
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17148 )
Change subject: [mini-cluster] Fix mini-cluster license check when built on CentOS 8
......................................................................
Patch Set 1: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/17148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Gerrit-Change-Number: 17148
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 03 Mar 2021 18:08:35 +0000
Gerrit-HasComments: No
[kudu-CR] [mini-cluster] Fix mini-cluster license check when built on CentOS 8
Posted by "Bankim Bhavsar (Code Review)" <ge...@cloudera.org>.
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17148 )
Change subject: [mini-cluster] Fix mini-cluster license check when built on CentOS 8
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/17148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Gerrit-Change-Number: 17148
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 03 Mar 2021 18:07:26 +0000
Gerrit-HasComments: No
[kudu-CR] [mini-cluster] Fix mini-cluster license check when built on CentOS 8
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17148 )
Change subject: [mini-cluster] Fix mini-cluster license check when built on CentOS 8
......................................................................
[mini-cluster] Fix mini-cluster license check when built on CentOS 8
When building the mini-cluser on CentOS 8 the Cyrus SASL
libraries also link in the libpcre2 library which is BSD licensed.
This patch adds that libary to the license file to fix the check
and allow the build to complete.
Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Reviewed-on: http://gerrit.cloudera.org:8080/17148
Reviewed-by: Bankim Bhavsar <ba...@cloudera.com>
Tested-by: Grant Henke <gr...@apache.org>
---
M build-support/mini-cluster/LICENSE-BINARY-JAR-LINUX.txt
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Bankim Bhavsar: Looks good to me, approved
Grant Henke: Verified
--
To view, visit http://gerrit.cloudera.org:8080/17148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Gerrit-Change-Number: 17148
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [mini-cluster] Fix mini-cluster license check when built on CentOS 8
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has removed a vote on this change.
Change subject: [mini-cluster] Fix mini-cluster license check when built on CentOS 8
......................................................................
Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/17148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Icc697668c0a81659c9f358487b037b1443a1b0d9
Gerrit-Change-Number: 17148
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)