Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23198 )
Change subject: KUDU-3678: Fix crcutil building on MacOS arm64 ...................................................................... KUDU-3678: Fix crcutil building on MacOS arm64 When building crcutil, it uses 'uname -p' to determine the processor, which prints 'arm'. This means that it doesn't match against the correct processor and adds -msse2 -msse4.2 to crcutil_flags, neither of which are supported on Apple silicon. This adds a new branch that specifically targets MacOS devices running on arm, allowing crcutil to compile correctly. Change-Id: I71a1e70a8bc29732a8d0b31724bdbf8e00e7a736 Reviewed-on: http://gerrit.cloudera.org:8080/23198 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M thirdparty/download-thirdparty.sh A thirdparty/patches/crcutil-fix-macos-arm64-flags.patch 2 files changed, 14 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23198 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I71a1e70a8bc29732a8d0b31724bdbf8e00e7a736 Gerrit-Change-Number: 23198 Gerrit-PatchSet: 2 Gerrit-Owner: Jeffrey Smith <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
