Dan Burkert has posted comments on this change. Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds ......................................................................
Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4511/1//COMMIT_MSG Commit Message: PS1, Line 23: unsafe not sure unsafe is the right term here- it just fails to link, right? http://gerrit.cloudera.org:8080/#/c/4511/1/build-support/dist_test.py File build-support/dist_test.py: Line 66: "thirdparty/installed-deps/bin/llvm-symbolizer", perhaps this should just use the clang-toolchain symlink? http://gerrit.cloudera.org:8080/#/c/4511/1/build-support/run-test.sh File build-support/run-test.sh: Line 101: export ASAN_SYMBOLIZER_PATH=$SOURCE_ROOT/thirdparty/installed-deps/bin/llvm-symbolizer and likewise here, if that's what you choose. http://gerrit.cloudera.org:8080/#/c/4511/1/build-support/run_dist_test.py File build-support/run_dist_test.py: Line 131: env['ASAN_SYMBOLIZER_PATH'] = os.path.join(ROOT, "thirdparty/installed-deps/bin/llvm-symbolizer") and here http://gerrit.cloudera.org:8080/#/c/4511/1/thirdparty/build-definitions.sh File thirdparty/build-definitions.sh: Line 114: ./configure What's the effect of this change? -- To view, visit http://gerrit.cloudera.org:8080/4511 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id9e68126ae21e04469053009c5b3e4b588415895 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes