public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: "Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail•com>
To: "Junio C Hamano" <gitster@pobox•com>
Cc: "Mathias Rav" <m@git•strova.dk>,
	"Phillip Wood" <phillip.wood@dunelm•org.uk>,
	"John Cai" <johncai86@gmail•com>,
	"Ævar Arnfjörð Bjarmason" <avarab@gmail•com>,
	"brian m. carlson" <sandals@crustytoothpaste•net>,
	"Patrick Steinhardt" <ps@pks•im>,
	git@vger•kernel.org, "Karthik Nayak" <karthik.188@gmail•com>
Subject: Re: [PATCH v2] merge-file: fix BUG when --object-id is used in a worktree
Date: Wed, 11 Mar 2026 21:16:13 +0100	[thread overview]
Message-ID: <4b2a192e-240a-45d2-92a5-7ab669f481a9@app.fastmail.com> (raw)
In-Reply-To: <xmqq5x72qnu6.fsf@gitster.g>

On Wed, Mar 11, 2026, at 18:26, Junio C Hamano wrote:
>>>[snip]
>>
>> Turns out that this has been attempted least once before:
>>
>> https://lore.kernel.org/git/1472230741-5161-1-git-send-email-dev+git@drbeat.li/
>
> True.
>
> Perhaps something like this patch makes it clear that what the gitk
> command gives and what --pretty=reference gives are not identical,
> to avoid confusion like this?  I dunno.
>
>
>  Documentation/SubmittingPatches | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git c/Documentation/SubmittingPatches w/Documentation/SubmittingPatches
> index e270ccbe85..fad0b41af0 100644
> --- c/Documentation/SubmittingPatches
> +++ w/Documentation/SubmittingPatches
> @@ -335,7 +335,7 @@ date)", like this:
>  ....
>
>  The "Copy commit reference" command of gitk can be used to obtain this
> -format (with the subject enclosed in a pair of double-quotes), or this

And now I see for the first time that the doc points out the difference
already... I’m really paying attention it turns out.

> +format (but with the subject enclosed in an extra pair of double-quotes), or this

I think replacing “with” with “but” is good. But why “extra” pairs? It’s
just a pair of double quotes.

>  invocation of `git show`:
>
>  ....

  reply	other threads:[~2026-03-11 20:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-10 11:46 [PATCH] merge-file: fix BUG when --object-id is used in a worktree Mathias Rav
2026-03-10 12:35 ` Karthik Nayak
2026-03-10 12:49 ` Patrick Steinhardt
2026-03-10 20:01   ` Junio C Hamano
2026-03-11  6:44     ` [PATCH v2] " Mathias Rav
2026-03-11  7:18       ` Kristoffer Haugsbakk
2026-03-11 11:14       ` Kristoffer Haugsbakk
2026-03-11 17:26         ` Junio C Hamano
2026-03-11 20:16           ` Kristoffer Haugsbakk [this message]
2026-03-18 19:40             ` Junio C Hamano
2026-03-18 19:16       ` Mathias Rav
2026-03-18 19:45         ` Junio C Hamano
2026-03-10 13:34 ` [PATCH] " Kristoffer Haugsbakk

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=4b2a192e-240a-45d2-92a5-7ab669f481a9@app.fastmail.com \
    --to=kristofferhaugsbakk@fastmail$(echo .)com \
    --cc=avarab@gmail$(echo .)com \
    --cc=git@vger$(echo .)kernel.org \
    --cc=gitster@pobox$(echo .)com \
    --cc=johncai86@gmail$(echo .)com \
    --cc=karthik.188@gmail$(echo .)com \
    --cc=m@git$(echo .)strova.dk \
    --cc=phillip.wood@dunelm$(echo .)org.uk \
    --cc=ps@pks$(echo .)im \
    --cc=sandals@crustytoothpaste$(echo .)net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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