instructions for cloning branch incorrect

Bug #654744 reported by Jelmer Vernooij
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
loggerhead
Triaged
Medium
Unassigned
loggerhead-breezy
Fix Released
Medium
Unassigned
loggerhead (Debian)
Fix Released
Unknown

Bug Description

From Debian bug 595986:

When viewing a branch in loggerhead on bzr.debian.org, the instructions to get
the branch give a bad location. For instance, viewing
<http://bzr.debian.org/loggerhead/pkg-bazaar/loggerhead/unstable/files> you
see:

To get this branch, use:
bzr branch /scm/loggerhead/pkg-bazaar/loggerhead/unstable

That will obviously not work.

Related branches

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

We should be displaying the public_branch if it set. This can still be wrong, but there isn't much we can do about that.

Changed in loggerhead:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

apps/branch.py makes the assumption that if a branch is accessed locally by loggerhead that the user will then also have local access to it.

Changed in loggerhead:
assignee: nobody → Jelmer Vernooij (jelmer)
Changed in loggerhead (Debian):
status: Unknown → Confirmed
Jelmer Vernooij (jelmer)
Changed in loggerhead:
assignee: Jelmer Vernooij (jelmer) → nobody
Revision history for this message
Kevin Jadin (marcspitz) wrote :

A fix is waiting inside the branch I linked to this bug report.

It worked for me, see my comment on the merge proposal :
https://code.launchpad.net/~j/loggerhead/loggerhead-add_url-prefix_to_serve-branches/+merge/20575

Can someone please clean the diff so it finally can be merged?

Changed in loggerhead:
status: Triaged → Fix Committed
Revision history for this message
Serrano Pereira (serrano-pereira) wrote :

When will this fix be released?

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

There isn't a working fix for this (there is an attached branch but it's waiting for updates from the author).

Changed in loggerhead:
status: Fix Committed → Triaged
Jelmer Vernooij (jelmer)
Changed in loggerhead-breezy:
status: New → Fix Released
importance: Undecided → Medium
Changed in loggerhead (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.