From: Martin Habets <habetsm.xilinx@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: davem@davemloft.net, netdev@vger.kernel.org, edumazet@google.com,
pabeni@redhat.com
Subject: Re: [PATCH net-next] docs: netdev: recommend against --in-reply-to
Date: Thu, 24 Aug 2023 10:08:54 +0100 [thread overview]
Message-ID: <20230824090854.GA464302@gmail.com> (raw)
In-Reply-To: <20230823154922.1162644-1-kuba@kernel.org>
On Wed, Aug 23, 2023 at 08:49:22AM -0700, Jakub Kicinski wrote:
> It's somewhat unfortunate but with (my?) the current tooling
> if people post new versions of a set in reply to an old version
> managing the review queue gets difficult. So recommend against it.
Is this something NIPA could catch?
>
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Reviewed-by: Martin Habets <habetsm.xilinx@gmail.com>
> ---
> Documentation/process/maintainer-netdev.rst | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/process/maintainer-netdev.rst b/Documentation/process/maintainer-netdev.rst
> index 2ab843cde830..c1c732e9748b 100644
> --- a/Documentation/process/maintainer-netdev.rst
> +++ b/Documentation/process/maintainer-netdev.rst
> @@ -167,6 +167,8 @@ Asking the maintainer for status updates on your
> patch is a good way to ensure your patch is ignored or pushed to the
> bottom of the priority list.
>
> +.. _Changes requested:
> +
> Changes requested
> ~~~~~~~~~~~~~~~~~
>
> @@ -359,6 +361,10 @@ Make sure you address all the feedback in your new posting. Do not post a new
> version of the code if the discussion about the previous version is still
> ongoing, unless directly instructed by a reviewer.
>
> +The new version of patches should be posted as a separate thread,
> +not as a reply to the previous posting. Change log should include a link
> +to the previous posting (see :ref:`Changes requested`).
> +
> Testing
> -------
>
> --
> 2.41.0
>
next prev parent reply other threads:[~2023-08-24 9:08 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-23 15:49 [PATCH net-next] docs: netdev: recommend against --in-reply-to Jakub Kicinski
2023-08-23 20:38 ` [Intel-wired-lan] " Przemek Kitszel
2023-08-23 20:38 ` Przemek Kitszel
2023-08-24 9:08 ` Martin Habets [this message]
2023-08-24 15:29 ` Jakub Kicinski
2023-10-30 21:10 ` Jacob Keller
2023-08-24 18:30 ` patchwork-bot+netdevbpf
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=20230824090854.GA464302@gmail.com \
--to=habetsm.xilinx@gmail.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
/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.