diff options
| author |   Paul Eggleton <paul.eggleton@linux.intel.com> | 2017-10-31 11:30:49 +1300 |
|---|---|---|
| committer | Paul Eggleton <paul.eggleton@linux.intel.com> | 2017-11-09 14:47:44 +1300 |
| commit | f88e40e440c9ee6e1e581a2e8a2059f1597edc08 (patch) | |
| tree | 10f09d33710d4e60e698293230529465e40532e6 | |
| parent | 2f0182931531272447e9a044847b2d585ed547b5 (diff) | |
| download | openembedded-core-contrib-f88e40e440c9ee6e1e581a2e8a2059f1597edc08.tar.gz | |
devtool: upgrade: reformat --no-patch warning message
* Only log one warning message instead of one per line
* Be a bit more verbose
* "if list" is more pythonic than "if len(list)"
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
| -rw-r--r-- | scripts/lib/devtool/upgrade.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/scripts/lib/devtool/upgrade.py b/scripts/lib/devtool/upgrade.py index 0db2a50e90..3cb523c2f4 100644 --- a/scripts/lib/devtool/upgrade.py +++ b/scripts/lib/devtool/upgrade.py @@ -263,10 +263,8 @@ def _extract_new_source(newpv, srctree, no_patch, srcrev, srcbranch, branch, kee if no_patch: patches = oe.recipeutils.get_recipe_patches(crd) - if len(patches): - logger.warn('By user choice, the following patches will NOT be applied') - for patch in patches: - logger.warn("%s" % os.path.basename(patch)) + if patches: + logger.warn('By user choice, the following patches will NOT be applied to the new source tree:\n %s' % '\n '.join([os.path.basename(patch) for patch in patches])) else: __run('git checkout devtool-patched -b %s' % branch) skiptag = False |