From: "Catalin Marinas" <catalin.marinas@gmail.com>
To: "Yann Dirson" <ydirson@altern.org>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Add new 'rebase' command.
Date: Sun, 21 Jan 2007 23:37:14 +0000	[thread overview]
Message-ID: <b0943d9e0701211537oac61eceq48d45501193e6790@mail.gmail.com> (raw)
In-Reply-To: <20070121232618.GD4665@nan92-1-81-57-214-146.fbx.proxad.net>
On 21/01/07, Yann Dirson <ydirson@altern.org> 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
next prev parent reply	other threads:[~2007-01-21 23:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-20 18:04 [PATCH] Add new 'rebase' command Yann Dirson
2007-01-21 23:00 ` Catalin Marinas
2007-01-21 23:26   ` Yann Dirson
2007-01-21 23:37     ` Catalin Marinas [this message]
  -- strict thread matches above, loose matches on Subject: below --
2007-01-20 15:46 Yann Dirson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox
  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):
  git send-email \
    --in-reply-to=b0943d9e0701211537oac61eceq48d45501193e6790@mail.gmail.com \
    --to=catalin.marinas@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=ydirson@altern.org \
    /path/to/YOUR_REPLY
  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
  Be sure your reply has a Subject: header at the top and a blank line
  before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).