From: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
To: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Cc: workflows@vger.kernel.org,
Johannes Berg <johannes@sipsolutions.net>,
Aditya Kumar Singh <aditya.kumar.singh@oss.qualcomm.com>
Subject: Re: Depends-on tag
Date: Wed, 28 May 2025 11:22:27 -0400 [thread overview]
Message-ID: <20250528-sparkling-expert-taipan-5ad429@meerkat> (raw)
In-Reply-To: <67f86f2e-d3ac-4fbe-87d6-186487de5721@oss.qualcomm.com>
On Wed, May 28, 2025 at 07:19:55AM -0700, Jeff Johnson wrote:
> patchwork itself very recently formalized the usage of a 'Depends-on:' tag for
> flagging patchset dependencies:
>
> https://lists.ozlabs.org/pipermail/patchwork/2025-January/007465.html
> https://lists.ozlabs.org/pipermail/patchwork/2025-March/007490.html
> https://github.com/getpatchwork/patchwork/commit/bc2d1b1dcf89c619e027ae4dfd3036cba30e5583
> https://patchwork.ozlabs.org/project/patchwork/list/?series=442332&state=*
>
> Note that this support is only in mainline and not part of any release yet.
>
> Curious if the Linux Kernel community will adopt this approach for flagging
> dependencies.
We've been supporting something like this in b4 for a while now, but we use
prerequisite-* trailers (not tags) and always expand that into
prerequisite-patch-id, since that's the only format documented by git itself.
E.g. here is an example of a series using prerequisite-change-id:
https://lore.kernel.org/b4-sent/20250526-sip_svc_upstream-v3-0-6a08a4502de3@altera.com/
Here's another one using prerequisite-message-id:
https://lore.kernel.org/b4-sent/20250514-ipq5018-pcie-v10-0-5b42a8eff7ea@outlook.com/
The documentation is here:
https://b4.docs.kernel.org/en/latest/contributor/prep.html#working-with-series-dependencies
I'm not sure "Depends-on:" as an actual commit tag will be accepted by the
kernel community, because it's information that is only useful during
pre-commit stages. However, if it's passed along as a cover letter/first patch
trailers, we can recognize it as an alternative for "prerequisite-message-id".
-K
prev parent reply other threads:[~2025-05-28 15:22 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-28 14:19 Jeff Johnson
2025-05-28 15:22 ` Konstantin Ryabitsev [this message]
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=20250528-sparkling-expert-taipan-5ad429@meerkat \
--to=konstantin@linuxfoundation.org \
--cc=aditya.kumar.singh@oss.qualcomm.com \
--cc=jeff.johnson@oss.qualcomm.com \
--cc=johannes@sipsolutions.net \
--cc=workflows@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox