Merge lp://staging/~mbp/bzr/trivial into lp://staging/bzr

Proposed by Martin Pool
Status: Merged
Approved by: Andrew Bennetts
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp://staging/~mbp/bzr/trivial
Merge into: lp://staging/bzr
Diff against target: 29 lines (+6/-2)
2 files modified
NEWS (+3/-0)
bzrlib/builtins.py (+3/-2)
To merge this branch: bzr merge lp://staging/~mbp/bzr/trivial
Reviewer Review Type Date Requested Status
Andrew Bennetts Approve
Review via email: mp+21998@code.staging.launchpad.net

Commit message

(mbp) avoid truncating svn urls

Description of the change

Be more careful in removing trailing slash, as described in https://bugs.edge.launchpad.net/bzr-svn/+bug/545185

To post a comment you must log in.
Revision history for this message
Andrew Bennetts (spiv) wrote :

But what if the branch_location has multiple trailing slashes? <wink>

Seems fine. :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.