diff mbox series

python3-yamllint: Add recipe

Message ID 20240122200704.21018-1-reatmon@ti.com
State Accepted, archived
Commit 128cfc5222752a6337a9cbb9bc9023c13ee19f2f
Headers show
Series python3-yamllint: Add recipe | expand

Commit Message

Ryan Eatmon Jan. 22, 2024, 8:07 p.m. UTC
Add recipe for yamllint.  There is an upcoming change in u-boot where
the binman tool is now configured to call yamllint to verify the configs
during compile time.

There was a previous patch a year ago from Trevor Woerner that never
made it into oe-core.  This patch is a reworking of his patch but
pointing to a newer version.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../python/python3-yamllint_1.33.0.bb             | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
 create mode 100644 meta/recipes-devtools/python/python3-yamllint_1.33.0.bb

Comments

Yoann Congal Jan. 22, 2024, 10:30 p.m. UTC | #1
Hi,

Le 22/01/2024 à 21:07, Ryan Eatmon via lists.openembedded.org a écrit :
> Add recipe for yamllint.  There is an upcoming change in u-boot where
> the binman tool is now configured to call yamllint to verify the configs
> during compile time.
> 
> There was a previous patch a year ago from Trevor Woerner that never
> made it into oe-core.  This patch is a reworking of his patch but
> pointing to a newer version.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../python/python3-yamllint_1.33.0.bb             | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>  create mode 100644 meta/recipes-devtools/python/python3-yamllint_1.33.0.bb

FYI, this new recipe lacks a maintainer and throw this warning :
WARNING: python3-yamllint-1.33.0-r0 do_recipe_qa: QA Issue: Recipe python3-yamllint in [...]/poky/meta/recipes-devtools/python/python3-yamllint_1.33.0.bb does not have an assigned maintainer. Please add an entry into meta/conf/distro/include/maintainers.inc. [missing-maintainer]

Regards,
Ryan Eatmon Jan. 22, 2024, 11:36 p.m. UTC | #2
On 1/22/2024 4:30 PM, Yoann Congal wrote:
> Hi,
> 
> Le 22/01/2024 à 21:07, Ryan Eatmon via lists.openembedded.org a écrit :
>> Add recipe for yamllint.  There is an upcoming change in u-boot where
>> the binman tool is now configured to call yamllint to verify the configs
>> during compile time.
>>
>> There was a previous patch a year ago from Trevor Woerner that never
>> made it into oe-core.  This patch is a reworking of his patch but
>> pointing to a newer version.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   .../python/python3-yamllint_1.33.0.bb             | 15 +++++++++++++++
>>   1 file changed, 15 insertions(+)
>>   create mode 100644 meta/recipes-devtools/python/python3-yamllint_1.33.0.bb
> 
> FYI, this new recipe lacks a maintainer and throw this warning :
> WARNING: python3-yamllint-1.33.0-r0 do_recipe_qa: QA Issue: Recipe python3-yamllint in [...]/poky/meta/recipes-devtools/python/python3-yamllint_1.33.0.bb does not have an assigned maintainer. Please add an entry into meta/conf/distro/include/maintainers.inc. [missing-maintainer]

Tim,

Do you mind being the maintainer for this new python recipe?  You are 
maintainer for many other python modules including pyyaml which this 
module depends on.  I can submit a v2 updating that file as well if you 
are willing.


> Regards,
Tim Orling Jan. 23, 2024, 4:56 a.m. UTC | #3
On Mon, Jan 22, 2024 at 3:36 PM Ryan Eatmon via lists.openembedded.org
<reatmon=ti.com@lists.openembedded.org> wrote:

>
>
> On 1/22/2024 4:30 PM, Yoann Congal wrote:
> > Hi,
> >
> > Le 22/01/2024 à 21:07, Ryan Eatmon via lists.openembedded.org a écrit :
> >> Add recipe for yamllint.  There is an upcoming change in u-boot where
> >> the binman tool is now configured to call yamllint to verify the configs
> >> during compile time.
> >>
> >> There was a previous patch a year ago from Trevor Woerner that never
> >> made it into oe-core.  This patch is a reworking of his patch but
> >> pointing to a newer version.
> >>
> >> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >> ---
> >>   .../python/python3-yamllint_1.33.0.bb             | 15
> +++++++++++++++
> >>   1 file changed, 15 insertions(+)
> >>   create mode 100644 meta/recipes-devtools/python/
> python3-yamllint_1.33.0.bb
> >
> > FYI, this new recipe lacks a maintainer and throw this warning :
> > WARNING: python3-yamllint-1.33.0-r0 do_recipe_qa: QA Issue: Recipe
> python3-yamllint in [...]/poky/meta/recipes-devtools/python/
> python3-yamllint_1.33.0.bb does not have an assigned maintainer. Please
> add an entry into meta/conf/distro/include/maintainers.inc.
> [missing-maintainer]
>
> Tim,
>
> Do you mind being the maintainer for this new python recipe?  You are
> maintainer for many other python modules including pyyaml which this
> module depends on.  I can submit a v2 updating that file as well if you
> are willing.
>

That’s fine with me.

—Tim

>
>
> > Regards,
>
> --
> Ryan Eatmon                reatmon@ti.com
> -----------------------------------------
> Texas Instruments, Inc.  -  LCPD  -  MGTS
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#194195):
> https://lists.openembedded.org/g/openembedded-core/message/194195
> Mute This Topic: https://lists.openembedded.org/mt/103894761/924729
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> ticotimo@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
diff mbox series

Patch

diff --git a/meta/recipes-devtools/python/python3-yamllint_1.33.0.bb b/meta/recipes-devtools/python/python3-yamllint_1.33.0.bb
new file mode 100644
index 0000000000..4b7bd065f9
--- /dev/null
+++ b/meta/recipes-devtools/python/python3-yamllint_1.33.0.bb
@@ -0,0 +1,15 @@ 
+SUMMARY = "A linter for YAML files."
+HOMEPAGE = "https://github.com/adrienverge/yamllint"
+LICENSE = "GPL-3.0-only"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=1ebbd3e34237af26da5dc08a4e440464"
+
+inherit pypi setuptools3
+
+PYPI_PACKAGE = "yamllint"
+
+SRC_URI[sha256sum] = "2dceab9ef2d99518a2fcf4ffc964d44250ac4459be1ba3ca315118e4a1a81f7d"
+
+DEPENDS += "${PYTHON_PN}-setuptools-scm-native"
+RDEPENDS:${PN} += "${PYTHON_PN}-pyyaml"
+
+BBCLASSEXTEND = "native nativesdk"