From df4782da0b907e08d13ce09d07d4941728563a90 Mon Sep 17 00:00:00 2001 From: Fabian Vogt Date: Mon, 19 Jun 2023 10:00:52 +0200 Subject: [PATCH] skippkg-finder.py: Handle empty configuration values properly Splitting an empty string with ' ' results in list(""), which the code does not handle properly. Instead use just split() which returns an empty list() insted. --- skippkg-finder.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/skippkg-finder.py b/skippkg-finder.py index 12d93b11..ac5e8ea6 100755 --- a/skippkg-finder.py +++ b/skippkg-finder.py @@ -38,14 +38,14 @@ class SkippkgFinder(object): # binary rpms of packages from `skippkg-finder-skiplist-ignores` # be found in the `package_binaries` thus format must to be like # SUSE:SLE-15:Update_libcdio.12032, PROJECT-NAME_PACKAGE-NAME - self.skiplist_ignored = set(config.get('skippkg-finder-skiplist-ignores', '').split(' ')) + self.skiplist_ignored = set(config.get('skippkg-finder-skiplist-ignores', '').split()) # supplement RPMs for skipping from the ftp-tree - self.skiplist_supplement_regex = set(config.get('skippkg-finder-skiplist-supplement-regex', '').split(' ')) + self.skiplist_supplement_regex = set(config.get('skippkg-finder-skiplist-supplement-regex', '').split()) # drops off RPM from a list of the supplement RPMs due to regex - self.skiplist_supplement_ignores = set(config.get('skippkg-finder-skiplist-supplement-ignores', '').split(' ')) + self.skiplist_supplement_ignores = set(config.get('skippkg-finder-skiplist-supplement-ignores', '').split()) # conditional support scenario - self.skiplist_conditionals = set(config.get('skippkg-finder-conditional-scenarios', '').split(' ')) + self.skiplist_conditionals = set(config.get('skippkg-finder-conditional-scenarios', '').split()) def is_sle_specific(self, package): """