Comment 2 for bug 62249

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 62249] Re: [Bug 62249] merge from bundles should support io redirection

On Mon, 2006-09-25 at 05:56 +0000, Martin Pool wrote:
> Are you saying that the default should be to merge from stdin?
>
> I don't think merging from the remembered location is very important:
> it's probably less commonly useful than for push or pull. So I don't
> mind removing it.
>
> However, commands that default to reading from stdin can be a bit
> confusing: if the user just types 'bzr merge' it will apparently just
> hang, while trying to read a patch. I suppose we could print 'reading
> bundle from stdin'.
>
> I'd prefer:
>
> * merge requires an argument
> * 'merge -' reads from stdin -- I thought this worked now, but
> apparently not

That would work. I was actually thinking that if stdin has content, it
should be sniffed for a bundle.

Rob
--
GPG key available at: <http://www.robertcollins.net/keys.txt>.