diff options
author | Alexander Kanavin <alex.kanavin@gmail.com> | 2023-07-27 16:36:11 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2023-07-30 09:16:52 +0100 |
commit | f6bd6b72f82bdc0aed4c08d2bbe41f1cd4bb4335 (patch) | |
tree | 38acbf96ca7474c690094f2d8486555c2add89ad /meta/classes-global | |
parent | ba961128e2d7d1b4c15daf749ea878efa5cf5da6 (diff) | |
download | poky-f6bd6b72f82bdc0aed4c08d2bbe41f1cd4bb4335.tar.gz |
insane.bbclass: add a RECIPE_MAINTAINER check (oe-core recipes only)
Absent maintainer entries are as well a frequent source of friction, as they are checked
only in selftest, and so aren't revealed until autobuilder runs.
The selftest is retained as it also checks for obsolete entries in maintainers.inc
(not possible to do in insane class).
(From OE-Core rev: 675dff0e37666c1cffa10a83f6f1f67d5fadd204)
Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes-global')
-rw-r--r-- | meta/classes-global/insane.bbclass | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index 78506c30b1..960dfc8924 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass | |||
@@ -50,7 +50,7 @@ ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch pkgconfig la \ | |||
50 | # Add usrmerge QA check based on distro feature | 50 | # Add usrmerge QA check based on distro feature |
51 | ERROR_QA:append = "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', ' usrmerge', '', d)}" | 51 | ERROR_QA:append = "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', ' usrmerge', '', d)}" |
52 | ERROR_QA:append:layer-core = " patch-status" | 52 | ERROR_QA:append:layer-core = " patch-status" |
53 | WARN_QA:append:layer-core = " missing-metadata" | 53 | WARN_QA:append:layer-core = " missing-metadata missing-maintainer" |
54 | 54 | ||
55 | FAKEROOT_QA = "host-user-contaminated" | 55 | FAKEROOT_QA = "host-user-contaminated" |
56 | FAKEROOT_QA[doc] = "QA tests which need to run under fakeroot. If any \ | 56 | FAKEROOT_QA[doc] = "QA tests which need to run under fakeroot. If any \ |
@@ -1485,7 +1485,16 @@ python do_recipe_qa() { | |||
1485 | else: | 1485 | else: |
1486 | oe.qa.handle_error("missing-metadata", "Recipe {} in {} does not contain a HOMEPAGE. Please add an entry.".format(pn, fn), d) | 1486 | oe.qa.handle_error("missing-metadata", "Recipe {} in {} does not contain a HOMEPAGE. Please add an entry.".format(pn, fn), d) |
1487 | 1487 | ||
1488 | def test_missing_maintainer(d): | ||
1489 | fn = d.getVar("FILE") | ||
1490 | pn = d.getVar("PN") | ||
1491 | if pn.endswith("-native") or pn.startswith("nativesdk-") or "packagegroup-" in pn or "core-image-ptest-" in pn: | ||
1492 | return | ||
1493 | if not d.getVar('RECIPE_MAINTAINER'): | ||
1494 | oe.qa.handle_error("missing-maintainer", "Recipe {} in {} does not have an assigned maintainer. Please add an entry into meta/conf/distro/include/maintainers.inc.".format(pn, fn), d) | ||
1495 | |||
1488 | test_missing_metadata(d) | 1496 | test_missing_metadata(d) |
1497 | test_missing_maintainer(d) | ||
1489 | oe.qa.exit_if_errors(d) | 1498 | oe.qa.exit_if_errors(d) |
1490 | } | 1499 | } |
1491 | 1500 | ||