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

Change subject: [cmake_modules] fix linker detection
......................................................................


Patch Set 3: Code-Review+2

(2 comments)

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

http://gerrit.cloudera.org:8080/#/c/23892/1//COMMIT_MSG@16
PS1, Line 16: because it uses:
            :   -fuse-ld={kudu}/thirdparty/clang-toolchain/bin
> Please run this on any normal linux system:
Thanks for the clarification and correcting the commit description.


http://gerrit.cloudera.org:8080/#/c/23892/1//COMMIT_MSG@19
PS1, Line 19: -fuse-ld=lld -B{kudu}/thirdparty/clang-toolchain/bin
> g++ does not accept a path (at least between 7.5 and 15.2). only clang does
Yes, this makes sense now.  Thanks for clarifying on this in the comment 
description!



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I567caf9b73b75020facce8ce800e50662af60f39
Gerrit-Change-Number: 23892
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Martonka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Sat, 07 Feb 2026 04:01:47 +0000
Gerrit-HasComments: Yes

Reply via email to