[MERGE] use urlutils.relative_url in info (Fixes: bug #76229)
John A Meinel
john at arbash-meinel.com
Tue Dec 19 14:23:35 GMT 2006
John Arbash Meinel has voted +1 (conditional).
Status is now: Conditionally approved
Comment:
I'm pretty sure you want rstrip('/') instead of strip('/') here:
+ result = urlutils.relative_url(repo_url, inner_url)
+ if result != inner_url:
+ result = result.strip('/')
+ return result
Otherwise it could strip an opening slash. That should never happen, since URLs start with a protocol, but just in case...
For details, see: http://bundlebuggy.aaronbentley.com/request/%3C4586B16E.6060508%40utoronto.ca%3E
More information about the bazaar
mailing list