Changes GetSVNBranch to prefer the first ref over the last. The...

scm.py

 # Copyright (c) 2006-2009 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.
 
 """SCM-specific utility classes."""
 
 import glob
 import os
 import re
 import shutil
@@ skipping 154 matching lines @@
     git_svn_re = re.compile(r'^\s*git-svn-id: (\S+)@', re.MULTILINE)
 
     # Get the refname and svn url for all refs/remotes/*.
     remotes = GIT.Capture(
         ['for-each-ref', '--format=%(refname)', 'refs/remotes'],
         cwd)[0].splitlines()
     svn_refs = {}
     for ref in remotes:
       match = git_svn_re.search(
           GIT.Capture(['cat-file', '-p', ref], cwd)[0])
-      if match:
+      if match and match.group(1) not in svn_refs:

[M-A Ruel, 2010/04/08 16:36:38]

nit: you could do instead
if match:
  # To prefer local refs over remote ones we only set the first occurrence.
  # The assumption being local refs are usually first.
  svn_refs.setdefault(match.group(1), ref)

(Fixed "occurrence")

+        # To prefer local refs over remote ones we only set the first occurence.
+        # The assumption being local refs are usually first.
         svn_refs[match.group(1)] = ref
 
     svn_branch = ''
     if len(svn_refs) == 1:
       # Only one svn branch exists -- seems like a good candidate.
       svn_branch = svn_refs.values()[0]
     elif len(svn_refs) > 1:
       # We have more than one remote branch available.  We don't
       # want to go through all of history, so read a line from the
       # pipe at a time.
@@ skipping 570 matching lines @@
     if not cur_dir_repo_root:
       return None
 
     while True:
       parent = os.path.dirname(directory)
       if (SVN.CaptureInfo(parent, print_error=False).get(
               "Repository Root") != cur_dir_repo_root):
         break
       directory = parent
     return GetCasedPath(directory)