ubuntuone-dev-tools

Merge lp:~dobey/ubuntuone-dev-tools/fix-689851 into lp:ubuntuone-dev-tools

  1. fix-689851
  2. Merge into trunk
Proposed by dobey
Status: Merged
Approved by: Stuart Colville
Approved revision: 14
Merged at revision: 14
Proposed branch: lp:~dobey/ubuntuone-dev-tools/fix-689851
Merge into: lp:ubuntuone-dev-tools
Diff against target: 13 lines (+3/-1)
1 file modified
bin/u1trial (+3/-1)
To merge this branch: bzr merge lp:~dobey/ubuntuone-dev-tools/fix-689851
Reviewer Review Type Date Requested Status
Stuart Colville (community) Approve
Natalia Bidart (community) Approve
Review via email: mp+43571@code.launchpad.net

Commit message

Iterate through the required services and only append if not already in the list, rather than doing list.extend()

To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) :
review: Approve
Revision history for this message
Stuart Colville (muffinresearch) wrote :

Looks good

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/u1trial'
--- bin/u1trial 2010-11-22 18:57:18 +0000
+++ bin/u1trial 2010-12-13 20:39:25 +0000
@@ -74,7 +74,9 @@
74 for member_type in members:74 for member_type in members:
75 if hasattr(member_type, 'required_services'):75 if hasattr(member_type, 'required_services'):
76 member = member_type()76 member = member_type()
77 self.required_services.extend(member.required_services())77 for service in member.required_services():
78 if service not in self.required_services:
79 self.required_services.append(service)
78 del member80 del member
79 gc.collect()81 gc.collect()
8082

Subscribers

People subscribed via source and target branches

to all changes: