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: (1 comment) http://gerrit.cloudera.org:8080/#/c/17626/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17626/1//COMMIT_MSG@9 PS1, Line 9: This patch fixes building crcutil with GCC 10. Prior to this patch, > Is it possible to land this change in the http://github.com/cloudera/crcuti Yep, I also was looking at that, but so far I saw everything there is done in a different order: first things appear as patches somewhere, and then those goes into crcutil. I guess this time we might do it in a different order, yep. -- 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: Wed, 23 Jun 2021 20:45:41 +0000 Gerrit-HasComments: Yes
