Message ID | 20220224152659.134461-1-saul.wold@windriver.com |
---|---|
State | Accepted, archived |
Commit | 7fbab1aefc127f0e1834f51a8a793b0d7e7b4f07 |
Headers | show |
Series | license.py: rename variables | expand |
> -----Original Message----- > From: openembedded-core@lists.openembedded.org <openembedded-core@lists.openembedded.org> On Behalf Of Saul Wold > Sent: den 24 februari 2022 16:27 > To: openembedded-core@lists.openembedded.org > Cc: Saul Wold <saul.wold@windriver.com> > Subject: [OE-core] [PATCH] license.py: rename variables > > Update the comment to reflect new variable names > > Signed-off-by: Saul Wold <saul.wold@windriver.com> > --- > meta/lib/oe/license.py | 28 +++++++++++++++------------- > 1 file changed, 15 insertions(+), 13 deletions(-) > > diff --git a/meta/lib/oe/license.py b/meta/lib/oe/license.py > index b5d378a549b..4cd382b4fd7 100644 > --- a/meta/lib/oe/license.py > +++ b/meta/lib/oe/license.py > @@ -99,20 +99,22 @@ def flattened_licenses(licensestr, choose_licenses): > raise LicenseSyntaxError(licensestr, exc) > return flatten.licenses > > -def is_included(licensestr, whitelist=None, blacklist=None): > - """Given a license string and whitelist and blacklist, determine if the > - license string matches the whitelist and does not match the blacklist. > - > - Returns a tuple holding the boolean state and a list of the applicable > - licenses that were excluded if state is False, or the licenses that were > - included if the state is True. > +def is_included(licensestr, include=None, exclude=None): I suggest calling the renamed arguments "include_licenses" and "exclude_licenses" instead. "include" and "exclude" are very generic. > + """Given a license string and include list and exclude list, > + determine if the license string matches the an included > + license and does dont match an excluded license. Change to: """Given a license string, a list of licenses to include and a list of licenses to exclude, determine if the license string matches the include list and that it does not match the exclude list. > + > + Returns a tuple holding the boolean state and a list of > + the applicable licenses that were excluded if state is > + False, or the licenses that were included if the state > + is True. > """ > > def include_license(license): > - return any(fnmatch(license, pattern) for pattern in whitelist) > + return any(fnmatch(license, pattern) for pattern in include) > > def exclude_license(license): > - return any(fnmatch(license, pattern) for pattern in blacklist) > + return any(fnmatch(license, pattern) for pattern in exclude) > > def choose_licenses(alpha, beta): > """Select the option in an OR which is the 'best' (has the most A few lines below the above is a comment line that starts "that the number". Can you change it to "than the number" while here? > @@ -131,11 +133,11 @@ def is_included(licensestr, whitelist=None, blacklist=None): > else: > return beta > > - if not whitelist: > - whitelist = ['*'] > + if not include: > + include = ['*'] > > - if not blacklist: > - blacklist = [] > + if not exclude: > + exclude = [] > > licenses = flattened_licenses(licensestr, choose_licenses) > excluded = [lic for lic in licenses if exclude_license(lic)] > -- > 2.31.1 //Peter
diff --git a/meta/lib/oe/license.py b/meta/lib/oe/license.py index b5d378a549b..4cd382b4fd7 100644 --- a/meta/lib/oe/license.py +++ b/meta/lib/oe/license.py @@ -99,20 +99,22 @@ def flattened_licenses(licensestr, choose_licenses): raise LicenseSyntaxError(licensestr, exc) return flatten.licenses -def is_included(licensestr, whitelist=None, blacklist=None): - """Given a license string and whitelist and blacklist, determine if the - license string matches the whitelist and does not match the blacklist. - - Returns a tuple holding the boolean state and a list of the applicable - licenses that were excluded if state is False, or the licenses that were - included if the state is True. +def is_included(licensestr, include=None, exclude=None): + """Given a license string and include list and exclude list, + determine if the license string matches the an included + license and does dont match an excluded license. + + Returns a tuple holding the boolean state and a list of + the applicable licenses that were excluded if state is + False, or the licenses that were included if the state + is True. """ def include_license(license): - return any(fnmatch(license, pattern) for pattern in whitelist) + return any(fnmatch(license, pattern) for pattern in include) def exclude_license(license): - return any(fnmatch(license, pattern) for pattern in blacklist) + return any(fnmatch(license, pattern) for pattern in exclude) def choose_licenses(alpha, beta): """Select the option in an OR which is the 'best' (has the most @@ -131,11 +133,11 @@ def is_included(licensestr, whitelist=None, blacklist=None): else: return beta - if not whitelist: - whitelist = ['*'] + if not include: + include = ['*'] - if not blacklist: - blacklist = [] + if not exclude: + exclude = [] licenses = flattened_licenses(licensestr, choose_licenses) excluded = [lic for lic in licenses if exclude_license(lic)]
Update the comment to reflect new variable names Signed-off-by: Saul Wold <saul.wold@windriver.com> --- meta/lib/oe/license.py | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-)