Dan Burkert has submitted this change and it was merged. Change subject: thirdparty: build C dependencies with TSAN instrumentation too ......................................................................
thirdparty: build C dependencies with TSAN instrumentation too Our C library dependencies are pretty small so this isn't a huge hit, and it makes the sanitizer-based dependency stack much more consistent. Now, 'common' is only for tools and header-only dependencies, while uninstrumented/tsan/... include every library. With additional sanitization come new suppressions. I didn't spend a whole lot of time trying to figure out which were legitimate. Change-Id: Ie8274ff7bc57016733cdd3c26858483f74e68faf Reviewed-on: http://gerrit.cloudera.org:8080/4559 Reviewed-by: Dan Burkert <d...@cloudera.com> Tested-by: Kudu Jenkins Tested-by: Dan Burkert <d...@cloudera.com> --- M build-support/tsan-suppressions.txt M cmake_modules/FindPmem.cmake M thirdparty/build-definitions.sh M thirdparty/build-thirdparty.sh 4 files changed, 144 insertions(+), 121 deletions(-) Approvals: Dan Burkert: Looks good to me, approved; Verified Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4559 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8274ff7bc57016733cdd3c26858483f74e68faf Gerrit-PatchSet: 2 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>