Fetch repository root for git-svn checkouts.

build/util/lastchange.py

 #!/usr/bin/env python
 # Copyright (c) 2011 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.
 
 """
 lastchange.py -- Chromium revision fetching utility.
 """
 
 import re
@@ skipping 122 matching lines @@
     proc = RunGitCommand(directory, ['svn', 'info', '--url'])
     if proc:
       output = proc.communicate()[0].strip()
       if proc.returncode == 0:
         match = re.search(r'^\w+://.*$', output, re.M)
         if match:
           return match.group(0)
   return ''
 
 
+def FetchGitSVNRoot(directory):
+  """
+  Fetch root of SVN repository bound to git.
+
+  Errors are swallowed.
+
+  Returns:
+    SVN root repository.
+  """
+  if IsGitSVN(directory):
+    git_command = ['config', '--get-regexp', '^svn-remote.svn.url$']
+    proc = RunGitCommand(directory, git_command)
+    if proc:
+      output = proc.communicate()[0].strip()
+      if proc.returncode == 0:
+        match = re.search(r'\S+$', output)

[Evan Martin, 2011/03/03 23:53:09]

What is this regex for?  Does this command sometimes output empty lines?

Do you mean to have the re.MULTILINE attr on there or is it always going to
output a single line?

[Denis Lagno, 2011/03/04 00:15:21]

Executing it on my checkout produces:

$ git config --get-regexp ^svn-remote.svn.url$
svn-remote.svn.url svn://svn.chromium.org/chrome

That is why I need regexp: to grab second field.
I guess there is no need for MULTILINE because \S does not match newline.

+        if match:
+          return match.group(0)
+  return ''
+
+
 def LookupGitSVNRevision(directory, depth):
   """
   Fetch the Git-SVN identifier for the local tree.
   Parses first |depth| commit messages.
 
   Errors are swallowed.
   """
   if not IsGitSVN(directory):
     return None
-  git_re = re.compile('^\s*git-svn-id:\s+(\S+)@(\d+)', re.M)
+  git_re = re.compile(r'^\s*git-svn-id:\s+(\S+)@(\d+)')
   proc = RunGitCommand(directory, ['log', '-' + str(depth)])
   if proc:
     for line in proc.stdout:
       match = git_re.search(line)

[Evan Martin, 2011/03/03 23:53:09]

I guess this could be .match() now

[Denis Lagno, 2011/03/04 00:15:21]

Done.

       if match:
         id = match.group(2)
         if id:
           proc.stdout.close()  # Cut pipe for fast exit.
           return id
   return None
 
 
 def IsGitSVNDirty(directory):
   """
@@ skipping 11 matching lines @@
   Fetch the Git-SVN identifier for the local tree.
 
   Errors are swallowed.
   """
   # We assume that at least first 999 commit messages contain svn evidence.
   revision = LookupGitSVNRevision(directory, 999)
   if not revision:
     return None
   if IsGitSVNDirty(directory):
     revision = revision + '-dirty'
-  return VersionInfo(FetchGitSVNURL(directory), 'git-svn', revision)
+  url = FetchGitSVNURL(directory)
+  root = FetchGitSVNRoot(directory)
+  return VersionInfo(url, root, revision)
 
 
 def FetchVersionInfo(default_lastchange, directory=None):
   """
   Returns the last change (in the form of a branch, revision tuple),
   from some appropriate revision control system.
   """
   version_info = (FetchSVNRevision(directory) or
       FetchGitSVNRevision(directory) or FetchGitRevision(directory))
   if not version_info:
@@ skipping 53 matching lines @@
     if out_file:
       WriteIfChanged(out_file, contents)
     else:
       sys.stdout.write(contents)
 
   return 0
 
 
 if __name__ == '__main__':
   sys.exit(main())