From 2da92f250a0825e63206e644faed88f3f5054907 Mon Sep 17 00:00:00 2001 From: Nils Philippsen Date: Mon, 25 Nov 2019 18:11:30 +0100 Subject: [PATCH] don't print EDITCOMP messages for every package Setting data['is_active'] before checking if it's empty makes subsequent code always think things have changed. Signed-off-by: Nils Philippsen --- distgit_bugzilla_sync/script.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/distgit_bugzilla_sync/script.py b/distgit_bugzilla_sync/script.py index e7b61fb..2915bc4 100644 --- a/distgit_bugzilla_sync/script.py +++ b/distgit_bugzilla_sync/script.py @@ -346,12 +346,11 @@ class BugzillaProxy: data['initialcclist'] = initial_cc_emails break - data["is_active"] = not retired - if data: # Changes occurred. Submit a request to change via xmlrpc data['product'] = bz_product_name data['component'] = package + data["is_active"] = not retired if self.config["verbose"]: print(f'[EDITCOMP] {data["product"]}/{data["component"]}')