diff options
| author | Joshua Lock <joshua.g.lock@intel.com> | 2016-12-14 21:13:06 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-12-16 10:23:23 +0000 |
| commit | 3c59b1bf93adb0b9f723bda1d8702c8720733677 (patch) | |
| tree | d1939643e2d02f0ce2e9ef33e235cd23cd5000d3 /meta/classes/license.bbclass | |
| parent | c0f2890c01882e9ea14e781c044f3a84f75bd0fc (diff) | |
| download | poky-3c59b1bf93adb0b9f723bda1d8702c8720733677.tar.gz | |
meta: remove True option to getVarFlag calls
getVarFlag() now defaults to expanding by default, thus remove the
True option from getVarFlag() calls with a regex search and
replace.
Search made with the following regex:
getVarFlag ?\(( ?[^,()]*, ?[^,()]*), True\)
(From OE-Core rev: 2dea9e490a98377010b3d4118d054814c317a735)
Signed-off-by: Joshua Lock <joshua.g.lock@intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/license.bbclass')
| -rw-r--r-- | meta/classes/license.bbclass | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass index c5d1204ca1..009ee4a37d 100644 --- a/meta/classes/license.bbclass +++ b/meta/classes/license.bbclass | |||
| @@ -249,7 +249,7 @@ def get_boot_dependencies(d): | |||
| 249 | 249 | ||
| 250 | for task in boot_tasks: | 250 | for task in boot_tasks: |
| 251 | boot_depends_string = "%s %s" % (boot_depends_string, | 251 | boot_depends_string = "%s %s" % (boot_depends_string, |
| 252 | d.getVarFlag(task, "depends", True) or "") | 252 | d.getVarFlag(task, "depends") or "") |
| 253 | boot_depends = [dep.split(":")[0] for dep | 253 | boot_depends = [dep.split(":")[0] for dep |
| 254 | in boot_depends_string.split() | 254 | in boot_depends_string.split() |
| 255 | if not dep.split(":")[0].endswith("-native")] | 255 | if not dep.split(":")[0].endswith("-native")] |
| @@ -431,10 +431,10 @@ def find_license_files(d): | |||
| 431 | # unless NO_GENERIC_LICENSE is set. | 431 | # unless NO_GENERIC_LICENSE is set. |
| 432 | for lic_dir in license_source_dirs: | 432 | for lic_dir in license_source_dirs: |
| 433 | if not os.path.isfile(os.path.join(lic_dir, license_type)): | 433 | if not os.path.isfile(os.path.join(lic_dir, license_type)): |
| 434 | if d.getVarFlag('SPDXLICENSEMAP', license_type, True) != None: | 434 | if d.getVarFlag('SPDXLICENSEMAP', license_type) != None: |
| 435 | # Great, there is an SPDXLICENSEMAP. We can copy! | 435 | # Great, there is an SPDXLICENSEMAP. We can copy! |
| 436 | bb.debug(1, "We need to use a SPDXLICENSEMAP for %s" % (license_type)) | 436 | bb.debug(1, "We need to use a SPDXLICENSEMAP for %s" % (license_type)) |
| 437 | spdx_generic = d.getVarFlag('SPDXLICENSEMAP', license_type, True) | 437 | spdx_generic = d.getVarFlag('SPDXLICENSEMAP', license_type) |
| 438 | license_source = lic_dir | 438 | license_source = lic_dir |
| 439 | break | 439 | break |
| 440 | elif os.path.isfile(os.path.join(lic_dir, license_type)): | 440 | elif os.path.isfile(os.path.join(lic_dir, license_type)): |
| @@ -442,7 +442,7 @@ def find_license_files(d): | |||
| 442 | license_source = lic_dir | 442 | license_source = lic_dir |
| 443 | break | 443 | break |
| 444 | 444 | ||
| 445 | non_generic_lic = d.getVarFlag('NO_GENERIC_LICENSE', license_type, True) | 445 | non_generic_lic = d.getVarFlag('NO_GENERIC_LICENSE', license_type) |
| 446 | if spdx_generic and license_source: | 446 | if spdx_generic and license_source: |
| 447 | # we really should copy to generic_ + spdx_generic, however, that ends up messing the manifest | 447 | # we really should copy to generic_ + spdx_generic, however, that ends up messing the manifest |
| 448 | # audit up. This should be fixed in emit_pkgdata (or, we actually got and fix all the recipes) | 448 | # audit up. This should be fixed in emit_pkgdata (or, we actually got and fix all the recipes) |
| @@ -451,7 +451,7 @@ def find_license_files(d): | |||
| 451 | 451 | ||
| 452 | # The user may attempt to use NO_GENERIC_LICENSE for a generic license which doesn't make sense | 452 | # The user may attempt to use NO_GENERIC_LICENSE for a generic license which doesn't make sense |
| 453 | # and should not be allowed, warn the user in this case. | 453 | # and should not be allowed, warn the user in this case. |
| 454 | if d.getVarFlag('NO_GENERIC_LICENSE', license_type, True): | 454 | if d.getVarFlag('NO_GENERIC_LICENSE', license_type): |
| 455 | bb.warn("%s: %s is a generic license, please don't use NO_GENERIC_LICENSE for it." % (pn, license_type)) | 455 | bb.warn("%s: %s is a generic license, please don't use NO_GENERIC_LICENSE for it." % (pn, license_type)) |
| 456 | 456 | ||
| 457 | elif non_generic_lic and non_generic_lic in lic_chksums: | 457 | elif non_generic_lic and non_generic_lic in lic_chksums: |
| @@ -505,7 +505,7 @@ def return_spdx(d, license): | |||
| 505 | """ | 505 | """ |
| 506 | This function returns the spdx mapping of a license if it exists. | 506 | This function returns the spdx mapping of a license if it exists. |
| 507 | """ | 507 | """ |
| 508 | return d.getVarFlag('SPDXLICENSEMAP', license, True) | 508 | return d.getVarFlag('SPDXLICENSEMAP', license) |
| 509 | 509 | ||
| 510 | def canonical_license(d, license): | 510 | def canonical_license(d, license): |
| 511 | """ | 511 | """ |
| @@ -514,7 +514,7 @@ def canonical_license(d, license): | |||
| 514 | 'X' if availabel and the tailing '+' (so GPLv3+ becomes GPL-3.0+), | 514 | 'X' if availabel and the tailing '+' (so GPLv3+ becomes GPL-3.0+), |
| 515 | or the passed license if there is no canonical form. | 515 | or the passed license if there is no canonical form. |
| 516 | """ | 516 | """ |
| 517 | lic = d.getVarFlag('SPDXLICENSEMAP', license, True) or "" | 517 | lic = d.getVarFlag('SPDXLICENSEMAP', license) or "" |
| 518 | if not lic and license.endswith('+'): | 518 | if not lic and license.endswith('+'): |
| 519 | lic = d.getVarFlag('SPDXLICENSEMAP', license.rstrip('+'), True) | 519 | lic = d.getVarFlag('SPDXLICENSEMAP', license.rstrip('+'), True) |
| 520 | if lic: | 520 | if lic: |
| @@ -531,7 +531,7 @@ def expand_wildcard_licenses(d, wildcard_licenses): | |||
| 531 | spdxmapkeys = d.getVarFlags('SPDXLICENSEMAP').keys() | 531 | spdxmapkeys = d.getVarFlags('SPDXLICENSEMAP').keys() |
| 532 | for wld_lic in wildcard_licenses: | 532 | for wld_lic in wildcard_licenses: |
| 533 | spdxflags = fnmatch.filter(spdxmapkeys, wld_lic) | 533 | spdxflags = fnmatch.filter(spdxmapkeys, wld_lic) |
| 534 | licenses += [d.getVarFlag('SPDXLICENSEMAP', flag, True) for flag in spdxflags] | 534 | licenses += [d.getVarFlag('SPDXLICENSEMAP', flag) for flag in spdxflags] |
| 535 | 535 | ||
| 536 | spdx_lics = (d.getVar('SRC_DISTRIBUTE_LICENSES', False) or '').split() | 536 | spdx_lics = (d.getVar('SRC_DISTRIBUTE_LICENSES', False) or '').split() |
| 537 | for wld_lic in wildcard_licenses: | 537 | for wld_lic in wildcard_licenses: |
