From be4501c54b97d3f839ee4dc7563cb126b3dda2bf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lubom=C3=ADr=20Sedl=C3=A1=C5=99?= Date: Wed, 16 Aug 2017 13:12:58 +0200 Subject: [PATCH] gather: Fix duplicated log line MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Due to overwriting an existing variable the logs are getting duplicated line about missing comps packages instead of announcement of gathering being finished. Rename the variable to fix the problem. Signed-off-by: Lubomír Sedlář --- pungi/phases/gather/methods/method_deps.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pungi/phases/gather/methods/method_deps.py b/pungi/phases/gather/methods/method_deps.py index 432e9e9d..dcffde8e 100644 --- a/pungi/phases/gather/methods/method_deps.py +++ b/pungi/phases/gather/methods/method_deps.py @@ -157,11 +157,11 @@ def resolve_deps(compose, arch, variant): packages, broken_deps, missing_comps_pkgs = pungi_wrapper.parse_log(f) if missing_comps_pkgs: - msg = ("Packages mentioned in comps do not exist for %s.%s: %s" - % (variant.uid, arch, ", ".join(sorted(missing_comps_pkgs)))) - compose.log_warning(msg) + log_msg = ("Packages mentioned in comps do not exist for %s.%s: %s" + % (variant.uid, arch, ", ".join(sorted(missing_comps_pkgs)))) + compose.log_warning(log_msg) if compose.conf['require_all_comps_packages']: - raise RuntimeError(msg) + raise RuntimeError(log_msg) compose.log_info("[DONE ] %s" % msg) return packages, broken_deps