[Qa-jenkins-scm] [jenkins.debian.net] 04/07: reproducible Debian: no need for a dictionary; let's just use a set.

Holger Levsen holger at layer-acht.org
Mon Nov 7 17:13:30 UTC 2016


This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit bf3fb6c2bb546ea05aca8e37e7b72a2134946d5e
Author: Chris Lamb <lamby at debian.org>
Date:   Mon Nov 7 13:25:37 2016 +0000

    reproducible Debian: no need for a dictionary; let's just use a set.
    
    Signed-off-by: Holger Levsen <holger at layer-acht.org>
---
 bin/reproducible_create_meta_pkg_sets.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bin/reproducible_create_meta_pkg_sets.sh b/bin/reproducible_create_meta_pkg_sets.sh
index e0836cc..4662186 100755
--- a/bin/reproducible_create_meta_pkg_sets.sh
+++ b/bin/reproducible_create_meta_pkg_sets.sh
@@ -33,13 +33,13 @@ import sys
 import yaml
 with open(sys.argv[1]) as fd:
 	manifest = yaml.load(fd)
-	seen = {}
+	seen = set()
 	try:
 		for pkg in (manifest['packages']['binary'] + manifest['packages']['source']):
 			pkgname = pkg['package']
 			if pkgname not in seen:
 				print(pkgname, end='|')
-				seen[pkgname] = True
+				seen.add(pkgname)
 	except Exception as exc:
 		print("Warning: something went wrong while parsing the build manifest as YAML file: {}".format(exc))
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git



More information about the Qa-jenkins-scm mailing list