launchpad-dev team mailing list archive

[Michael Hudson-Doyle <michael.hudson@xxxxxxxxxxxxx>]

On Wed, 27 Oct 2010 08:53:13 -0400, Jonathan Lange <jml@xxxxxxxxxxxxx> wrote:
> On Wed, Oct 27, 2010 at 8:46 AM, Gavin Panella
> <gavin.panella@xxxxxxxxxxxxx> wrote:
> > On 27 October 2010 13:17, Jonathan Lange <jml@xxxxxxxxxxxxx> wrote:
> >> On Wed, Oct 27, 2010 at 6:48 AM, Gavin Panella
> >> <gavin.panella@xxxxxxxxxxxxx> wrote:
> >>> {{{
> >>> $ bzr info lp:launchpad/stable
> >>> bzr: ERROR: Permission denied: "Cannot create 'stable'. Only Bazaar
> >>> branches are allowed."
> >>>
> >>
> >> That looks like a bug, in so far as we shouldn't be assuming "create"
> >> on a read-only request.
> >>
> >> We should instead be correctly saying that there is no branch called
> >> lp:launchpad/stable
> >
> > Do you think that's in bzr or lp code? Or shall I file a bug with
> > tasks for both and see which one sticks?
> >
> 
> Probably launchpad-code, but maybe it'll take bzr changes to fix it.

In this case, definitely launchpad-code.  Our path-translating transport
for all methods thunks through to a method that translates the path and
calls the method on the underlying transport.  We need to also pass
along whether the method is a read or write to the translation part and
it can then react differently to invalid paths.  Not hard, but a bunch
of typing.

Cheers,
mwh