Warn users about getting readonly launchpad transports
Martin Pool
mbp at canonical.com
Fri Mar 14 01:40:08 GMT 2008
On 14/03/2008, Martin Albisetti <argentina at gmail.com> wrote:
> On Thu, Mar 13, 2008 at 8:16 PM, James Westby <jw+debian at jameswestby.net> wrote:
> > Hi,
> >
> > Does anyone think the attached patch is a good idea?
>
>
> It seems like a good step forward, although I'd personally try and
> send the warning only on write transports, as it might be annoying for
> the users who use that just to checkout/branch stuff.
I believe the problem is that at the code level we don't know whether
the user is going to write to it or not until after the redirection
has been done. (And indeed, they may be getting a checkout but later
wanting to write to it.)
> Anyhow, it's a step forward :)
+1, I agree, though as Aaron recently touched this redirector it could
be good to check with him too.
James, please put [merge] in your subject line (even if you're not
totally sure it should merge) so that BB will track it. I actually
think it should track all patches; the number of things that look like
patches but are not intended to merge is pretty negligible.
--
Martin
More information about the bazaar
mailing list