=== matsubara-afk is now known as matsubara === matsubara is now known as matsubara-afk [05:06] thumper around? [05:06] aye [05:06] thumper, can I get a code review for https://code.edge.launchpad.net/~rockstar/launchpad/remove-sourcepackage-from-recipe/+merge/25809 ? [05:06] It has it's two db reviews [05:07] ok [05:23] rockstar: done === mrevell is now known as mrevell-lunch === henninge changed the topic of #launchpad-reviews to: On Call: henninge || reviewing: sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === mrevell-lunch is now known as mrevell === gmb changed the topic of #launchpad-reviews to: On Call: henninge, gmb || reviewing: sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [14:01] Hi henninge. [14:01] * gmb keeps forgetting he's on the rota again [14:02] Hi gmb! ;) [14:02] bac and I never finished that conversation about changing my slot, so I am still here ;-) [14:07] :) === sinzui changed the topic of #launchpad-reviews to: On Call: henninge, gmb || reviewing: sinzui || queue: [sinzui] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [15:37] henninge, gmb, I just added a trivial branch the to review queue [15:38] sinzui, I'll take a look shortly; otp atm === gmb changed the topic of #launchpad-reviews to: On Call: henninge, gmb || reviewing: sinzui, sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [15:47] sinzui, The over-and-over branch, right? [15:50] gmb, yes [15:54] sinzui, r=me === gmb changed the topic of #launchpad-reviews to: On Call: henninge, gmb || reviewing: sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [15:55] fab === salgado is now known as salgado-lunch [16:47] gmb, henninge: could you please review this MP: https://code.edge.launchpad.net/~adeuring/launchpad/bug-511269/+merge/25971 ? (sorry, more than 900 lines...) [16:47] adeuring, Sure, I'll take a look [16:47] gmb: thanks! === gmb changed the topic of #launchpad-reviews to: On Call: henninge, gmb || reviewing: sinzui, adeuring || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === matsubara is now known as matsubara-lunch === Ursinha is now known as Ursinha-lunch [17:30] sinzui: review sent [17:30] thank === henninge changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: adeuring || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [17:34] adeuring, r=me; nice work! [17:34] gmb: thanks! === gmb changed the topic of #launchpad-reviews to: On Call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [17:45] gmb: could you please update the MP? [17:47] adeuring, Bugger. Yes, sorry. [17:47] gmb: ;) thanks! [17:47] adeuring, Done. === salgado-lunch is now known as salgado [18:06] danilos: did you have time to take another look at the POTemplate API branch and send it to ec2-test? [18:06] adiroiban, sorry, no [18:09] danilos: ok. and will you have time to discuss the pre-implementation details for bug 583934 and bug 583979 ? Not now, but before the start of June :) [18:09] Bug #583934: API for POFile attributes [18:13] adiroiban, yeah, I should have some time later this week [18:14] danilos: ok. just poke me when you have time . For both bugs I have added comments or pushed/attached a branch containing some ideas of how I see this can be done. [18:14] abentley, ping [18:16] adiroiban, sure [18:18] abentley, unping === Ursinha-lunch is now known as Ursinha === ajmitch_ is now known as ajmitch === kiko is now known as kiko-afk === matsubara is now known as matsubara-afk === salgado is now known as salgado-afk === kiko-afk is now known as kiko