git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Will Chandler <wfc@wfchandler.org>
Cc: Bagas Sanjaya <bagasdotme@gmail.com>,
	Konstantin Ryabitsev <konstantin@linuxfoundation.org>,
	Jeff King <peff@peff.net>,
	git@vger.kernel.org, ps@pks.im
Subject: Re: [PATCH] refs: cleanup directories when deleting packed ref
Date: Mon, 10 May 2021 10:15:49 +0900	[thread overview]
Message-ID: <xmqqzgx3s81m.fsf@gitster.g> (raw)
In-Reply-To: <YJgt2UzAdhFfrDHm@mini.wfchandler.org> (Will Chandler's message of "Sun, 9 May 2021 14:45:45 -0400")

Will Chandler <wfc@wfchandler.org> writes:

> Thank you, Junio. Would it be helpful if I sent a separate '[PATCH v2]'
> email?
>
> Apologies for not following the correct procedure for the revised patch.
> I was relying on 'SubmittingPatches', but I've since found the detailed
> instructions on submitting revisions in 'MyFirstContribution.txt'.

I am not sure if there is anything to apologize for on your part.
The procedures we use have grown over time, and there certainly
would be documentation gaps.

I think the best current practice is

1. In a thread that originates at a non-patch message, or in a
   review discussion thread for a patch, it is welcome to use "How
   about doing it this way?" patches as an illustration to explain
   your idea in a more concrete way than just in prose.  But it is
   unwelcome to leave the patch buried in the discussion, without
   making it easier to find it (see 3.)

2. In such a thread, "By the way, I thought of this unrelated
   tangent, and here is a patch to demonstrate the idea" is not
   entirely unwelcome, but keep it brief and make sure you get out
   of the thread quickly to avoid distracting the main discussion.

3. In either case, it makes it easier to find the final submission
   of the patch if it is not buried deep in the discussion.

   a. It is OK to start a new thread (without in-reply-to), with
      "here is a polished version of the patch and/or the idea I
      floated in <message-id>" under the three-dash line (for a
      single patch) or in the cover letter (for a series).

   b. An updated iteration of a multi-patch series sometimes is made
      as a direct response to the cover letter of the previous
      iteration (iow, the cover letter for vN+1 has the message id
      of the cover letter for vN on its in-reply-to header).

I think the "b4" tool prefers 3b. over 3a., and it may be also easy
on human readers who read the list in threaded mail/news reader.

Thanks.

  reply	other threads:[~2021-05-10  1:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-07 14:37 Will Chandler
2021-05-07 21:56 ` Jeff King
2021-05-07 22:02   ` Jeff King
2021-05-07 22:57     ` Jeff King
2021-05-08  4:27       ` Will Chandler
2021-05-08  5:00         ` Will Chandler
2021-05-08  5:21           ` Bagas Sanjaya
2021-05-08  6:24             ` Junio C Hamano
2021-05-09 18:45               ` Will Chandler
2021-05-10  1:15                 ` Junio C Hamano [this message]
2021-05-11  1:35           ` Jeff King
2021-05-11  4:58             ` Junio C Hamano

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=xmqqzgx3s81m.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=bagasdotme@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=konstantin@linuxfoundation.org \
    --cc=peff@peff.net \
    --cc=ps@pks.im \
    --cc=wfc@wfchandler.org \
    --subject='Re: [PATCH] refs: cleanup directories when deleting packed ref' \
    /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

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).