From: Roland Hieber <rhi@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Roland Hieber <rhi@pengutronix.de>
Subject: [ptxdist] [PATCH] [RFC] doc/contributing: require Upstream-Status: headers for new patches
Date: Wed, 5 Feb 2025 13:09:22 +0100 [thread overview]
Message-ID: <20250205120922.2379021-1-rhi@pengutronix.de> (raw)
The amount of patches in the patches/ folder in PTXdist has grown over
the years, and for most of them it is hard to track whether they have
already been applied upstream, or denied by the maintainer, or if they
are otherwise inappropriate for upstream submission. To simplify the
migration process to a new package version, any patches should include
appropriate documentation about their relation to the upstream source.
the Yocto Project already has a syntax for that; let's re-use that for
PTXdist too. This way patches can also easily be cherry-picked between
the two projects.
Signed-off-by: Roland Hieber <rhi@pengutronix.de>
---
doc/contributing.rst | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/doc/contributing.rst b/doc/contributing.rst
index 63da67802222..1b0b0804b6ce 100644
--- a/doc/contributing.rst
+++ b/doc/contributing.rst
@@ -145,6 +145,16 @@ in mind:
``_LICENSE`` variable if necessary.
Often the difference is only in the copyright year, but in any case, describe
the changes in the license file when sending your patch!
+- When adding new patches to a package (in ``patches/``), these patches must
+ carry ``Upstream-Status:`` pseudo-headers to track the state of the patches
+ and give more context for the change in relation to the upstream source.
+ Refer to `Yocto Project's documentation <patch-upstream-status_>`_ regarding
+ the syntax of these pseudo-headers. (Note: instead of the ``oe specific``
+ keyword, use ``ptxdist specific``.)
+
+ Adding such headers for existing patches is good practise, but not required.
+
+.. _patch-upstream-status: https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#patch-upstream-status
Misc
~~~~
--
2.39.5
reply other threads:[~2025-02-05 12:09 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20250205120922.2379021-1-rhi@pengutronix.de \
--to=rhi@pengutronix.de \
--cc=ptxdist@pengutronix.de \
/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