diff options
| author | Saul Wold <Saul.Wold@windriver.com> | 2022-02-24 13:55:33 -0800 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2022-02-25 12:41:24 +0000 |
| commit | e62965346314544919e4a529ca860a242d76b303 (patch) | |
| tree | 143e1a3fa8f1feb13c9a9ed631d6a37d425e0ce2 /meta/lib | |
| parent | fc2b2c7927acc8047d71433342e73912b93405f7 (diff) | |
| download | poky-e62965346314544919e4a529ca860a242d76b303.tar.gz | |
license.py: rename variables
Update the comment to reflect new variable names
(From OE-Core rev: 7fbab1aefc127f0e1834f51a8a793b0d7e7b4f07)
Signed-off-by: Saul Wold <saul.wold@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib')
| -rw-r--r-- | meta/lib/oe/license.py | 31 |
1 files changed, 17 insertions, 14 deletions
diff --git a/meta/lib/oe/license.py b/meta/lib/oe/license.py index b5d378a549..b1105f6149 100644 --- a/meta/lib/oe/license.py +++ b/meta/lib/oe/license.py | |||
| @@ -99,26 +99,29 @@ def flattened_licenses(licensestr, choose_licenses): | |||
| 99 | raise LicenseSyntaxError(licensestr, exc) | 99 | raise LicenseSyntaxError(licensestr, exc) |
| 100 | return flatten.licenses | 100 | return flatten.licenses |
| 101 | 101 | ||
| 102 | def is_included(licensestr, whitelist=None, blacklist=None): | 102 | def is_included(licensestr, include_licenses=None, exclude_licenses=None): |
| 103 | """Given a license string and whitelist and blacklist, determine if the | 103 | """Given a license a list of list to include and a list of |
| 104 | license string matches the whitelist and does not match the blacklist. | 104 | licenses to exclude, determine if the license string |
| 105 | 105 | matches the an include list and does not match the | |
| 106 | Returns a tuple holding the boolean state and a list of the applicable | 106 | exclude list. |
| 107 | licenses that were excluded if state is False, or the licenses that were | 107 | |
| 108 | included if the state is True. | 108 | Returns a tuple holding the boolean state and a list of |
| 109 | the applicable licenses that were excluded if state is | ||
| 110 | False, or the licenses that were included if the state | ||
| 111 | is True. | ||
| 109 | """ | 112 | """ |
| 110 | 113 | ||
| 111 | def include_license(license): | 114 | def include_license(license): |
| 112 | return any(fnmatch(license, pattern) for pattern in whitelist) | 115 | return any(fnmatch(license, pattern) for pattern in include_licenses) |
| 113 | 116 | ||
| 114 | def exclude_license(license): | 117 | def exclude_license(license): |
| 115 | return any(fnmatch(license, pattern) for pattern in blacklist) | 118 | return any(fnmatch(license, pattern) for pattern in exclude_licenses) |
| 116 | 119 | ||
| 117 | def choose_licenses(alpha, beta): | 120 | def choose_licenses(alpha, beta): |
| 118 | """Select the option in an OR which is the 'best' (has the most | 121 | """Select the option in an OR which is the 'best' (has the most |
| 119 | included licenses and no excluded licenses).""" | 122 | included licenses and no excluded licenses).""" |
| 120 | # The factor 1000 below is arbitrary, just expected to be much larger | 123 | # The factor 1000 below is arbitrary, just expected to be much larger |
| 121 | # that the number of licenses actually specified. That way the weight | 124 | # than the number of licenses actually specified. That way the weight |
| 122 | # will be negative if the list of licenses contains an excluded license, | 125 | # will be negative if the list of licenses contains an excluded license, |
| 123 | # but still gives a higher weight to the list with the most included | 126 | # but still gives a higher weight to the list with the most included |
| 124 | # licenses. | 127 | # licenses. |
| @@ -131,11 +134,11 @@ def is_included(licensestr, whitelist=None, blacklist=None): | |||
| 131 | else: | 134 | else: |
| 132 | return beta | 135 | return beta |
| 133 | 136 | ||
| 134 | if not whitelist: | 137 | if not include_licenses: |
| 135 | whitelist = ['*'] | 138 | include = ['*'] |
| 136 | 139 | ||
| 137 | if not blacklist: | 140 | if not exclude_licenses: |
| 138 | blacklist = [] | 141 | exclude = [] |
| 139 | 142 | ||
| 140 | licenses = flattened_licenses(licensestr, choose_licenses) | 143 | licenses = flattened_licenses(licensestr, choose_licenses) |
| 141 | excluded = [lic for lic in licenses if exclude_license(lic)] | 144 | excluded = [lic for lic in licenses if exclude_license(lic)] |
