Pass arguments to pylint child process via a pipe.

third_party/pylint.py

Index: third_party/pylint.py
diff --git a/third_party/pylint.py b/third_party/pylint.py
index 1a8207db39731586c24222298237695649cc9047..3972281d004108cc18f729529d6a82fae198ac78 100644
--- a/third_party/pylint.py
+++ b/third_party/pylint.py
@@ -13,7 +13,22 @@
 # 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 """ Copyright (c) 2002-2008 LOGILAB S.A. (Paris, FRANCE).
 http://www.logilab.fr/ -- mailto:contact@logilab.fr
+
+Copyright (c) 2012 The Chromium Authors. All rights reserved.
 """
 import sys
 from pylint import lint
-lint.Run(sys.argv[1:])
+
+args = sys.argv[1:]
+
+# Add support for a custom mode where arguments are fed line by line on
+# stdin. This allows us to get around command line length limitations.
+ARGS_ON_STDIN = '--args-on-stdin'
+if ARGS_ON_STDIN in args:
+  args = [arg for arg in args if arg != ARGS_ON_STDIN]
+  for arg in sys.stdin:
+    arg = arg.strip()
+    if arg:

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

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]

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

Done.

+      args.append(arg)
+
+lint.Run(args)