Merge lp://staging/~rziai/bzr-java-lib/url-schemes into lp://staging/~verterok/bzr-java-lib/bzr-java-lib

Proposed by Ramon Ziai
Status: Merged
Approved by: Guillermo Gonzalez
Approved revision: 217
Merged at revision: 217
Proposed branch: lp://staging/~rziai/bzr-java-lib/url-schemes
Merge into: lp://staging/~verterok/bzr-java-lib/bzr-java-lib
Diff against target: 13 lines (+2/-1)
1 file modified
src/main/java/org/vcs/bazaar/client/core/BranchLocation.java (+2/-1)
To merge this branch: bzr merge lp://staging/~rziai/bzr-java-lib/url-schemes
Reviewer Review Type Date Requested Status
Guillermo Gonzalez Approve
Review via email: mp+52200@code.staging.launchpad.net

Description of the change

I had trouble pushing to or pulling from https+urllib:// URLs with BzrEclipse. I tracked down the issue to missing entries in BranchLocation.java, where I then added "bzr+http", "bzr+https" and "https+urllib". That very small change made things work for me, so I suggest to merge this into trunk.

To post a comment you must log in.
Revision history for this message
Ramon Ziai (rziai) wrote :

I should add that this change does not break any test, nor did I encounter other problems so far. Is there any other information I should provide?

Revision history for this message
Guillermo Gonzalez (verterok) wrote :

thanks!!!

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.

Subscribers

People subscribed via source and target branches