[oe-commits] [openembedded-core] 04/10: license: fix non-SPDX license being removed from INCOMPATIBLE_LICENSE

[git at git.openembedded.org]

This is an automated email from the git hooks/post-receive script.

rpurdie pushed a commit to branch master-next
in repository openembedded-core.

commit a3c9a416fec7ea97e9e6d79a73a8b8c0c833faf2
Author: Quentin Schulz <quentin.schulz at streamunlimited.com>
AuthorDate: Wed Jun 5 17:23:52 2019 +0200

    license: fix non-SPDX license being removed from INCOMPATIBLE_LICENSE
    
    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>
    Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
---
 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 ed91a4b..adca881 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)

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.