From 4005c3ff5e673f9d561026c76326054bd0a3a2b6 Mon Sep 17 00:00:00 2001 From: Andrei Gherzan Date: Tue, 10 Jan 2012 17:17:58 +0200 Subject: license.bbclass base.bbclass: support for 'or' operand in LICENSE and for SPDX license names A new function was defined in license.bbclass in order to correctly exclude packages where OE-Style licence naming is used. In this way licenses as GPL-3, GPLv3, GPLv3.0 etc will be excluded from a non-GPLv3 build. This function takes into consideration if 'or' operand is used. The function defined in license.bbclass is called in base.bbclass where packages are excluded based on INCOMPATIBLE_LICENSE variable. [YOCTO #1884] [YOCTO #1844] (From OE-Core rev: 28456593be0b7e15bb51595d547d7e5347cce24b) Signed-off-by: Andrei Gherzan Signed-off-by: Richard Purdie --- meta/classes/base.bbclass | 3 +-- meta/classes/license.bbclass | 45 ++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 46 insertions(+), 2 deletions(-) diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass index e65a7220f5..f0c358ea32 100644 --- a/meta/classes/base.bbclass +++ b/meta/classes/base.bbclass @@ -398,9 +398,8 @@ python () { dont_want_whitelist = (d.getVar('WHITELIST_%s' % dont_want_license, 1) or "").split() if pn not in hosttools_whitelist and pn not in lgplv2_whitelist and pn not in dont_want_whitelist: - import re this_license = d.getVar('LICENSE', 1) - if this_license and re.search(dont_want_license, this_license): + if incompatible_license(d,dont_want_license): bb.note("SKIPPING %s because it's %s" % (pn, this_license)) raise bb.parse.SkipPackage("incompatible with license %s" % this_license) diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass index d351b5aaed..4b98e29916 100644 --- a/meta/classes/license.bbclass +++ b/meta/classes/license.bbclass @@ -237,6 +237,51 @@ python do_populate_lic() { } +def incompatible_license(d,dont_want_license): + """ + This function checks if a package has only incompatible licenses. It also take into consideration 'or' + operand. + """ + import re + import oe.license + from fnmatch import fnmatchcase as fnmatch + + dont_want_licenses = [] + dont_want_licenses.append(d.getVar('INCOMPATIBLE_LICENSE', 1)) + if d.getVarFlag('SPDXLICENSEMAP', dont_want_license): + dont_want_licenses.append(d.getVarFlag('SPDXLICENSEMAP', dont_want_license)) + + def include_license(license): + if any(fnmatch(license, pattern) for pattern in dont_want_licenses): + return False + else: + spdx_license = d.getVarFlag('SPDXLICENSEMAP', license) + if spdx_license and any(fnmatch(spdx_license, pattern) for pattern in dont_want_licenses): + return False + else: + return True + + def choose_licenses(a, b): + if all(include_license(lic) for lic in a): + return a + else: + return b + + """ + If you want to exlude license named generically 'X', we surely want to exlude 'X+' as well. + In consequence, we will exclude the '+' character from LICENSE in case INCOMPATIBLE_LICENSE + is not a 'X+' license. + """ + if not re.search(r'[+]',dont_want_license): + licenses=oe.license.flattened_licenses(re.sub(r'[+]', '', d.getVar('LICENSE', True)), choose_licenses) + else: + licenses=oe.license.flattened_licenses(d.getVar('LICENSE', True), choose_licenses) + + for onelicense in licenses: + if not include_license(onelicense): + return True + return False + SSTATETASKS += "do_populate_lic" do_populate_lic[sstate-name] = "populate-lic" do_populate_lic[sstate-inputdirs] = "${LICSSTATEDIR}" -- cgit v1.2.3-54-g00ecf