Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16785 )

Change subject: [thirdparty] Upgrade CMake to 3.19.1
......................................................................


Patch Set 2:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/16785/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16785/1//COMMIT_MSG@10
PS1, Line 10: with
> with
Done


http://gerrit.cloudera.org:8080/#/c/16785/1//COMMIT_MSG@12
PS1, Line 12: includes
> includes
Done


http://gerrit.cloudera.org:8080/#/c/16785/1//COMMIT_MSG@14
PS1, Line 14: occurre
> occurred
Done


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

http://gerrit.cloudera.org:8080/#/c/16785/1/src/kudu/codegen/CMakeLists.txt@100
PS1, Line 100:     --sysroot="${CMAKE_OSX_SYSROOT}")
> Yup, it's in CMake 3.11 already, so it should be safe (we require cmake of
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I125448a3057d0c9da8c82f7e29683ef7844b0aa8
Gerrit-Change-Number: 16785
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Comment-Date: Wed, 25 Nov 2020 23:20:55 +0000
Gerrit-HasComments: Yes

Reply via email to