[PATCH] Avoid bashisms and switch to /bin/sh

Alexey Samsonov vonosmas at gmail.com
Tue Oct 7 13:27:57 PDT 2014


LGTM

================
Comment at: lib/sanitizer_common/scripts/check_lint.sh:102-103
@@ -94,3 +101,4 @@
 LSAN_RTL=${COMPILER_RT}/lib/lsan
-run_lint ${LSAN_RTL_LINT_FILTER} ${LSAN_RTL}/*.{cc,h}
+run_lint ${LSAN_RTL_LINT_FILTER} ${LSAN_RTL}/*.cc \
+                                 ${LSAN_RTL}/*.h
 run_lint ${LSAN_LIT_TEST_LINT_FILTER} ${LIT_TESTS}/lsan/*/*.cc &
----------------
emaste wrote:
> For some reason there's no `&` on this run_lint invocation - is that just an oversight?
Yes, please add & here.

http://reviews.llvm.org/D5654






More information about the llvm-commits mailing list