diff options
author | Paul Eggleton <bluelightning@bluelightning.org> | 2023-04-22 15:19:45 +1200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2023-04-22 10:59:12 +0100 |
commit | 23d946b9ba9b2a9989849e5714c8eddfe9c045ce (patch) | |
tree | 28d7c577a88e82b4120c1b59adcc06fd491cd64c | |
parent | 1b9bcc7b19f795af1a7c4d571f9eeaadd8440ad5 (diff) | |
download | poky-23d946b9ba9b2a9989849e5714c8eddfe9c045ce.tar.gz |
ref-manual: document new patch-status-* QA checks
Newly added in 4.2.
(From yocto-docs rev: 12e576fb6a932c4089651371c71c61fd1e96cd48)
Signed-off-by: Paul Eggleton <bluelightning@bluelightning.org>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | documentation/ref-manual/classes.rst | 7 | ||||
-rw-r--r-- | documentation/ref-manual/qa-checks.rst | 23 |
2 files changed, 30 insertions, 0 deletions
diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst index aacbb8a495..0762d1d8bc 100644 --- a/documentation/ref-manual/classes.rst +++ b/documentation/ref-manual/classes.rst | |||
@@ -1336,6 +1336,13 @@ Here are the tests you can list with the :term:`WARN_QA` and | |||
1336 | - ``patch-fuzz:`` Checks for fuzz in patch files that may allow | 1336 | - ``patch-fuzz:`` Checks for fuzz in patch files that may allow |
1337 | them to apply incorrectly if the underlying code changes. | 1337 | them to apply incorrectly if the underlying code changes. |
1338 | 1338 | ||
1339 | - ``patch-status-core:`` Checks that the Upstream-Status is specified | ||
1340 | and valid in the headers of patches for recipes in the OE-Core layer. | ||
1341 | |||
1342 | - ``patch-status-noncore:`` Checks that the Upstream-Status is specified | ||
1343 | and valid in the headers of patches for recipes in layers other than | ||
1344 | OE-Core. | ||
1345 | |||
1339 | - ``perllocalpod:`` Checks for ``perllocal.pod`` being erroneously | 1346 | - ``perllocalpod:`` Checks for ``perllocal.pod`` being erroneously |
1340 | installed and packaged by a recipe. | 1347 | installed and packaged by a recipe. |
1341 | 1348 | ||
diff --git a/documentation/ref-manual/qa-checks.rst b/documentation/ref-manual/qa-checks.rst index 13096816d2..6fdb0fbde9 100644 --- a/documentation/ref-manual/qa-checks.rst +++ b/documentation/ref-manual/qa-checks.rst | |||
@@ -750,6 +750,29 @@ Errors and Warnings | |||
750 | other things in the patches, those can be discarded. | 750 | other things in the patches, those can be discarded. |
751 | 751 | ||
752 | 752 | ||
753 | .. _qa-check-patch-status: | ||
754 | |||
755 | - ``Missing Upstream-Status in patch <patchfile> Please add according to <url> [patch-status-core/patch-status-noncore]`` | ||
756 | |||
757 | The Upstream-Status value is missing in the specified patch file's header. | ||
758 | This value is intended to track whether or not the patch has been sent | ||
759 | upstream, whether or not it has been merged, etc. | ||
760 | |||
761 | There are two options for this same check - ``patch-status-core`` (for | ||
762 | recipes in OE-Core) and ``patch-status-noncore`` (for recipes in any other | ||
763 | layer). | ||
764 | |||
765 | For more information on setting Upstream-Status see: | ||
766 | https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines#Patch_Header_Recommendations:_Upstream-Status | ||
767 | |||
768 | |||
769 | - ``Malformed Upstream-Status in patch <patchfile> Please correct according to <url> [patch-status-core/patch-status-noncore]`` | ||
770 | |||
771 | The Upstream-Status value in the specified patch file's header is invalid - | ||
772 | it must be a specific format. See the "Missing Upstream-Status" entry above | ||
773 | for more information. | ||
774 | |||
775 | |||
753 | .. _qa-check-buildpaths: | 776 | .. _qa-check-buildpaths: |
754 | 777 | ||
755 | - ``File <filename> in package <packagename> contains reference to TMPDIR [buildpaths]`` | 778 | - ``File <filename> in package <packagename> contains reference to TMPDIR [buildpaths]`` |