Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17626 )
Change subject: [thirdparty] fix building crcutil with GCC 10 ...................................................................... Patch Set 1: > The change looks good to me. I don't have a strong opinion on the > ordering of patching. Thank you for the review! I already posted a pull request for the crcutil repo athttps://github.com/cloudera/crcutil/pull/4 I hope to get that merged, and then I'll need to update this patch to use the newer snapshot of the crcutil package. -- To view, visit http://gerrit.cloudera.org:8080/17626 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibaba794ce00eb64a29e8038a8a2f4883f35c975a Gerrit-Change-Number: 17626 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 26 Jun 2021 00:10:43 +0000 Gerrit-HasComments: No
