check_return_value.py should drop stderr of the command it runs.

build/check_return_value.py

Index: build/check_return_value.py
diff --git a/build/check_return_value.py b/build/check_return_value.py
index 6f0daec3a751bdf4f043274f1196193929b5dd4b..b7b0b406d039a92db7069ee02b745c3cb531fe3a 100755
--- a/build/check_return_value.py
+++ b/build/check_return_value.py
@@ -6,10 +6,12 @@
 """This program wraps an arbitrary command and prints "1" if the command ran
 successfully."""
 
+import os
 import subprocess
 import sys
 
-if not subprocess.call(sys.argv[1:]):
+devnull = open(os.devnull, 'wb')
+if not subprocess.call(sys.argv[1:], stderr=devnull):

[Mark Mentovai, 2014/04/02 20:12:30]

I think you should also suppress stdout, since you want to replace it with your
own stdout.

   print 1
 else:
   print 0