Ubuntu
bzr package

merge from bundles should support io redirection

Bug #62249 reported by Robert Collins
2
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Wishlist
Unassigned
Breezy
Triaged
Wishlist
Unassigned
bzr (Ubuntu)
Triaged
Wishlist
Unassigned

Bug Description

I just tried to do:
 $ bzr merge < ~/email-improvements.patch
Merging from remembered
location /home/robertc/source/baz/plugins/email/trunk/
Nothing to do.

What I expected that to do is what patch < a-patch does- that is
 $ bzr merge ~/email-improvements.patch
All changes applied successfully.

I think this should be supported.
 affects /bzr

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

Tags: merge
Revision history for this message
Martin Pool (mbp) wrote : Re: [Bug 62249] merge from bundles should support io redirection

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

--
Martin

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>.

Revision history for this message
Paul Dufresne (paulduf) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. You reported this bug a while ago and there hasn't been any activity in it recently. We were wondering is this still an issue for you? Can you try with latest Ubuntu release? Thanks in advance.

Changed in bzr:
assignee: nobody → dufresnep
status: New → Incomplete
Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 62249] Re: merge from bundles should support io redirection

On Fri, 2007-11-23 at 16:35 +0000, Paul Dufresne wrote:
> Thank you for taking the time to report this bug and helping to make
> Ubuntu better. You reported this bug a while ago and there hasn't been
> any activity in it recently. We were wondering is this still an issue
> for you? Can you try with latest Ubuntu release? Thanks in advance.

This should have been forwarded upstream.

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

Revision history for this message
Paul Dufresne (paulduf) wrote :

Marking as 'In progress' (was incomplete ) as it have been reported 'Upstream'.

Changed in bzr:
status: Incomplete → In Progress
assignee: dufresnep → nobody
James Westby (james-w)
Changed in bzr:
importance: Undecided → Wishlist
status: New → Confirmed
James Westby (james-w)
Changed in bzr (Ubuntu):
status: In Progress → Triaged
importance: Undecided → Wishlist
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
Jelmer Vernooij (jelmer)
tags: added: merge
removed: check-for-breezy
Changed in brz:
status: New → Triaged
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.