diff mbox series

[v3,14/16] ref-manual: add mission pep517-backend sanity check

Message ID 20241010-release-note-5-1-v3-14-2cef1a58f1ab@bootlin.com
State Accepted
Headers show
Series Updates for styhead | expand

Commit Message

Antonin Godard Oct. 10, 2024, 12:10 p.m. UTC
New sanity check to check for PEP517-backend compliance.

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

Comments

Quentin Schulz Oct. 14, 2024, 12:16 p.m. UTC | #1
Hi Antonin,

On 10/10/24 2:10 PM, Antonin Godard via lists.yoctoproject.org wrote:
> New sanity check to check for PEP517-backend compliance.
> 
> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> ---
>   documentation/ref-manual/classes.rst | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
> index cdfd1f7e3..a715be97d 100644
> --- a/documentation/ref-manual/classes.rst
> +++ b/documentation/ref-manual/classes.rst
> @@ -1478,6 +1478,9 @@ The tests you can list with the :term:`WARN_QA` and
>   
>   -  ``perms:`` Currently, this check is unused but reserved.
>   
> +-  ``pep517-backend:`` checks that a recipe inheriting
> +   :ref:`ref-classes-setuptools3` has a PEP517-compliant backend.
> +

Not sure we follow this for each entry in that list, but this one is not 
in its proper location if we were to alphabetically order the list.

Cheers,
Quentin
Antonin Godard Oct. 14, 2024, 1:46 p.m. UTC | #2
On 14/10/2024 14:16:07+0000, Quentin Schulz wrote:
> Hi Antonin,
>
> On 10/10/24 2:10 PM, Antonin Godard via lists.yoctoproject.org wrote:
> > New sanity check to check for PEP517-backend compliance.
> >
> > Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> > ---
> >   documentation/ref-manual/classes.rst | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
> > index cdfd1f7e3..a715be97d 100644
> > --- a/documentation/ref-manual/classes.rst
> > +++ b/documentation/ref-manual/classes.rst
> > @@ -1478,6 +1478,9 @@ The tests you can list with the :term:`WARN_QA` and
> >   -  ``perms:`` Currently, this check is unused but reserved.
> > +-  ``pep517-backend:`` checks that a recipe inheriting
> > +   :ref:`ref-classes-setuptools3` has a PEP517-compliant backend.
> > +
>
> Not sure we follow this for each entry in that list, but this one is not in
> its proper location if we were to alphabetically order the list.

Ha, you're right I got it wrong. Thanks, will fix.

Cheers,
Antonin

--
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
diff mbox series

Patch

diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
index cdfd1f7e3..a715be97d 100644
--- a/documentation/ref-manual/classes.rst
+++ b/documentation/ref-manual/classes.rst
@@ -1478,6 +1478,9 @@  The tests you can list with the :term:`WARN_QA` and
 
 -  ``perms:`` Currently, this check is unused but reserved.
 
+-  ``pep517-backend:`` checks that a recipe inheriting
+   :ref:`ref-classes-setuptools3` has a PEP517-compliant backend.
+
 -  ``pkgconfig:`` Checks ``.pc`` files for any
    :term:`TMPDIR`/:term:`WORKDIR` paths.
    Any ``.pc`` file containing these paths is incorrect since