All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nico Williams <nico@cryptonector.com>
To: Alejandro Colomar <alx@kernel.org>
Cc: Junio C Hamano <gitster@pobox.com>,
	git@vger.kernel.org, Ben Boeckel <mathstuf@gmail.com>,
	Viktor Dukhovni <viktor@openssl.org>
Subject: Re: [RFC] git-brebase
Date: Sat, 3 Oct 2026 16:17:22 -0500	[thread overview]
Message-ID: <asFw4gsn253MQtxp@ubby> (raw)
In-Reply-To: <asFtLJDJliQBPe1c@debian>

On Sat, Oct 03, 2026 at 11:07:04PM +0200, Alejandro Colomar wrote:
> Since --abort doesn't go all the way back, I think --continue shouldn't
> continue all the way forward, for consistency.

Fair.

> If that's desired behavior, it should go in this tool, and not as part
> of git-rebase(1).  git-rebase(1) is a much simpler and much more
> fundamental tool, which is used to build this more complex tool.
> That's one reason I'm rather opposed to having this as part of
> git-rebase(1); it would confuse about the responsibility of
> git-rebase(1).  IMO, git-rebase(1) is a plumbing command, and
> git-brebase would be a porcelain thingy.
> 
> > If we take this approach then we'd need an option not to enable this
> > behavior but to disable it, something like `--direct`.
> 
> That hints it might be just be a different command.

If this was 2007, and you were writing the first version of `rebase`,
and you had already worked out that you wanted this feature, what would
you do then?  Would you make it the default?  I _think_ I would.

Basically, this makes rebasing much nicer, so why not make it the
default?

Nico
-- 

  reply	other threads:[~2026-10-03 21:17 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-03 20:11 [RFC] git-brebase Alejandro Colomar
2026-10-03 20:13 ` Alejandro Colomar
2026-10-03 20:39 ` Nico Williams
2026-10-03 20:48   ` Nico Williams
2026-10-03 21:07     ` Alejandro Colomar
2026-10-03 21:17       ` Nico Williams [this message]
2026-10-03 21:29         ` Alejandro Colomar
2026-10-03 21:50           ` Nico Williams
2026-10-03 20:56   ` Alejandro Colomar
2026-10-03 21:13     ` Nico Williams
2026-10-03 21:38       ` Alejandro Colomar
2026-10-03 22:19         ` Nico Williams
2026-10-03 21:17     ` Alejandro Colomar
2026-10-08 22:18 ` [RFC v3] git-bisect-rebase Alejandro Colomar

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=asFw4gsn253MQtxp@ubby \
    --to=nico@cryptonector.com \
    --cc=alx@kernel.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=mathstuf@gmail.com \
    --cc=viktor@openssl.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.