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 1231 matching lines @@
 
 @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)
 

[M-A Ruel, 2012/10/26 12:45:42]

2 lines

[Isaac (away), 2012/11/05 07:08:49]

Done.

+@attr('usage', '[args ...]')
+def CMDgrep(parser, args):
+  """Greps through git repos managed by gclient.
+
+Runs 'git grep [args ...]' for each module.
+"""
+  parser.disable_interspersed_args()
+  parser.set_defaults(jobs=1)

[M-A Ruel, 2012/10/26 12:45:42]

I think it'd be preferable to remove the option with
parser.remove_option('--jobs')

[Isaac (away), 2012/11/05 07:08:49]

I'd like to leave this option available in some form.  -j=<high> is useful if
one isn't expecting a lot of matches.  I re-wrote the parsing code in the latest
patch.

+  parser.add_option('--color', default='Always',
+                    help='--color param passed to git grep, default=Always')
+  (options, args) = parser.parse_args(args)
+  print 'jobs', options.jobs
+  args = ['-j%d' % options.jobs, '-i', '-s', 'git', 'git', 'grep',

[M-A Ruel, 2012/10/26 12:45:42]

'-jobs', '1', '--ignore', '--scm', 'git', ...
Can you use the long argument names instead? This makes the command line clearer
when reading the code.

[Isaac (away), 2012/11/05 07:08:49]

Done.  I was copying CMDfetch from above.  I've updated that one too.

+          '--color=%s' % options.color] + args
+  return CMDrecurse(parser, 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
 URL.
@@ skipping 301 matching lines @@
   """Returns the default parser."""
   parser = optparse.OptionParser(version='%prog ' + __version__)
   # some arm boards have issues with parallel sync.
   if platform.machine().startswith('arm'):
     jobs = 1
   else:
     jobs = max(8, gclient_utils.NumLocalCpus())
   gclientfile_default = os.environ.get('GCLIENT_FILE', '.gclient')
   parser.add_option('-j', '--jobs', default=jobs, type='int',
                     help='Specify how many SCM commands can run in parallel; '
-                          'defaults to number of cpu cores (%default)')
+                          'default=%default')

[M-A Ruel, 2012/10/26 12:45:42]

Don't change here.

[Isaac (away), 2012/11/05 07:08:49]

Done.

   parser.add_option('-v', '--verbose', action='count', default=0,
                     help='Produces additional output for diagnostics. Can be '
                           'used up to three times for more logging info.')
   parser.add_option('--gclientfile', dest='config_filename',
                     default=None,
                     help='Specify an alternate %s file' % gclientfile_default)
   parser.add_option('--spec',
                     default=None,
                     help='create a gclient file containing the provided '
                          'string. Due to Cygwin/Python brokenness, it '
@@ skipping 74 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: