[OE-core] [PATCH] meta: license: fix non-SPDX license being removed from INCOMPATIBLE_LICENSE
Adrian Bunk
bunk at stusta.de
Thu Jun 6 14:52:36 UTC 2019
This breaks the build for me:
ERROR: test-image-1.0-r0 do_rootfs: Error executing a python function in exec_python_func() autogenerated:
The stack trace of python calls that resulted in this exception/failure was:
File: 'exec_python_func() autogenerated', lineno: 2, function: <module>
0001:
*** 0002:license_create_manifest(d)
0003:
File: '/home/bunk/yoctomusl/yocto-test/sources/poky/meta/classes/license_image.bbclass', lineno: 35, function: license_create_manifest
0031: pkg_dic[pkg_name]["LICENSE"] = pkg_dic[pkg_name][pkg_lic_name]
0032:
0033: rootfs_license_manifest = os.path.join(d.getVar('LICENSE_DIRECTORY'),
0034: d.getVar('IMAGE_NAME'), 'license.manifest')
*** 0035: write_license_files(d, rootfs_license_manifest, pkg_dic, rootfs=True)
0036:}
0037:
0038:def write_license_files(d, license_manifest, pkg_dic, rootfs=True):
0039: import re
File: '/home/bunk/yoctomusl/yocto-test/sources/poky/meta/classes/license_image.bbclass', lineno: 44, function: write_license_files
0040: import stat
0041:
0042: bad_licenses = (d.getVar("INCOMPATIBLE_LICENSE") or "").split()
0043: bad_licenses = map(lambda l: canonical_license(d, l), bad_licenses)
*** 0044: bad_licenses = expand_wildcard_licenses(d, bad_licenses)
0045:
0046: with open(license_manifest, "w") as license_file:
0047: for pkg in sorted(pkg_dic):
0048: if bad_licenses:
File: '/home/bunk/yoctomusl/yocto-test/sources/poky/meta/classes/license.bbclass', lineno: 271, function: expand_wildcard_licenses
0267: Return actual spdx format license names if wildcard used. We expand
0268: wildcards from SPDXLICENSEMAP flags and SRC_DISTRIBUTE_LICENSES values.
0269: """
0270: import fnmatch
*** 0271: licenses = wildcard_licenses[:]
0272: spdxmapkeys = d.getVarFlags('SPDXLICENSEMAP').keys()
0273: for wld_lic in wildcard_licenses:
0274: spdxflags = fnmatch.filter(spdxmapkeys, wld_lic)
0275: licenses += [d.getVarFlag('SPDXLICENSEMAP', flag) for flag in spdxflags]
Exception: TypeError: 'map' object is not subscriptable
ERROR: test-image-1.0-r0 do_rootfs: 'map' object is not subscriptable
ERROR: test-image-1.0-r0 do_rootfs: Function failed: license_create_manifest
ERROR: Logfile of failure stored in: /home/bunk/yoctomusl/yocto-test/build_test/tmp/work/test-test-linux-musl/test-image/1.0-r0/temp/log.do_rootfs.29928
ERROR: Task (/home/bunk/yoctomusl/yocto-test/sources/meta-test/recipes-test/images/test-image.bb:do_rootfs) failed with exit code '1'
cu
Adrian
On Wed, Jun 05, 2019 at 05:23:52PM +0200, Quentin Schulz wrote:
> A non-SPDX license (which is not an alias to an SPDX license) cannot
> currently be marked as incompatible in INCOMPATIBLE_LICENSE.
> In the current state, we take all INCOMPATIBLE_LICENSE and pass them
> through expand_wildcard_licenses which is only adding SPDX licenses that
> match the glob regexp of what is in INCOMPATIBLE_LICENSE (be it a direct
> match to an SPDX license or via an alias).
>
> This does not work well with custom licenses.
>
> E.g.:
>
> foo.bb:
> LICENSE = "FooLicense"
>
> conf/local.conf:
> INCOMPATIBLE_LICENSE = "FooLicense"
>
> `bitbake foo`
>
> Gives no warning, no error, builds and packages successfully, because
> INCOMPATIBLE_LICENSE is basically empty since FooLicense is neither in
> SPDXLICENSEMAP nor in SRC_DISTRIBUTE_LICENSES.
>
> Let's add the original licenses to the list returned by
> expand_wildcard_licenses to be able to handle the aforementioned case.
>
> INCOMPATIBLE_LICENSE = "FooLicense GPLv2 GPLv3+" used to "resolve" to
> "GPLv2 GPLv3". It now resolves to "FooLicense GPLv2 GPLv3 GPLv3+" which
> fixes the issue with custom licenses not being in SPDXLICENSEMAP or
> SRC_DISTRIBUTE_LICENSES and thus being left out of the blacklisted
> licenses.
>
> Signed-off-by: Quentin Schulz <quentin.schulz at streamunlimited.com>
> ---
> meta/classes/license.bbclass | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
> index ed91a4b4db..adca881c85 100644
> --- a/meta/classes/license.bbclass
> +++ b/meta/classes/license.bbclass
> @@ -268,7 +268,7 @@ def expand_wildcard_licenses(d, wildcard_licenses):
> wildcards from SPDXLICENSEMAP flags and SRC_DISTRIBUTE_LICENSES values.
> """
> import fnmatch
> - licenses = []
> + licenses = wildcard_licenses[:]
> spdxmapkeys = d.getVarFlags('SPDXLICENSEMAP').keys()
> for wld_lic in wildcard_licenses:
> spdxflags = fnmatch.filter(spdxmapkeys, wld_lic)
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
More information about the Openembedded-core
mailing list