Handle review URLs pointing directly at Gerrit

If a review URL is set to 'http://host/Gerrit' because the user
thinks that is the correct way to point repo at Gerrit, we should
be a bit more flexible and fix the URL by dropping the '/Gerrit'
suffix and replace it with '/ssh_info'.

Likewise, if a review URL points already at '/ssh_info' for a Gerrit
instance, we should leave it alone.

Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce 2009-03-25 13:54:54 -07:00
parent feabbdb440
commit 13cc3844d7

View File

@ -273,6 +273,9 @@ class Remote(object):
u = self.review u = self.review
if not u.startswith('http:') and not u.startswith('https:'): if not u.startswith('http:') and not u.startswith('https:'):
u = 'http://%s' % u u = 'http://%s' % u
if u.endswith('/Gerrit'):
u = u[:len(u) - len('/Gerrit')]
if not u.endswith('/ssh_info'):
if not u.endswith('/'): if not u.endswith('/'):
u += '/' u += '/'
u += 'ssh_info' u += 'ssh_info'