diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-03-03 11:07:51 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-03-03 11:07:51 +0000 |
commit | ba871e0118d5ecd7379ad6495baa8416bbfbba44 (patch) | |
tree | 214a1d6928560dc15ba04c81ae2ed32c50c8b3ed /lib/bb/providers.py | |
parent | 9c3fbc7b2964f001f2dd49b6efd98ba9a42528ad (diff) | |
download | bitbake-ba871e0118d5ecd7379ad6495baa8416bbfbba44.tar.gz |
bitbake: Convert getVar/getVarFlag(xxx, 1) -> (xxx, True)
Using "1" with getVar is bad coding style and "True" is preferred.
This patch is a sed over bitbake directory of the form:
sed \
-e 's:\(\.getVar([^,()]*, \)1 *):\1True):g' \
-e 's:\(\.getVarFlag([^,()]*, [^,()]*, \)1 *):\1True):g' \
-i `grep -ril getVar *`
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'lib/bb/providers.py')
-rw-r--r-- | lib/bb/providers.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/bb/providers.py b/lib/bb/providers.py index 1dc6a8e8b..bd729df93 100644 --- a/lib/bb/providers.py +++ b/lib/bb/providers.py @@ -284,7 +284,7 @@ def filterProviders(providers, item, cfgData, dataCache): eligible = _filterProviders(providers, item, cfgData, dataCache) - prefervar = cfgData.getVar('PREFERRED_PROVIDER_%s' % item, 1) + prefervar = cfgData.getVar('PREFERRED_PROVIDER_%s' % item, True) if prefervar: dataCache.preferred[item] = prefervar @@ -322,7 +322,7 @@ def filterProvidersRunTime(providers, item, cfgData, dataCache): pn = dataCache.pkg_fn[p] provides = dataCache.pn_provides[pn] for provide in provides: - prefervar = cfgData.getVar('PREFERRED_PROVIDER_%s' % provide, 1) + prefervar = cfgData.getVar('PREFERRED_PROVIDER_%s' % provide, True) logger.debug(1, "checking PREFERRED_PROVIDER_%s (value %s) against %s", provide, prefervar, pns.keys()) if prefervar in pns and pns[prefervar] not in preferred: var = "PREFERRED_PROVIDER_%s = %s" % (provide, prefervar) |