Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11176 )
Change subject: [build] Move default sanitizer options into build from shell scripts ...................................................................... Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/11176/8/build-support/run-test.sh File build-support/run-test.sh: http://gerrit.cloudera.org:8080/#/c/11176/8/build-support/run-test.sh@115 PS8, Line 115: # By default the sanitizers use addr2line utility to symbolize reports. : # llvm-symbolizer is faster, consumes less memory and produces much better reports. : # https://github.com/google/sanitizers/wiki/SanitizerCommonFlags : SYMBOLIZER_PATH="$SOURCE_ROOT/thirdparty/installed/uninstrumented/bin/llvm-symbolizer" : for SANITIZER in ASAN LSAN MSAN TSAN UBSAN; do : VAR_NAME="${SANITIZER}_OPTIONS" : # Only add the external_symbolizer_path if it is not set already. : if [[ ${!VAR_NAME} != *"external_symbolizer_path="* ]]; then : export ${SANITIZER}_OPTIONS="$(echo ${!VAR_NAME}) external_symbolizer_path=$SYMBOLIZER_PATH" : fi : done > There are only 2 copies right now, and I think this one can go away once we I added TODOs to follow up and remove both of these with a patch that pushes the logic into the build. http://gerrit.cloudera.org:8080/#/c/11176/8/src/kudu/cfile/CMakeLists.txt File src/kudu/cfile/CMakeLists.txt: http://gerrit.cloudera.org:8080/#/c/11176/8/src/kudu/cfile/CMakeLists.txt@58 PS8, Line 58: set(KUDU_TEST_LINK_LIBS ${KUDU_MIN_TEST_LIBS} cfile) > I tried modifying the build with functions and macros all failing to work. I figured this out. I need to use a macro for scoping reasons and appending ARGN directly wasn't working as I expected. -- To view, visit http://gerrit.cloudera.org:8080/11176 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9eab2d3ef0b107ae9a4e971bf8a6514bf425f645 Gerrit-Change-Number: 11176 Gerrit-PatchSet: 8 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Fri, 24 Aug 2018 14:00:15 +0000 Gerrit-HasComments: Yes
