Labels: Type-Defect Priority-Medium
New issue 1676 by jan.kneschke: bzr repos with local absolute path fails to
*NOTE: Do not post confidential information in this bug report.*
What version are you running?
1.0.8 and 1.5b2
What's the URL of the page containing the problem?
admin -> repositories -> new repo -> bzr
What steps will reproduce the problem?
1. register a bzr repo with a local, absolute path
What is the expected output? What do you see instead?
reviewboard .strip()s the path which also removes the leading / which later
leads to prepending the 'cwd' of the webserver to it.
Use .rstrip() instead to only strip the trailing / from the path.
What operating system are you using? What browser?
Please provide any additional information below.
--- reviewboard/scmtools/bzr.py 2010-05-12 09:31:05.000000000 +0200
2010-06-07 11:55:52.000000000 +0200
@@ -65,7 +65,7 @@
def _get_full_path(self, path, basedir=None):
"""Returns the full path to a file."""
- parts = [self.repository.path.strip("/")]
+ parts = [self.repository.path.rstrip("/")]
You received this message because you are subscribed to the Google Groups
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to
For more options, visit this group at