Re: [PATCH] Add new 'rebase' command.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 21/01/07, Yann Dirson <ydirson@xxxxxxxxxx> 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 . <target>" 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 [<stack>|--to <target>]" 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
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]