[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