/srv/irclogs.ubuntu.com/2010/06/01/#launchpad-reviews.txt

=== matsubara-afk is now known as matsubara
=== matsubara is now known as matsubara-afk
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
adeuringgmb: could you please review this mp: https://code.edge.launchpad.net/~adeuring/launchpad/bug-586808/+merge/26480 ?11:37
gmbadeuring, Sure11:37
adeuringthanks!11:37
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: adeuring || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
gmbadeuring, I think the description of the new status could be rephrased. How about something like: "The bug remains open for discussion only. This status is usually used where there is disagreement over whether the bug is relevant to the current target and whether it should be fixed."11:43
gmbadeuring, I'm also assuming that the work for ACL'ing the OPINION status so that it can't be unset except by the bug supervisor is going to happen in another branch (or not at all; I haven't been keeping up with the LEP on this)11:44
adeuringgmb: thanks for the suggestion for the description! In a discussion with deryck we decided to _not_ ACL the status11:49
gmbadeuring, Righto. r=me with that change, then.11:50
adeuringwe think it is better to see if people will really accep the new status when everybody can change it.11:50
adeuringgmb: thanks!11:50
gmbAgreed.11:50
=== matsubara-afk is now known as matsubara
=== Ursinha_ is now known as Ursinha
matsubaragmb, hi13:53
matsubaragmb, can you review a RC candidate for me: https://code.launchpad.net/~matsubara/launchpad/bug-504124-oops-handling-email/+merge/26457?13:53
gmbmatsubara, Sure13:59
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: matsubara || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
matsubarathanks gmb14:00
=== NCommand1r is now known as mcasadevall
gmbmatsubara, r=me14:04
matsubaragmb, thank you!14:04
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== bac changed the topic of #launchpad-reviews to: On Call: gmb, bac || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bacgood morning gmb.  just like old times!14:30
gmbbac, Indeed :)14:31
=== henninge_ is now known as henninge
=== mcasadevall is now known as NCommander
=== noodles775 changed the topic of #launchpad-reviews to: On Call: gmb, bac || reviewing: - || queue: [noodles] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
noodles775Hi guys... not an RC, but when you've time: https://code.edge.launchpad.net/~michael.nelson/launchpad/588288-log-parser-dont-read-entire-file/+merge/2649815:35
noodles775gmb, bac: I'll be around for another hr if you want ask any questions, otherwise, do you mind doing it non-interactive?15:44
gmbnoodles775, Sure, no problem. I'll take a look presently.15:45
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb, bac || reviewing: noodles775, - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
noodles775Thanks gmb.15:45
gmbnoodles775, Is 10000 an arbitrary value? (I'm assuming that won't cause any problems, but it's worth asking)15:59
noodles775gmb: yes, it's arbitrary - it won't even be set in lp-production-configs initially (we'll add it for one machine, germanium).16:02
gmbnoodles775, Fair enough. r=me16:02
noodles775Thanks gmb.16:02
=== deryck is now known as deryck[lunch]
abentleyrockstar, would you mind doing some reviews for me?16:27
=== Ursinha is now known as Ursinha-lunch
=== gary_poster is now known as gary-lunch
=== deryck[lunch] is now known as deryck
leonardrgmb, bac, a non-urgent branch for you to review17:12
leonardrhttps://code.edge.launchpad.net/~leonardr/launchpad/test-representation-cache/+merge/2651317:12
=== gmb changed the topic of #launchpad-reviews to: On Call: gmb, bac || reviewing: -, - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
gmbbac, Fancy taking leonardr's branch? It's quite close to EOD for me now.17:13
rockstarabentley, shoot, didn't see your pings. Yes, yes I an.17:15
=== salgado is now known as salgado-lunch
=== gmb changed the topic of #launchpad-reviews to: On Call: bac || reviewing: - || queue: [leonardr] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
gmbleonardr, I've put your branch in the queue for when bac returns.17:20
leonardrgmb, tx17:20
=== matsubara is now known as matsubara-lunch
=== gary-lunch is now known as gary_poster
=== matsubara-lunch is now known as matsubara
bachi leonardr, i'll look at your branch now.18:50
=== bac changed the topic of #launchpad-reviews to: On Call: bac || reviewing: leonardr || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
leonardrbac, thanks18:51
=== salgado-lunch is now known as salgado
abentleyrockstar, could you review https://code.edge.launchpad.net/~abentley/launchpad/daily-builds/+merge/26232 ?19:07
rockstarabentley, ack. On it.19:08
abentleyrockstar, thanks.19:08
=== leonardr is now known as leonardr-afk
=== Ursinha-lunch is now known as Ursinha
abentleybac, could you review https://code.launchpad.net/~abentley/launchpad/poppy-isolation/+merge/26531 please?20:02
bacabentley: sure20:03
=== abentley changed the topic of #launchpad-reviews to: On Call: bac || reviewing: leonardr || queue: [abentley] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bacabentley: done20:05
=== bac changed the topic of #launchpad-reviews to: On Call: bac || reviewing: - || queue: [-] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
abentleybac, thanks.20:11
=== leonardr-afk is now known as leonardr
abentleybac, could you please review https://code.launchpad.net/~abentley/launchpad/binary-to-recipe/+merge/26537 ?21:42
=== matsubara is now known as matsubara-afk
=== bac changed the topic of #launchpad-reviews to: On Call: - || reviewing: - || queue: [-] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== salgado is now known as salgado-afk

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!