diff options
| author | Simon Glass <sjg@chromium.org> | 2016-07-25 18:59:00 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2016-09-18 21:04:38 -0600 |
| commit | 785f1548a9e74cf4796c96e6f32ed67b25f79b81 (patch) | |
| tree | 33e5bf6ef78a66caf34372759283f3affb6a0f86 /tools/patman/command.py | |
| parent | 49afb37988a397ce215e99563e0ab5d2970c6a39 (diff) | |
patman: Adjust command.Output() to raise an error by default
It is more useful to have this method raise an error when something goes
wrong. Make this the default and adjust the few callers that don't want to
use it this way.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/patman/command.py')
| -rw-r--r-- | tools/patman/command.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/patman/command.py b/tools/patman/command.py index d586f11158..d1f0ca505c 100644 --- a/tools/patman/command.py +++ b/tools/patman/command.py @@ -104,8 +104,9 @@ def RunPipe(pipe_list, infile=None, outfile=None, raise Exception("Error running '%s'" % user_pipestr) return result -def Output(*cmd): - return RunPipe([cmd], capture=True, raise_on_error=False).stdout +def Output(*cmd, **kwargs): + raise_on_error = kwargs.get('raise_on_error', True) + return RunPipe([cmd], capture=True, raise_on_error=raise_on_error).stdout def OutputOneLine(*cmd, **kwargs): raise_on_error = kwargs.pop('raise_on_error', True) |