Chromium Code Reviews Chromium Code Reviews
Issue 2013943002:
Changing license header, again! (Closed)
Base URL: git@github.com:luci/luci-py.git@master| OLD | NEW |
|---|---|
| 1 # coding=utf8 | 1 # coding=utf8 |
| 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 3 # Use of this source code is governed by a BSD-style license that can be | 3 # Use of this source code is governed by a BSD-style license that can be |
| 4 # found in the LICENSE file. | 4 # that can be found in the LICENSE file. |
|
estaab
2016/05/25 20:29:09
Do you need to modify these third_party ones?
M-A Ruel
2016/05/25 23:21:53
Oops, thanks for catching!
| |
| 5 """Collection of subprocess wrapper functions. | 5 """Collection of subprocess wrapper functions. |
| 6 | 6 |
| 7 In theory you shouldn't need anything else in subprocess, or this module failed. | 7 In theory you shouldn't need anything else in subprocess, or this module failed. |
| 8 """ | 8 """ |
| 9 | 9 |
| 10 import cStringIO | 10 import cStringIO |
| 11 import errno | 11 import errno |
| 12 import logging | 12 import logging |
| 13 import os | 13 import os |
| 14 import Queue | 14 import Queue |
| (...skipping 500 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 515 | 515 |
| 516 - Throws if return code is not 0. | 516 - Throws if return code is not 0. |
| 517 - Works even prior to python 2.7. | 517 - Works even prior to python 2.7. |
| 518 - Blocks stdin by default if not specified since no output will be visible. | 518 - Blocks stdin by default if not specified since no output will be visible. |
| 519 - As per doc, "The stdout argument is not allowed as it is used internally." | 519 - As per doc, "The stdout argument is not allowed as it is used internally." |
| 520 """ | 520 """ |
| 521 kwargs.setdefault('stdin', VOID) | 521 kwargs.setdefault('stdin', VOID) |
| 522 if 'stdout' in kwargs: | 522 if 'stdout' in kwargs: |
| 523 raise ValueError('stdout argument not allowed, it would be overridden.') | 523 raise ValueError('stdout argument not allowed, it would be overridden.') |
| 524 return check_call_out(args, stdout=PIPE, **kwargs)[0] | 524 return check_call_out(args, stdout=PIPE, **kwargs)[0] |
| OLD | NEW |
