[Merge] lp:~amoog/ubuntu-dev-tools/fix-896884 into lp:ubuntu-dev-tools

Stefano Rivera launchpad at rivera.za.net
Sun Nov 27 18:20:29 UTC 2011


I tested on a bug in staging:

sponsor-patch: I: Set bug #851798 status to Confirmed.
sponsor-patch: I: Acknowledged sync request bug #851798.
sponsor-patch: I: Subscribed ubuntu-archive to bug #851798.
sponsor-patch: I: Subscribed me to bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: I: Couldn't unsubscribe ubuntu-sponsors from bug #851798.
sponsor-patch: Successfully acknowledged sync request bug #851798.

That looks wrong :)

Oh, and the rest of sponsor_patch isn't using lpapicache, so we probably shouldn't here.
How about just storing launchpad.people['ubuntu-sponsors'] in a variable before the loop?
-- 
https://code.launchpad.net/~amoog/ubuntu-dev-tools/fix-896884/+merge/83514
Your team Ubuntu Development Team is requested to review the proposed merge of lp:~amoog/ubuntu-dev-tools/fix-896884 into lp:ubuntu-dev-tools.



More information about the Ubuntu-reviews mailing list