Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18822 )
Change subject: KUDU-3386 Update macOS docs for ARM ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/18822/1/docs/installation.adoc File docs/installation.adoc: http://gerrit.cloudera.org:8080/#/c/18822/1/docs/installation.adoc@609 PS1, Line 609: .ARM macs > Should I then move this out of the "ARM" label? Could someone please, with I guess the build fails on macOS (both on Intel and ARM) if not pointing directly to [email protected] because of some FIPS-related functions that our code uses: by default, newer versions of macOS include LibreSSL 2.8.x where that FIPS-related API is different from OpenSSL 1.x.x. Also, in many cases MacPorts and homebrew packages install OpenSSL 3.0.x by dependency, and there the FIPS-related API is also different from what it is in OpenSSL 1.x.x. At least, that's what happens at my macOS (Intel) laptop. http://gerrit.cloudera.org:8080/#/c/18822/2/docs/installation.adoc File docs/installation.adoc: http://gerrit.cloudera.org:8080/#/c/18822/2/docs/installation.adoc@568 PS2, Line 568: Xcode introduced support for Apple silicon in version : link:https://developer.apple.com/documentation/xcode-release-notes/xcode-12_2-release-notes[12.2]. : Xcode versions up until 13.2.1 can be used with Kudu. > Basically Xcode version 13.2.1 inclusive, can be used to build all thirdpar Thanks for the clarification. It would be great to add reference to KUDU-3391 along with the information on the highest version of Xcode which can be used to build Kudu: something like '..., for details see KUDU-3391.' >From the other side, I guess we could update gtest up to 1.12.1 version (it >seems the issue has been fixed at least in 1.11.1) now and remove that >constraint. -- To view, visit http://gerrit.cloudera.org:8080/18822 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I02e5993979b88b525006344324e6b3be29aa869c Gerrit-Change-Number: 18822 Gerrit-PatchSet: 1 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 25 Aug 2022 01:23:55 +0000 Gerrit-HasComments: Yes
