From: Catalin Marinas Date: Sun, 21 Jan 2007 23:37:14 GMT Subject: Re: [PATCH] Add new 'rebase' command. Message-ID: In-Reply-To: <20070121232618.GD4665@nan92-1-81-57-214-146.fbx.proxad.net> On 21/01/07, Yann Dirson wrote: > On Sun, Jan 21, 2007 at 11:00:57PM +0000, Catalin Marinas wrote: > > I applied it but changed the patch a bit so that there is no need for > > the --to option. I understood your point but I find it a bit strange > > to have this mandatory option. Anyway, command line syntax doesn't > > have to follow the natural language exactly :-). > > Well, I thought the natural next patch to this command would be, > according to the discussions with Jakub, to make first "--to target" > optional, as a natural replacement for "stg pull . " when the > target is the parent branch, and then possibly add the ability of > running rebase from the target branch as also sugested by Jakub, > leading to the natural "rebase [|--to ]" syntax. > > Although I'm not sure we'll do that in the near future, removing the > need for --to would somewhat get in the way of this possibility. I'll go through the other e-mails tomorrow (too late here) and either add the option back or come with counter arguments :-) -- Catalin