diff options
author | Peter Kjellerstedt <peter.kjellerstedt@axis.com> | 2022-03-24 00:14:57 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2022-03-24 17:41:41 +0000 |
commit | 59eb405a80f0a0acf9b754b2b78399bacb0094ae (patch) | |
tree | 3a33d8d5973ef4f28b35f8937cdcba88ba01f7cb /meta/classes/create-spdx.bbclass | |
parent | 1c7802f025eb3d9ee3310e083a5d8bef7178de9c (diff) | |
download | openembedded-core-contrib-59eb405a80f0a0acf9b754b2b78399bacb0094ae.tar.gz |
create-spdx.bbclass: Simplify extraction of license text
There is no reason to first search for all available licenses using
avail_licenses() and then search through the same paths looking for
the actual license texts. Just look for the license texts directly
instead.
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Diffstat (limited to 'meta/classes/create-spdx.bbclass')
-rw-r--r-- | meta/classes/create-spdx.bbclass | 25 |
1 files changed, 10 insertions, 15 deletions
diff --git a/meta/classes/create-spdx.bbclass b/meta/classes/create-spdx.bbclass index 1a4804a7c5..cae8cccd8d 100644 --- a/meta/classes/create-spdx.bbclass +++ b/meta/classes/create-spdx.bbclass @@ -94,7 +94,6 @@ def convert_license_to_spdx(lic, document, d, existing={}): from pathlib import Path import oe.spdx - avail_licenses = available_licenses(d) license_data = d.getVar("SPDX_LICENSE_DATA") extracted = {} @@ -112,8 +111,8 @@ def convert_license_to_spdx(lic, document, d, existing={}): if name == "PD": # Special-case this. extracted_info.extractedText = "Software released to the public domain" - elif name in avail_licenses: - # This license can be found in COMMON_LICENSE_DIR or LICENSE_PATH + else: + # Seach for the license in COMMON_LICENSE_DIR and LICENSE_PATH for directory in [d.getVar('COMMON_LICENSE_DIR')] + (d.getVar('LICENSE_PATH') or '').split(): try: with (Path(directory) / name).open(errors="replace") as f: @@ -122,18 +121,14 @@ def convert_license_to_spdx(lic, document, d, existing={}): except FileNotFoundError: pass if extracted_info.extractedText is None: - # Error out, as the license was in avail_licenses so should - # be on disk somewhere. - bb.error("Cannot find text for license %s" % name) - else: - # If it's not SPDX, or PD, or in avail_licenses, then NO_GENERIC_LICENSE must be set - filename = d.getVarFlag('NO_GENERIC_LICENSE', name) - if filename: - filename = d.expand("${S}/" + filename) - with open(filename, errors="replace") as f: - extracted_info.extractedText = f.read() - else: - bb.error("Cannot find any text for license %s" % name) + # If it's not SPDX or PD, then NO_GENERIC_LICENSE must be set + filename = d.getVarFlag('NO_GENERIC_LICENSE', name) + if filename: + filename = d.expand("${S}/" + filename) + with open(filename, errors="replace") as f: + extracted_info.extractedText = f.read() + else: + bb.error("Cannot find any text for license %s" % name) extracted[name] = extracted_info document.hasExtractedLicensingInfos.append(extracted_info) |