diff mbox series

[2/3] ref-manual/classes.rst: drop obsolete QA errors

Message ID 20250723-refactor-qa-checks-v1-2-3a72c751ba83@bootlin.com
State Superseded
Headers show
Series ref-manual: gather all QA checks in qa-checks.rst | expand

Commit Message

Antonin Godard July 23, 2025, 2:27 p.m. UTC 
Drop after commit 068d38214307 ("insane: remove obsolete QA errors")
from OE-Core.

Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
---
 documentation/ref-manual/classes.rst | 13 -------------
 1 file changed, 13 deletions(-)

Comments

Quentin Schulz Aug. 1, 2025, 10:49 a.m. UTC | #1 
Hi Antonin,

On 7/23/25 4:27 PM, Antonin Godard via lists.yoctoproject.org wrote:
> Drop after commit 068d38214307 ("insane: remove obsolete QA errors")
> from OE-Core.
> 

Actually, they were removed in bcc03ea19e103f6aa93bada2f49fcc5cc7bc0790 
for split-strip and a67e9ebfd5b8002fd4a7d8d27ff0d997817f76e1 for the 
other two accordingo the commit log.

The former goes back to Thud and the latter to Hardknott. Considering we 
aren't supporting any version before Kirkstone, it's fine to keep both 
changes in one commit :)

However, this should be backported to kirkstone and later still 
supported branches!

Maybe fix the commit log so we aren't confused by a backport to branches 
which do not contain this fix.

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin
diff mbox series

Patch

diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
index f22af7b04..5f7b7c5ce 100644
--- a/documentation/ref-manual/classes.rst
+++ b/documentation/ref-manual/classes.rst
@@ -1390,11 +1390,6 @@  The tests you can list with the :term:`WARN_QA` and
    directive, that the recipe also inherits the :ref:`ref-classes-gettext`
    class to ensure that gettext is available during the build.
 
--  ``compile-host-path:`` Checks the
-   :ref:`ref-tasks-compile` log for indications that
-   paths to locations on the build host were used. Using such paths
-   might result in host contamination of the build output.
-
 -  ``debug-deps:`` Checks that all packages except ``-dbg`` packages
    do not depend on ``-dbg`` packages, which would cause a packaging
    bug.
@@ -1462,11 +1457,6 @@  The tests you can list with the :term:`WARN_QA` and
    being created due to being marked with a license that is in
    :term:`INCOMPATIBLE_LICENSE`.
 
--  ``install-host-path:`` Checks the
-   :ref:`ref-tasks-install` log for indications that
-   paths to locations on the build host were used. Using such paths
-   might result in host contamination of the build output.
-
 -  ``installed-vs-shipped:`` Reports when files have been installed
    within :ref:`ref-tasks-install` but have not been included in any package by
    way of the :term:`FILES` variable. Files that do not
@@ -1599,9 +1589,6 @@  The tests you can list with the :term:`WARN_QA` and
    in a packaged script is not longer than 128 characters, which can cause
    an error at runtime depending on the operating system.
 
--  ``split-strip:`` Reports that splitting or stripping debug symbols
-   from binaries has failed.
-
 -  ``staticdev:`` Checks for static library files (``*.a``) in
    non-``staticdev`` packages.