[merge][#129701] better message when unable to commit to readonly branch

Martin Pool mbp at sourcefrog.net
Wed Oct 10 00:57:02 BST 2007


On 10/10/07, John Arbash Meinel <john at arbash-meinel.com> wrote:
> John Arbash Meinel has voted tweak.
> Status is now: Conditionally approved
> Comment:
> (submitted too soon)
>
> It might be nice to include a '\n' in there. Something like:
>    bzr: ERROR: Cannot lock
> LockDir(http://sourcefrog.net/bzr/.bzr/repository/lock):
> Transport operation not possible: http does not support mkdir()
>
> Since we know that the URL is going to be long.
>
> Also, it seems reasonable to do a more specific readonly check, rather
> than failing because "http does not support mkdir()".
> Or we could make the "mkdir" failure say something about being readonly.
>
> Otherwise, this is *much* better than before, even if it isn't perfect.
> (It would be nice to recommend 'bzr unbind' or 'commit --local')

I'd like to do that.  This particular patch is not at all specific to
commit, so I can't suggest what to do instead.  It probably would make
sense in the command layer to catch particular exceptions and add some
suggestions as to what to try.

-- 
Martin



More information about the bazaar mailing list