Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10352 )
Change subject: thirdparty: tweak clang compiler flags ...................................................................... Patch Set 2: (1 comment) I had to tweak this to get the tsan build to pass, since it didn't appear to like the EXTRA_LDFLAGS to be passed in the new location. http://gerrit.cloudera.org:8080/#/c/10352/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/10352/1//COMMIT_MSG@9 PS1, Line 9: Prior to this small tweak the thirdparty clang build output thousands of : warnings similar to this one > FWIW, I think that's only if you're compiling LLVM with clang. Obviously th Done -- To view, visit http://gerrit.cloudera.org:8080/10352 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ide4ddbff14d3745c6f2c2f9b14b00da790a6cec6 Gerrit-Change-Number: 10352 Gerrit-PatchSet: 2 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 09 May 2018 18:28:08 +0000 Gerrit-HasComments: Yes