Unify the version string to be displayed on "About Chromium" dialog.

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 84 matching lines @@
     a VersionInfo object or None on error.
   """
   proc = RunGitCommand(directory, ['rev-parse', 'HEAD'])
   if proc:
     output = proc.communicate()[0].strip()
     if proc.returncode == 0 and output:
       return VersionInfo('git', 'git', output[:7])
   return None
 
 
-def IsGitSVN(directory):
-  """
-  Checks whether git-svn has been set up.
-
-  Errors are swallowed.
-
-  Returns:
-    whether git-svn has been set up.
-  """
-  # To test whether git-svn has been set up, query the config for any
-  # svn-related configuration.  This command exits with an error code
-  # if there aren't any matches, so ignore its output.
-  proc = RunGitCommand(directory, ['config', '--get-regexp', '^svn'])
-  if proc:
-    return (proc.wait() == 0)
-  return False
-
-
-def FetchGitSVNURL(directory):
-  """
-  Fetch URL of SVN repository bound to git.
-
-  Errors are swallowed.
-
-  Returns:
-    SVN URL.
-  """
-  if IsGitSVN(directory):
-    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:
-        # Zero return code implies presence of requested configuration variable.
-        # Its value is second (last) field of output.
-        match = re.search(r'\S+$', output)
-        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(r'^\s*git-svn-id:\s+(\S+)@(\d+)')
-  proc = RunGitCommand(directory, ['log', '-' + str(depth)])
-  if proc:
-    for line in proc.stdout:
-      match = git_re.match(line)
-      if match:
-        id = match.group(2)
-        if id:
-          proc.stdout.close()  # Cut pipe for fast exit.
-          return id
-  return None
-
-
-def IsGitSVNDirty(directory):
-  """
-  Checks whether our git-svn tree contains clean trunk or some branch.
-
-  Errors are swallowed.
-  """
-  # For git branches the last commit message is either
-  # some local commit or a merge.
-  return LookupGitSVNRevision(directory, 1) is None
-
-
 def FetchGitSVNRevision(directory):
   """
-  Fetch the Git-SVN identifier for the local tree.
+  Fetch the SVN revision for a given directory through Git.
 
   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'
-  url = FetchGitSVNURL(directory)
-  root = FetchGitSVNRoot(directory)
-  return VersionInfo(url, root, revision)
+  proc = RunGitCommand(directory, ['log', '-1',
+                                   '--grep=git-svn-id', '--format=%b'])
+  if proc:
+    output = proc.communicate()[0].strip()
+    if proc.returncode == 0 and output:
+      # Extracts both the last changed SVN revision and the SVN URL.
+      # git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85528 0039d316...
+      match = re.search(r'git-svn-id:\s*([^@]*)@([0-9]+)', output)

[Denis Lagno, 2011/06/15 12:25:34]

sometimes it will give incorrect revision:
for example, below is real commit message:
Code will detect 17605 instead of correct 17638:

commit 8688ab4a4657533405da0d2d247ff063b8e9c933
Author: willchan@chromium.org
<willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date:   Thu Jun 4 17:12:48 2009 +0000

    Fix purify freeze.  Revert "Use a priority queue to assure that subresources
are resolved asap"
    This reverts git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17605
0039d316-1c4b-4281-b951-d872f2087c98
    TBR=jar
    
    Review URL: http://codereview.chromium.org/118239
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17638
0039d316-1c4b-4281-b951-d872f2087c98

[haraken1, 2011/06/15 16:22:53]

I modified so that it chooses the "git-svn-id: ..." line that appears first
after "Review URL: ..." line. How about it??

[Denis Lagno, 2011/06/15 17:14:55]

maybe just pick last match instead of first?

[haraken1, 2011/06/16 00:38:01]

Done.

+      if match:
+        url = match.group(1)
+        revision = match.group(2)
+
+        proc = RunGitCommand(directory, ['log', '-1'])
+        if proc:
+          output = proc.communicate()[0].strip()
+          if proc.returncode == 0 and output:
+            # Checks if there are any local commits.

[Denis Lagno, 2011/06/15 12:25:34]

as long as we are here I'd prefer to do this in more correct way: it should be
marked -dirty not only if there are local commits but also if there is any local
uncommitted change (i.e. if "git checkout" command produces non-zero output).

[haraken1, 2011/06/15 16:22:53]

Done.

+            match = re.search(r'git-svn-id:', output)
+            if not match:
+              revision = revision + "-dirty"
+        return VersionInfo(url, 'git-svn', revision)
+  return None
 
 
 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())