os.system didn't return the correct exit code (124). And I believe subprocess.call is intended to replace os.system anyway according to <a href="https://docs.python.org/2/library/subprocess.html#module-subprocess">this</a>.<br><br><div class="gmail_quote">On Wed Dec 03 2014 at 4:33:01 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: test/dosep.py:42<br>
@@ +41,3 @@<br>
+        )<br>
+        exit_code = subprocess.call(command, shell=True)<br>
+        if 0 != exit_code:<br>
----------------<br>
This appears to be equivalent to the previous code, but using subprocess instead of os.  If this is the only reason in the entire file for importing subprocess, then can we change it back to os.system to avoid importing the extra module?  Or alternatively, if the previous call to os.system was the only reason in the entire file for importing os, then keep the subprocess.call but remove the import of os.system.<br>
<br>
Other than that, LGTM<br>
<br>
<a href="http://reviews.llvm.org/D6364" target="_blank">http://reviews.llvm.org/D6364</a><br>
<br>
<br>
</blockquote></div>