Choosing a default remote for determining base_url.

git_cl.py

Index: git_cl.py
===================================================================
--- git_cl.py	(revision 128068)
+++ git_cl.py	(working copy)
@@ -48,6 +48,7 @@
 DEFAULT_SERVER = 'https://codereview.appspot.com'
 POSTUPSTREAM_HOOK_PATTERN = '.git/hooks/post-cl-%s'
 DESCRIPTION_BACKUP_FILE = '~/.git_cl_description_backup'
+GIT_INSTRUCTIONS_URL = 'http://code.google.com/p/chromium/wiki/UsingNewGit'
 
 
 # Initialized in main()
@@ -308,6 +309,7 @@
     self._rpc_server = None
     self.cc = None
     self.watchers = ()
+    self._remote = None
 
   def GetCCList(self):
     """Return the users cc'd on this CL.
@@ -386,14 +388,39 @@
       self.upstream_branch = upstream_branch
     return self.upstream_branch
 
+  def GetRemote(self):
+    if not self._remote:
+      self._remote = self.FetchUpstreamTuple()[0]
+      if self._remote == '.':
+        def warn_default():
+          logging.warning('Could not determine which remote this change is '
+                         'associated with, so defaulting to "%s".  This may '
+                         'not be what you want.  You may prevent this message '
+                         'by running "git svn info" as documented here:  %s',
+                          self._remote,
+                          GIT_INSTRUCTIONS_URL)
+
+        remotes = RunGit(['remote'], error_ok=True).split()
+        if len(remotes) == 1:
+          self._remote, = remotes
+          warn_default()

[M-A Ruel, 2012/03/22 19:39:00]

Actually, I'd prefer to not warn here. It's fine actually. The user pulled from
a remote, pipelined CLs and is uploading that. The remote _is_ known and not
ambiguous.

[jmbaker1, 2012/03/22 19:56:58]

Done.

+        elif 'origin' in remotes:
+          self._remote = 'origin'
+          warn_default()
+        else:
+          logging.warn('Could not determine which remote this change is '
+                       'associated with.  You may prevent this message by '
+                       'running "git svn info" as documented here:  %s',
+                       GIT_INSTRUCTIONS_URL)
+    return self._remote
+
+
   def GetRemoteUrl(self):
     """Return the configured remote URL, e.g. 'git://example.org/foo.git/'.
 
     Returns None if there is no remote.
     """
-    remote = self.FetchUpstreamTuple()[0]
-    if remote == '.':
-      return None
+    remote = self.GetRemote()
     return RunGit(['config', 'remote.%s.url' % remote], error_ok=True).strip()
 
   def GetIssue(self):