From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Thomas Rast <trast@student.ethz.ch>
Cc: git@vger.kernel.org,
"Stephen Haberman" <stephen@exigencecorp.com>,
spearce@spearce.org, "Björn Steinbrink" <B.Steinbrink@gmx.de>
Subject: Re: Heads up: major rebase -i -p rework coming up
Date: Sun, 25 Jan 2009 03:05:38 +0100 (CET) [thread overview]
Message-ID: <alpine.DEB.1.00.0901250303150.14855@racer> (raw)
In-Reply-To: <200901242347.23187.trast@student.ethz.ch>
Hi,
On Sat, 24 Jan 2009, Thomas Rast wrote:
> Johannes Schindelin wrote:
> > Worse, the whole concept of "pick <merge-sha1>" just does not fly well.
> [...]
> > - merge $sha1 [$sha1...] was $sha1 "Merge ..."
> >
> > will merge the given list of commits into the current HEAD, for
> > the user's reference and to keep up-to-date what was rewritten, the
> > original merge is shown after the keyword "was" (which is not a valid
> > SHA-1, luckily)
>
> I really like the underlying idea. I'm not even sure if the current
> semantics are well-defined in all cases; an explicit merge command at
> least makes it very clear what is going on.
>
> However, I think the syntax as proposed above is a bit confusing in the
> usual two-parent merge. I couldn't tell whether
>
> merge A was B
>
> was intended to be read as "the merge of A into the current branch" or
> "the merge with sha1 A" right away, and I doubt I'll be able to tell
> without looking in the (rare) cases I have to invoke rebase -i -p.
>
> I can't really come up with a better replacement for 'was', so how about
>
> merge A # was B "Merge..."
>
> which would make it more clear that the "was B..." has no effect
> whatsoever on the merge's semantics.
Hmm. You're right, that is not really intuitive. How about
merge (B) A # Merge...
instead?
> > A - B - - - E
> > \ /
> > C - D
> >
> > could yield this TODO script:
> >
> > pick A
> > pick C
> > pick D
> > goto A'
> > pick B
> > merge D' was E
>
> I kind of wonder if it would be possible to decorate the TODO with
> 'git log --graph' output, to make it easier to follow the history as
> it is built.
I wondered about that, too, and abandoned it as my common operation is cut
& past lines around. The result would look _utterly_ confusing.
Maybe I should have mentioned that to spare you the brain cycles thinking
about --graph...
Ciao,
Dscho
next prev parent reply other threads:[~2009-01-25 2:07 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-24 20:25 Heads up: major rebase -i -p rework coming up Johannes Schindelin
2009-01-24 20:37 ` Junio C Hamano
2009-01-24 21:04 ` Johannes Schindelin
2009-01-24 21:09 ` Johannes Schindelin
2009-01-24 23:01 ` Junio C Hamano
2009-01-25 2:23 ` Johannes Schindelin
2009-01-26 16:21 ` Marc Branchaud
2009-01-24 22:47 ` Thomas Rast
2009-01-25 2:05 ` Johannes Schindelin [this message]
2009-01-25 2:25 ` Johannes Schindelin
2009-01-25 13:46 ` Jakub Narebski
2009-01-25 14:17 ` Johannes Schindelin
2009-01-25 15:07 ` Sverre Rabbelier
2009-01-25 15:24 ` Johannes Schindelin
2009-01-25 20:35 ` Junio C Hamano
2009-01-25 20:59 ` Johannes Schindelin
2009-01-25 22:03 ` Jakub Narebski
2009-01-25 23:29 ` Johannes Schindelin
2009-02-03 10:05 ` Nanako Shiraishi
2009-02-03 11:47 ` Johannes Schindelin
2009-01-25 16:22 ` Jakub Narebski
2009-01-25 17:18 ` Björn Steinbrink
2009-01-26 16:10 ` Marc Branchaud
2009-01-27 15:21 ` Stephen Haberman
2009-01-27 18:08 ` Johannes Schindelin
2009-01-27 22:10 ` Nanako Shiraishi
2009-01-27 22:36 ` Stephen Haberman
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=alpine.DEB.1.00.0901250303150.14855@racer \
--to=johannes.schindelin@gmx.de \
--cc=B.Steinbrink@gmx.de \
--cc=git@vger.kernel.org \
--cc=spearce@spearce.org \
--cc=stephen@exigencecorp.com \
--cc=trast@student.ethz.ch \
/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