Pass arguments to pylint child process via a pipe.

Pass arguments to pylint child process via a pipe.

This gets around command-line length limitations.

BUG=
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=143620

[chrisha, 2012-06-22 14:51:48]

PTAL.

[M-A Ruel, 2012-06-22 15:26:41]

lgtm

https://chromiumcodereview.appspot.com/10654002/diff/2001/third_party/pylint.py
File third_party/pylint.py (right):

https://chromiumcodereview.appspot.com/10654002/diff/2001/third_party/pylint....
third_party/pylint.py:31: if arg:
Do you think empty args are a problem? If not, you could simplify the loop with:

args.extend(arg.rstrip() for arg in sys.stdin)

[chrisha, 2012-06-22 17:05:11]

Thanks, submitting to CQ.

https://chromiumcodereview.appspot.com/10654002/diff/2001/third_party/pylint.py
File third_party/pylint.py (right):

https://chromiumcodereview.appspot.com/10654002/diff/2001/third_party/pylint....
third_party/pylint.py:31: if arg:
On 2012/06/22 15:26:41, Marc-Antoine Ruel wrote:
> Do you think empty args are a problem? If not, you could simplify the loop
with:
> 
> args.extend(arg.rstrip() for arg in sys.stdin)

(No, I suppose I'm not really worried about this.)

Done.

[commit-bot: I haz the power, 2012-06-22 17:05:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/chrisha@chromium.org/10654002/4003

[commit-bot: I haz the power, 2012-06-22 17:08:13]

Change committed as 143620