Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(189)

Unified Diff: cros_mark_as_stable.py

Issue 5063004: Don't use output argument for CalledProcessError (Python 2.7 only) (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
Patch Set: Created 10 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cros_mark_as_stable.py
diff --git a/cros_mark_as_stable.py b/cros_mark_as_stable.py
index b2f2cd4067600aa7c275427dfd9a466def960752..5df6a83fef1989d811fe28c895ae6d228387185f 100755
--- a/cros_mark_as_stable.py
+++ b/cros_mark_as_stable.py
@@ -248,7 +248,8 @@ def _SimpleRunCommand(command):
stdout = proc_handle.communicate()[0]
retcode = proc_handle.wait()
if retcode != 0:
- raise subprocess.CalledProcessError(retcode, command, output=stdout)
+ _Print(stdout)
+ raise subprocess.CalledProcessError(retcode, command)
return stdout
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698