[PATCH] D71402: [sanitizers] Redirect stdout and stderr to TASK_LOG

[Eric Christopher via Phabricator via llvm-commits]

echristo created this revision.
echristo added a reviewer: vitalybuka.
echristo added a project: Sanitizers.
Herald added subscribers: llvm-commits, Sanitizers, mcrosier.
Herald added a project: LLVM.

At some point cpplint.py became very noisy during a build spewing a few hundred lines of "Done processing..." even with SILENT=1 in cmake. This attempts to redirect the stdout of "Done processing" to the task log along with any errors.

Tested by this with and without SILENT=1 to check things.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D71402

Files:
  compiler-rt/lib/sanitizer_common/scripts/check_lint.sh


Index: compiler-rt/lib/sanitizer_common/scripts/check_lint.sh
===================================================================
--- compiler-rt/lib/sanitizer_common/scripts/check_lint.sh
+++ compiler-rt/lib/sanitizer_common/scripts/check_lint.sh
@@ -55,7 +55,7 @@
   FILTER=$1
   shift
   TASK_LOG=$(${MKTEMP})
-  ${CPPLINT} --filter=${FILTER} "$@" 2>$TASK_LOG
+  ${CPPLINT} --filter=${FILTER} "$@" > $TASK_LOG 2>&1
   if [ "$?" != "0" ]; then
     cat $TASK_LOG | grep -v "Done processing" | grep -v "Total errors found" \
       | grep -v "Skipping input" >> $ERROR_LOG


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71402.233536.patch
Type: text/x-patch
Size: 574 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191212/bdef1e20/attachment.bin>