From: Paul Eggleton Date: Mon, 30 Oct 2017 22:30:49 +0000 (+1300) Subject: devtool: upgrade: reformat --no-patch warning message X-Git-Tag: uninative-1.8~1132 X-Git-Url: https://code.ossystems.io/gitweb?a=commitdiff_plain;h=2d11e9e6e73648c1cb514c0c10111c7886acae78;p=openembedded-core.git 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 Signed-off-by: Ross Burton --- 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