Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18461 )

Change subject: [build] Fix codegen build on MacOS Monterey
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18461/3/src/kudu/codegen/CMakeLists.txt
File src/kudu/codegen/CMakeLists.txt:

http://gerrit.cloudera.org:8080/#/c/18461/3/src/kudu/codegen/CMakeLists.txt@72
PS3, Line 72: CMAKE_OSX_SYSROOT
> With this change now the compilation is broken on my macOS laptop
More information from my laptop, just in case it helps to narrow down the issue:

-- Using CMAKE_OSX_SYSROOT: 
/Applications/Xcode.12.4.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.1.sdk

tents/Developer/Platforms/MacOSX.platform/Developer/SDKs
total 0
drwxr-xr-x  5 root  wheel  160 Feb 28  2021 .
drwxr-xr-x  6 root  wheel  192 Jan  8  2021 ..
drwxr-xr-x  5 root  wheel  160 Nov 30  2020 DriverKit20.2.sdk
drwxr-xr-x  7 root  wheel  224 Nov 30  2020 MacOSX.sdk
lrwxr-xr-x  1 root  wheel   10 Feb 28  2021 MacOSX11.1.sdk -> MacOSX.sdk


And indeed, there isn't cstdint file in there:
MacBook-Pro:codegen[attila-sysroot-fix]$ find 
/Applications/Xcode.12.4.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.1.sdk
 -iname cstdint
MacBook-Pro:codegen[attila-sysroot-fix]$



-- 
To view, visit http://gerrit.cloudera.org:8080/18461
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc5721cb2445303b2e79d08b547e746929c7486d
Gerrit-Change-Number: 18461
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Khazar Mammadli <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Mon, 02 May 2022 20:39:05 +0000
Gerrit-HasComments: Yes

Reply via email to