On Mon, Mar 9, 2026 at 7:29 AM <[email protected]> wrote: > > From: Stefano Tondo <[email protected]> > > Have each ecosystem bbclass set its own Package URL by prepending to > SPDX_PACKAGE_URLS, rather than detecting inherited classes from the > SPDX code. This follows the principle that each class should know how > to describe itself.
Much better! Except for something I noticed below.... > > The following bbclasses now generate ecosystem PURLs: > - pypi.bbclass: pkg:pypi/<normalized-name>@PV > - npm.bbclass: pkg:npm/<name>@PV > - cargo_common.bbclass: pkg:cargo/<name>@PV > - go-mod.bbclass: pkg:golang/<GO_IMPORT>@PV > - cpan.bbclass: pkg:cpan/<name>@PV > > Additional ecosystems (nuget, maven, dotnet) can follow the same > pattern in their respective layers. > > Signed-off-by: Stefano Tondo <[email protected]> > --- > meta/classes-recipe/cargo_common.bbclass | 3 +++ > meta/classes-recipe/cpan.bbclass | 11 +++++++++++ > meta/classes-recipe/go-mod.bbclass | 3 +++ > meta/classes-recipe/npm.bbclass | 7 +++++++ > meta/classes-recipe/pypi.bbclass | 3 +++ > 5 files changed, 27 insertions(+) > > diff --git a/meta/classes-recipe/cargo_common.bbclass > b/meta/classes-recipe/cargo_common.bbclass > index bc44ad7918..e884b344ef 100644 > --- a/meta/classes-recipe/cargo_common.bbclass > +++ b/meta/classes-recipe/cargo_common.bbclass > @@ -240,3 +240,6 @@ EXPORT_FUNCTIONS do_configure > # https://github.com/rust-lang/libc/issues/3223 > # https://github.com/rust-lang/libc/pull/3175 > INSANE_SKIP:append = " 32bit-time" > + > +# Generate ecosystem-specific Package URL for SPDX > +SPDX_PACKAGE_URLS:prepend = "pkg:cargo/${BPN}@${PV} " > diff --git a/meta/classes-recipe/cpan.bbclass > b/meta/classes-recipe/cpan.bbclass > index bb76a5b326..355e7e6adf 100644 > --- a/meta/classes-recipe/cpan.bbclass > +++ b/meta/classes-recipe/cpan.bbclass > @@ -68,4 +68,15 @@ cpan_do_install () { > done > } > > +# Generate ecosystem-specific Package URL for SPDX > +def cpan_spdx_name(d): > + bpn = d.getVar('BPN') > + if bpn.startswith('perl-'): > + return bpn[5:] > + elif bpn.startswith('libperl-'): > + return bpn[8:] > + return bpn > + > +SPDX_PACKAGE_URLS:prepend = "pkg:cpan/${@cpan_spdx_name(d)}@${PV} " > + > EXPORT_FUNCTIONS do_configure do_compile do_install > diff --git a/meta/classes-recipe/go-mod.bbclass > b/meta/classes-recipe/go-mod.bbclass > index a15dda8f0e..344712b193 100644 > --- a/meta/classes-recipe/go-mod.bbclass > +++ b/meta/classes-recipe/go-mod.bbclass > @@ -32,3 +32,6 @@ do_compile[dirs] += "${B}/src/${GO_WORKDIR}" > # Make go install unpack the module zip files in the module cache directory > # before the license directory is polulated with license files. > addtask do_compile before do_populate_lic > + > +# Generate ecosystem-specific Package URL for SPDX > +SPDX_PACKAGE_URLS:prepend = "pkg:golang/${GO_IMPORT}@${PV} " > diff --git a/meta/classes-recipe/npm.bbclass b/meta/classes-recipe/npm.bbclass > index 344e8b4bec..aec69ebfd3 100644 > --- a/meta/classes-recipe/npm.bbclass > +++ b/meta/classes-recipe/npm.bbclass > @@ -354,4 +354,11 @@ FILES:${PN} += " \ > ${nonarch_libdir} \ > " > > +# Generate ecosystem-specific Package URL for SPDX > +def npm_spdx_name(d): > + bpn = d.getVar('BPN') > + return bpn[4:] if bpn.startswith('node-') else bpn return bpn[5:] ... otherwise the "-" is kept. > + > +SPDX_PACKAGE_URLS:prepend = "pkg:npm/${@npm_spdx_name(d)}@${PV} " > + > EXPORT_FUNCTIONS do_configure do_compile do_install > diff --git a/meta/classes-recipe/pypi.bbclass > b/meta/classes-recipe/pypi.bbclass > index 1372d85e8d..fd5cd7af95 100644 > --- a/meta/classes-recipe/pypi.bbclass > +++ b/meta/classes-recipe/pypi.bbclass > @@ -55,3 +55,6 @@ UPSTREAM_CHECK_URI ?= > "https://pypi.org/simple/${@pypi_normalize(d)}/" > UPSTREAM_CHECK_REGEX ?= > "${UPSTREAM_CHECK_PYPI_PACKAGE}-(?P<pver>(\d+[\.\-_]*)+).(tar\.gz|tgz|zip|tar\.bz2)" > > CVE_PRODUCT ?= "python:${PYPI_PACKAGE}" > + > +# Generate ecosystem-specific Package URL for SPDX > +SPDX_PACKAGE_URLS:prepend = "pkg:pypi/${@pypi_normalize(d)}@${PV} " > -- > 2.53.0 >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#232909): https://lists.openembedded.org/g/openembedded-core/message/232909 Mute This Topic: https://lists.openembedded.org/mt/118221137/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
