Merge lp:~zyga/lava-dashboard-tool/fix-pull into lp:lava-dashboard-tool/linaro-11.05

Proposed by Zygmunt Krynicki
Status: Merged
Merged at revision: 134
Proposed branch: lp:~zyga/lava-dashboard-tool/fix-pull
Merge into: lp:lava-dashboard-tool/linaro-11.05
Diff against target: 25 lines (+5/-3)
1 file modified
launch_control_tool/commands/dashboard.py (+5/-3)
To merge this branch: bzr merge lp:~zyga/lava-dashboard-tool/fix-pull
Reviewer Review Type Date Requested Status
Zygmunt Krynicki Pending
Review via email: mp+59445@code.launchpad.net

Description of the change

Fix a bug preventing pull from working in certain cases.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'launch_control_tool/commands/dashboard.py'
--- launch_control_tool/commands/dashboard.py 2011-04-28 12:27:14 +0000
+++ launch_control_tool/commands/dashboard.py 2011-04-29 00:33:23 +0000
@@ -748,7 +748,11 @@
748 local = self.server.streams()748 local = self.server.streams()
749 missing_pathnames = set([stream["pathname"] for stream in remote]) - set([stream["pathname"] for stream in local])749 missing_pathnames = set([stream["pathname"] for stream in remote]) - set([stream["pathname"] for stream in local])
750 for stream in remote:750 for stream in remote:
751 local_bundles = [bundle for bundle in self.server.bundles(stream["pathname"])]751 if stream["pathname"] in missing_pathnames:
752 self.server.make_stream(stream["pathname"], stream["name"])
753 local_bundles = []
754 else:
755 local_bundles = [bundle for bundle in self.server.bundles(stream["pathname"])]
752 remote_bundles = [bundle for bundle in self.remote_server.bundles(stream["pathname"])]756 remote_bundles = [bundle for bundle in self.remote_server.bundles(stream["pathname"])]
753 missing_bundles = set((bundle["content_sha1"] for bundle in remote_bundles)) - set((bundle["content_sha1"] for bundle in local_bundles))757 missing_bundles = set((bundle["content_sha1"] for bundle in remote_bundles)) - set((bundle["content_sha1"] for bundle in local_bundles))
754 try:758 try:
@@ -762,8 +766,6 @@
762 print "Stream %s needs update" % (stream["pathname"],)766 print "Stream %s needs update" % (stream["pathname"],)
763 else:767 else:
764 print "Stream %s is up to date" % (stream["pathname"],)768 print "Stream %s is up to date" % (stream["pathname"],)
765 if stream["pathname"] in missing_pathnames:
766 self.server.make_stream(stream["pathname"], stream["name"])
767 for content_sha1 in missing_bundles:769 for content_sha1 in missing_bundles:
768 print "Getting %s" % (content_sha1,),770 print "Getting %s" % (content_sha1,),
769 sys.stdout.flush()771 sys.stdout.flush()

Subscribers

People subscribed via source and target branches

to all changes: