gclient grep: support pathspec arguments

gclient.py

Index: gclient.py
diff --git a/gclient.py b/gclient.py
index 48ac3a7f01002c98d4554801e8b8b78614c66818..812d071a0c1c024ab1ae6a3f0fce5e06c89e01d1 100644
--- a/gclient.py
+++ b/gclient.py
@@ -644,20 +644,26 @@ class Dependency(gclient_utils.WorkItem, DependencySettings):
           if options.prepend_dir:

[M-A Ruel, 2012/11/28 01:43:11]

if options.prepend_dir and scm == 'git':

[Isaac (away), 2012/11/28 02:02:36]

Done.

             print_stdout = False
             def filter_fn(line):

[M-A Ruel, 2012/11/28 01:43:11]

Add docstring along the lines
"""Git-specific path marshaling. It is optimized for git-grep."""

[Isaac (away), 2012/11/28 02:02:36]

Done.

+              def mod_path(git_pathspec):
+                match = re.match('(\S+?:)?([^\0]+)$', git_pathspec)

[M-A Ruel, 2012/11/28 01:43:11]

r

[Isaac (away), 2012/11/28 02:02:36]

Can't use r for these, because I am looking for a nullbyte (\0)

+                modified_path = os.path.join(self.name, match.group(2))
+                branch = match.group(1) or ''

[M-A Ruel, 2012/11/28 01:43:11]

When r'(\S+?:)?' doesn't match, match.group(1) is None?

[Isaac (away), 2012/11/28 02:02:36]

Yep

+                return '%s%s' % (branch, modified_path)
+
+              match = re.match('Binary file ([^\0]+) matches$', line)

[M-A Ruel, 2012/11/28 01:43:11]

r'^ ...

[Isaac (away), 2012/11/28 02:02:36]

ditto on the r.  The ^ isn't necessary because of the syntax of re.match (only
matches from beginning of string), but added anyway.

+              if match:
+                print 'Binary file %s matches' % mod_path(match.group(1))
+                return
+
               items = line.split('\0')
-              if len(items) == 1:
-                match = re.match('Binary file (.*) matches$', line)
-                if match:
-                  print 'Binary file %s matches' % os.path.join(
-                      self.name, match.group(1))
-                else:
-                  print line
-              elif len(items) == 2 and items[1]:
-                print '%s : %s' % (os.path.join(self.name, items[0]), items[1])
-              else:
+              if len(items) == 2 and items[1]:
+                print '%s : %s' % (mod_path(items[0]), items[1])

[M-A Ruel, 2012/11/28 01:43:11]

normal git-grep output doesn't have a whitespace before ':'

[Isaac (away), 2012/11/28 02:02:36]

Adding a space lets you double click and highlight the filename so I made a
concession.

+              elif len(items) >= 2:
                 # Multiple null bytes or a single trailing null byte indicate
                 # git is likely displaying filenames only (such as with -l)
-                print '\n'.join(os.path.join(self.name, f) for f in items if f)
+                print '\n'.join([mod_path(path) for path in items if path])

[M-A Ruel, 2012/11/28 01:43:11]

Please don't convert the generator into a list before sending to the join()
call.

[Isaac (away), 2012/11/28 02:02:36]

Nice!  I always forget about these.

+              else:
+                print line
           else:
             print_stdout = True
             filter_fn = None