[Qa-jenkins-scm] [jenkins.debian.net] 01/01: reproducible debian: remote scheduler: de-duplicate the list later to keep the whole script functional

[Holger Levsen]

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 d4bcc0bfe31c49444746a372d74016486030f0f9
Author: Mattia Rizzolo <mattia at debian.org>
Date:   Sat Dec 3 16:51:47 2016 +0100

    reproducible debian: remote scheduler: de-duplicate the list later to keep the whole script functional
    
    otherwise the filter functions wouldn't work
    
    Signed-off-by: Holger Levsen <holger at layer-acht.org>
---
 bin/reproducible_remote_scheduler.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bin/reproducible_remote_scheduler.py b/bin/reproducible_remote_scheduler.py
index ff22e5c..47c345c 100755
--- a/bin/reproducible_remote_scheduler.py
+++ b/bin/reproducible_remote_scheduler.py
@@ -78,7 +78,7 @@ issue = scheduling_args.issue
 status = scheduling_args.status
 built_after = scheduling_args.after
 built_before = scheduling_args.before
-packages = set(x for x in scheduling_args.packages if x)
+packages = [x for x in scheduling_args.packages if x]
 artifacts = scheduling_args.keep_artifacts
 notify = scheduling_args.notify or scheduling_args.noisy
 notify_on_start = scheduling_args.noisy
@@ -163,7 +163,7 @@ query1 = """SELECT id FROM sources WHERE name='{pkg}' AND suite='{suite}'
 query2 = """SELECT p.date_build_started
             FROM sources AS s JOIN schedule as p ON p.package_id=s.id
             WHERE p.package_id='{id}'"""
-for pkg in packages:
+for pkg in set(packages):
     # test whether the package actually exists
     result = query_db(query1.format(pkg=pkg, suite=suite, arch=arch))
     # tests whether the package is already building

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