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 2:

(2 comments)

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.
How about rephrasing this to be more specific what 'use' means and

  To build Kudu on ARM-based Macs (Apple M chips), use Xcode of versions from 
<version A> to <version B> inclusive.

BTW, what's broken in Kudu build on M1 with Xcode 13.2.1?  Is this just the 
maximum version reported to build Kudu or there is some fundamental issue that 
prevents Kudu to be build with newer versions?  It would be great to clarify.


http://gerrit.cloudera.org:8080/#/c/18822/2/docs/installation.adoc@611
PS2, Line 611: :
It would be great to clarify why this is necessary.



--
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: 2
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: Tue, 09 Aug 2022 00:36:51 +0000
Gerrit-HasComments: Yes

Reply via email to