bzr pull lp:upstart fails

On Fri, 2008-01-11 at 07:17 +0000, Martin Pool wrote:
> Public bug reported:
>
> bzr pull lp:upstart fails, complaining lp:upstart is not a directory.
> That url works for other commands, like log.
>
> Probably a problem in the command class.
>

The issue is with _get_mergeable_helper, it does

    url = urlutils.normalize_url(location)
    url, filename = urlutils.split(url, exclude_trailing_slash=False)
    location_transport = transport.get_transport(url)

which leads to location_transport pointing to /whatever/lp:upstart

normalize_url returns 'file:///whatever/lp%3Aupstart' as it does not
recognise lp: as a scheme.

bzr pull lp:///upstart will work.

Thanks,

James

The bug is still there in 1.2rc1, and worse, the workaround doesn't work anymore:

$ bzr pull lp:///bzr-dbus
lp:///bzr-dbus is redirected to bzr+ssh://<email address hidden>/~bzr/bzr-dbus/trunk
bzr: ERROR: exceptions.NotImplementedError: <bound method LaunchpadTransport.clone of <bzrlib.plugins.launchpad.lp_indirect.LaunchpadTransport url=lp:///>>

Bug still exists in 1.3 ... and workaround fails now with just another error message

$ bzr pull lp:///portato
bzr: ERROR: Invalid url supplied to transport: "": You must specify a product.