Bazaar stores only expanded lp: branch URL

Bug #511817 reported by Russel Winder
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Low
Unassigned

Bug Description

When a user (i.e. me :-) issues a bzr command that uses an lp: URL, Bazaar stores the expanded version of this in .bzr/branch/branch.conf. This means that the lp: URL is stored in its interpreted at the time form. If the binding of the lp: URL is changed on Launchpad the user has no knowledge of this. If Bazaar stored the lp: URL directly then interpretation would occur at the time of command issuing which means the current binding at Launchpad would be used.

Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: [Bug 511817] [NEW] Bazaar stores expanded lp: branch URL

  status confirmed
  importance low

Changed in bzr:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Matthew Fuller (fullermd) wrote :

> If Bazaar stored the lp: URL directly then interpretation would
> occur at the time of command issuing which means the current binding
> at Launchpad would be used.

On the other hand, you'd eat the directory lookup every time too.

Revision history for this message
Russel Winder (russel) wrote : Re: Bazaar stores expanded lp: branch URL

Not sure what is meant by "eat the directory lookup".

Revision history for this message
Matt Nordhoff (mnordhoff) wrote :

I imagine he just means that you'd have to do the directory lookup every time you pull, push, etc. Not a big deal, but there is a cost. (Especially if you're on a high-latency connection.)

Revision history for this message
Matt Nordhoff (mnordhoff) wrote :

(By "you'd have to" I just mean bzr...)

Revision history for this message
Russel Winder (russel) wrote :

I agree there would likely be some overhead but until experiments are done we have no scale, i.e. no idea if it is 0.005% or 50%. I am not sure what my point of pain would be, a 5% overhead would likely be fine. All in all, I would like the option of having a static binding or a dynamic binding of the client to the server.

(I have been bitten too often now by knowledge of changed lp: URLs not being promulgated in any way around the system.)

Revision history for this message
Jonathan Lange (jml) wrote :

I've had corridor conversations about this in the past.

In my ideal world, Bazaar would store both what you asked for (e.g. lp:launchpadlib) and what you got (e.g. bzr+ssh://bazaar.launchpad.net/~launchpad-pqm/launchpadlib/trunk). This way, users would be able to make an informed decision if ever the expansion of the directory URL changed.

To me, this is a bug in all directory services, not just Launchpad's.

summary: - Bazaar stores expanded lp: branch URL
+ Bazaar stores only expanded lp: branch URL
Revision history for this message
John A Meinel (jameinel) wrote : Re: [Bug 511817] Re: Bazaar stores expanded lp: branch URL

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Russel Winder wrote:
> I agree there would likely be some overhead but until experiments are
> done we have no scale, i.e. no idea if it is 0.005% or 50%. I am not
> sure what my point of pain would be, a 5% overhead would likely be fine.
> All in all, I would like the option of having a static binding or a
> dynamic binding of the client to the server.
>
> (I have been bitten too often now by knowledge of changed lp: URLs not
> being promulgated in any way around the system.)
>

You can do:

echo "parent_location = lp:foo" >> .bzr/branch/branch.conf

And have it preserved (until you do --remember, etc).

As for the default... If a project changes their development focus, but
you are tracking the specific branch, the opposite would be desired.

I would guess that always resolving lp:foo would be what most people
would expect, though.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkte9vEACgkQJdeBCYSNAAP2BQCgzIJsICYqu0lPzzhNcH5fNC/N
cysAnj8ZJp6E+ke96pvy0QnCY3NK4UQV
=pDQj
-----END PGP SIGNATURE-----

Martin Pool (mbp)
tags: added: lpurl
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
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.