Add gclient grep for git repos

gclient.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Meta checkout manager supporting both Subversion and GIT.
 
 Files
   .gclient      : Current client configuration, written by 'config' command.
                   Format is a Python script defining 'solutions', a list whose
@@ skipping 1243 matching lines @@
   return client.RunOnDeps('recurse', args)
 
 
 @attr('usage', '[args ...]')
 def CMDfetch(parser, args):
   """Fetches upstream commits for all modules.
 
 Completely git-specific. Simply runs 'git fetch [args ...]' for each module.
 """
   (options, args) = parser.parse_args(args)
-  args = ['-j%d' % options.jobs, '-s', 'git', 'git', 'fetch'] + args
-  return CMDrecurse(parser, args)
+  return CMDrecurse(Parser(), [
+      '--jobs=%d' % options.jobs, '--scm=git', 'git', 'fetch'] + args)
+
+
+def CMDgrep(parser, args):
+  """Greps through git repos managed by gclient.
+
+Runs 'git grep [args...]' for each module.
+"""
+  if not args:
+    print 'Usage [-j <N>] git-grep args...'

[M-A Ruel, 2012/11/05 17:39:05]

print >> sys.stderr, 'Usage: git cl grep [-j <N>] <git-grep args>...'
or:
parser.remove('--help')
parser.help()

but I'm not sure it'd work properly either. So I'm fine with just printing usage
to stderr.

[Isaac (away), 2012/11/07 07:13:53]

Done.

+    return 1
+
+  # We can't use optparse because it will try to parse arguments sent

[M-A Ruel, 2012/11/05 13:51:14]

You can, you need to use parser.disable_interspersed_args()

[Isaac (away), 2012/11/05 17:09:53]

That does not allow users to pass arguments to git grep. 
i.e. gclient grep -i foo to call git grep with -i. 

[M-A Ruel, 2012/11/05 17:39:05]

Ok, I always used -- naturally but I agree it's user hostile.

[Isaac (away), 2012/11/07 07:13:53]

SG.  Some git commands also use -- so prefer not to confuse people.

+  # to git grep and throw an error. :-(
+  if re.match(r'-j\d+|--jobs=\d+', args[0]):
+    jobs_arg, args = args[:1], args[1:]
+  elif re.match(r'-j|--jobs', args[0]):
+    jobs_arg, args = args[:2], args[2:]
+  else:
+    jobs_arg = ['--jobs=1']
+

[M-A Ruel, 2012/11/05 17:39:05]

You also lose support for -h and --help. But it's not much a big deal.

[Isaac (away), 2012/11/07 07:13:53]

I fixed this w/ a regex -- see new review.

+  return CMDrecurse(
+      parser,
+      jobs_arg + ['--ignore', '--scm=git', 'git', 'grep', '--color=Always']
+      + args)
 
 
 @attr('usage', '[url] [safesync url]')
 def CMDconfig(parser, args):
   """Create a .gclient file in the current directory.
 
 This specifies the configuration for further commands. After update/sync,
 top-level DEPS files in each module are read to determine dependent
 modules to operate on as well. If optional [url] parameter is
 provided, then configuration is read from a specified Subversion server
@@ skipping 397 matching lines @@
   except (gclient_utils.Error, subprocess2.CalledProcessError), e:
     print >> sys.stderr, 'Error: %s' % str(e)
     return 1
 
 
 if '__main__' == __name__:
   fix_encoding.fix_encoding()
   sys.exit(Main(sys.argv[1:]))
 
 # vim: ts=2:sw=2:tw=80:et: