Title Mercurial integration: verify repo when adding
Priority bug Status resolved
Superseder Nosy List eric.araujo, ezio.melotti, loewis, ncoghlan
Assigned To Topics

Created on 2011-08-06.14:52:13 by eric.araujo, last changed 2018-06-16.04:03:06 by ncoghlan.

msg2206 (view) Author: eric.araujo Date: 2011-08-06.14:52:13
It has happened more than one time that people put an ssh or private https URI in the remote repo field, which makes Roundup error when someone wants to create a patch.  If possible, access should be checked when a repo is added.

(FTR, the Bitbucket https URIs are not private, this may be yet another bug.)
msg2580 (view) Author: ezio.melotti Date: 2012-09-01.17:23:35
> (FTR, the Bitbucket https URIs are not private, this may be yet another bug.)

This should be fixed with ebed4eccab36.

The error reported by the tracker was "('invalid token', 90)", and was caused by:

>>> old = 'max(ancestors(branch("%s")))-outgoing("%s"))' % (patchbranch, base)
>>> commands.diff(repo.ui, repo, rev=[old, patchbranch])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/dist-packages/mercurial/", line 2319, in diff
    node1, node2 = scmutil.revpair(repo, revs)
  File "/usr/lib/python2.7/dist-packages/mercurial/", line 497, in revpair
    l = revrange(repo, revs)
  File "/usr/lib/python2.7/dist-packages/mercurial/", line 549, in revrange
    m = revset.match(repo.ui, spec)
  File "/usr/lib/python2.7/dist-packages/mercurial/", line 1046, in match
    raise error.ParseError(_("invalid token"), pos)
mercurial.error.ParseError: ('invalid token', 90)

The problem was an extra ')', and told me which one had to be removed.  Bitbucket remote repos should now work.
msg3514 (view) Author: ncoghlan Date: 2018-06-16.04:03:06
Marking as obsolete given CPython's migration to GitHub (part of the issue review)
Date User Action Args
2018-06-16 04:03:06ncoghlansetstatus: chatting -> resolved
nosy: + ncoghlan
messages: + msg3514
2012-09-01 17:23:36ezio.melottisetstatus: unread -> chatting
nosy: + ezio.melotti
messages: + msg2580
2011-08-06 14:52:13eric.araujocreate