Marton Greber has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18822 )

Change subject: KUDU-3386 Update macOS docs for ARM
......................................................................


Patch Set 4:

(2 comments)

> Could you please give us a rendered site of the docs, for example,
 > the github links?

Please find the current patch on my github pages site, pointing to the relevant 
macOS part: 
https://martongreber.github.io/docs/installation.html#osx_from_source

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: +
> I guess the build fails on macOS (both on Intel and ARM) if not pointing di
Thank you for the detailed explanation! Moved this part out of the "ARM label" 
as it is applicable of Intel and ARM Macs as well.


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: .ARM Macs
             : ====
             : Apple introduced support for Apple silicon in Xcode v
> Thanks for the clarification.
Thank you for making the gtest upgrade patch 
(9a8570b9c6b3d4e24e47ca830c089ab04667ef41)! Thanks to that Kudu is working with 
the latest Xcode as well (13.4.1). Thereby removed the version upper limit. 
Sometimes one might want to use older Xcode. For reference the lowest supported 
ARM version is kept here.



--
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: 4
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: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Thu, 08 Sep 2022 18:27:17 +0000
Gerrit-HasComments: Yes

Reply via email to