[00:20] thumper: done [00:34] mwhudson: ta [01:07] * maxb has a merge, if someone's free: https://code.edge.launchpad.net/~maxb/launchpad/py2.6-importfascist-again/+merge/19927 [01:13] maxb: done [01:14] maxb: want to set a commit message? [01:14] oh, I always forget that [01:14] any reason not to just copy the commit message of the single revision, for single revision branches? [01:15] i guess that would be a sensible default [01:15] I've set that === jamalta is now known as jamalta-afk [02:40] thumper: hi, can you review a simple branch? [02:40] thumper: found a problem in the incremental import stuff, luckily it's super easy [02:41] maxb: your branch is in ec2 btw [02:41] (has been for a while) [02:41] mwhudson: sure [02:43] thumper: https://code.edge.launchpad.net/~mwhudson/launchpad/incremental-oops/+merge/19931 [02:44] thumper: otherwise the incremental stuff seems to be going ok [02:44] mwhudson: testing on staging? [02:45] although the setup part before it starts importing revisions is a bit long [02:45] mwhudson: done [02:46] so maybe we should up the number of revisions we import [02:47] hooray for QA i guess [02:47] maxb: actually i lied there [02:47] trying again... === henninge_ is now known as henninge [08:47] Hi henninge, did you get a chance to start/finish my MP? If not, I'll ask todays ocr to take a look. [08:48] noodles775: I did start and I will finish it this morning ...;-) [08:48] Great, thanks henninge ! [09:59] noodles775: review sent. === henninge 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 [09:59] Thanks henninge. === 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 === bigjools changed the topic of #launchpad-reviews to: on call: gmb || reviewing: - || queue [bigjools] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [11:23] an easy soyuz branch in the queue for ya gmb :) [11:23] bigjools: Ta. [11:23] https://code.launchpad.net/~julian-edwards/launchpad/publish-copy-archives-bug-520520-getPubConfig-proc-accept/+merge/19955 === gmb changed the topic of #launchpad-reviews to: on call: gmb || reviewing: bigjools || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [11:23] sorry for the terse cover note [11:23] I wrote a long one then my mail client crashed [11:23] I was too depressed to re-create it [11:24] so I am here to answer your questions [11:29] noodles775: can i bother you for a code and ui review for a small lazr-js change? 20 lines of code, and the ui review is quite trivial too [11:32] intellectronica: sure. [11:32] noodles775: lovely. sending an MP your way [11:40] noodles775: https://code.edge.launchpad.net/~intellectronica/lazr-js/choiceedit-position/+merge/19956 [11:45] intellectronica: thanks, I'll just finish off a branch for review, and then start it. [11:46] cheers [11:46] bigjools: No answers needed; it actually made sense for a change... r=me with some minor coding-style fixes [11:46] gmb: for a change? :) [11:46] thanks for the review [11:46] np [11:47] blame vim's python mode for the closing brace positioning :/ === noodles775 changed the topic of #launchpad-reviews to: on call: gmb || reviewing: bigjools || queue [noodles775] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [12:11] Hi gmb, this one is pure test refactoring, and is just over 800lines: https://code.edge.launchpad.net/~michael.nelson/launchpad/ppa-privatisation-test-refactor4/+merge/19961 [12:11] Eeesh. [12:11] noodles775: Okay, I'll take a look === gmb changed the topic of #launchpad-reviews to: on call: gmb || reviewing: noodles775 || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [12:11] seems like the week of the long branches. [12:12] Yeah, and it's the 4th pipe in the test refactoring! Thanks gmb, let me know if you want me to move some of the changes to the next branch to make it more manageable. === matsubara-afk is now known as matsubara [12:13] noodles775: I like how "just over" means "+11%" ;) [12:13] noodles775: But no, it's fine. I'll take care of it. [12:13] gmb: thanks. === daniloff is now known as danilos [12:50] noodles775: r=me === gmb changed the topic of #launchpad-reviews to: on call: gmb || reviewing: lunch || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [12:50] Thanks gmb [12:52] intellectronica: I can see from the code what your lazr-js branch is meant to do, but when testing it in a browser, I can find a way to show the difference between trunk and your branch? [12:53] noodles775: i'm not sure i understand the question. do you mean that it behaves in the same way the version in trunk behaves? [12:53] noodles775: and if so, could it be a cache issue? [12:53] intellectronica: yes, with the two side-by-side, I haven't found a way to ... [12:53] ah... [13:01] intellectronica: http://people.canonical.com/~michaeln/tmp/intellectronica.ogv [13:02] intellectronica: forget that... [13:03] I just saw the issue (my end, too many tabs) [13:04] noodles775: looks like it's working correctly. some of the entries you choose cause the box to go out of screen, and that's why the position is not as expected. as i explained in the MP, i can't think of a better way of handling that situation [13:05] i wish i could cause the mouse pointer to move [13:05] intellectronica: yeah, I was just thinking the same thing ;) [13:10] intellectronica: code-wise, any reason for not using boundingBox.query('span.current') instead of iterating through all the spans? [13:10] noodles775: no, i just didn't think of that. let me give that a try. [13:12] noodles775: yeah, that works fine. changing. [13:13] Great, r=me. [13:13] noodles775: thanks! [13:20] Does anyone know if ec2test is having problems? [13:20] jpds: Twice in the past few weeks I've had an ec2 run not come back (gets stalled during windmill and then I have to terminate it). But I've just rerun them and it worked fine. [13:21] * noodles775 -> lunch [13:21] Two branches of mine were approved yesterday and still haven't landed. :/ === bac changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: lunch || 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: lunch,- || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [13:24] jpds: do you need me to re-submit them or will your reviewer do it? [13:25] bac: They're https://code.launchpad.net/~jpds/launchpad/fix_517839/+merge/19774 and https://code.launchpad.net/~jpds/launchpad/fix_520469/+merge/19890 [13:25] jpds: ok [13:29] bac: trivial branch: https://code.edge.launchpad.net/~leonardr/lazr.restful/prep-for-release/+merge/19968 [13:30] leonardr: ok, i'll get right on it === bac changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: lunch,leonard || 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: lunch,- || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [13:38] leonardr: that really was trivial [13:38] bac: yes, but gary dings me if i commit to lazr.* without an r= [13:38] and i agree with him [13:38] glad to help [13:51] jpds: both sent off. cross your fingers [13:52] bac: "What could possibly go wrong?" [13:52] nothing i can think of! [13:55] gmb, bac: could one of you do review this mp: https://code.edge.launchpad.net/~adeuring/launchpad/bug-513382/+merge/19970 (should be really simple) [13:56] noodles775: could you ui--review this ^^^^branch? [13:56] adeuring: yes [13:56] bac: thanks === bac changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: lunch,abel || 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,bac || reviewing: - , abel || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [14:01] intellectronica, if you're around, i need one more review of https://code.edge.launchpad.net/~leonardr/launchpad/multiversion-integration [14:01] leonardr: ok [14:01] intellectronica: i'm preparing an incremental diff now [14:02] leonardr: thanks [14:10] intellectronica: http://pastebin.ubuntu.com/382273/ [14:10] writing up the changes now [14:11] leonardr: why the change in doc/menus.txt? [14:11] oh, ok, i'll wait then :) [14:14] intellectronica:https://code.edge.launchpad.net/~leonardr/launchpad/multiversion-integration/+merge/19346 [14:15] leonardr: so my question (above) still stands [14:15] intellectronica: what's missing from the explanation i gave? [14:16] oh, menus.txt [14:16] yes, i don't understand why the error changed [14:16] sorry, i thouht you were talking about folder.txt [14:16] i'll write another comment [14:16] cool [14:18] intellectronica: added [14:19] ah, got it [14:19] leonardr: r=me [14:19] ok, time to test it and hopefully see no more failures === jamalta-afk is now known as jamalta [14:27] Hi adeuring, I can, but it'd be great to first request a UI review from one of the people with an asterisk at: https://dev.launchpad.net/UI/Reviews - I can then do the second UI review. [14:27] This'll help us build up more ui reviewers. [14:27] But if it's urgent, I'll do it straight away. [14:27] noodles775: sure, but this one is really trivial... [14:27] adeuring: ok, I'll do it straight away then :) [14:32] adeuring: wow, the reviews today keep getting more trivial === bac changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: - , - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [14:36] adeuring: I think you've got the wrong bug linked on your MP ;) ui=me, thanks! [14:37] adeuring: I think you've got the wrong bug linked on your MP ;) ui=me, thanks! [14:42] noodles775: thanks! and right, the bug is 513380... === deryck is now known as deryck[lunch] === salgado is now known as salgado-lunch === matsubara is now known as matsubara-lunch === bac changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: - , jtv || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [16:20] hello gmb, here's a branch that remedies the merge conflict between stable and db-devel, https://code.edge.launchpad.net/~al-maisan/launchpad/merge-conflict/+merge/19981 [16:20] Could you please have a look at it? [16:20] al-maisan: Sure [16:20] thanks! === gmb changed the topic of #launchpad-reviews to: on call: gmb,bac || reviewing: al-maisan, jtv || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [16:21] al-maisan: Is there some extra data in the diff? It looks like there's other changes in there besides conflict resolutions. [16:22] gmb: I actually merged stable into db-devel and then resolved the conflicts .. that's what you're seeing.. [16:22] Ah. [16:26] al-maisan: Looks fair enough. r=me [16:26] gmb: thank you very much! [16:26] np === gmb changed the topic of #launchpad-reviews to: on call: bac || reviewing: jtv || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === deryck[lunch] is now known as deryck [16:49] bac, noodles775: could you please re-review my MP https://code.edge.launchpad.net/~adeuring/launchpad/bug-513382/+merge/19970 ? [16:50] adeuring: i will. i'm in the middle of a long one and my lunch is fast approaching. i'll do it later if that's ok [16:50] bac: sure, thanks === jamalta is now known as jamalta-afk [16:54] adeuring: I'll do it first thing tomorrow (as I'm off for dinner in a tick). [16:54] noodles775: thanks! === matsubara-lunch is now known as matsubara === salgado-lunch is now known as salgado [17:50] rockstar, do you want to review https://code.edge.launchpad.net/~leonardr/lazr.restful/register-operations-when-version-list-is-known/+merge/19985 ? [17:52] leonardr, well, I'm a bit tied up at the moment. You might be better served having bac look at it. [17:53] rockstar: sure, just wanted to give you right of first refusal since it's multiversion [17:53] bac: -^ [17:55] bac: sorry for the extensive drive-by damage :) === gary_poster is now known as gary-lunch [19:35] hello bac, this branch resolves the "stable -> db-devel" merge conflicts, https://code.edge.launchpad.net/~al-maisan/launchpad/merge-conflicts-2/+merge/19994 [19:35] Can you please take a look? [19:36] al-maisan: is this different from the similar branch you did earlier today that gmb looked at? [19:36] it is similar in the sense that it resolves the merge conflict [19:37] hrmpff .. the diff generation takes ages [19:37] bac: the diff is available now [19:38] al-maisan: the diff isn't available here yet. propogation delay [19:38] and now i have it [19:38] great! [19:39] al-maisan: done [19:39] thanks muharem [19:39] bac: thank you! [19:53] bac: can you review https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-526001-edit-packaging-oops/+merge/19997 [19:55] EdwinGrubbs: yes. please add to the queue === EdwinGrubbs changed the topic of #launchpad-reviews to: on call: Edwin || reviewing: jtv || queue [adiroiban(bug-509252-take-2), Edwin(bug-526001)] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [20:01] leonardr: i didn't see your request. i'll put it on the queue where it would've landed === bac changed the topic of #launchpad-reviews to: on call: Edwin || reviewing: - || queue [adiroiban(bug-509252-take-2), leonard, Edwin(bug-526001)] || 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: bac || reviewing: abel || queue [adiroiban(bug-509252-take-2), leonard, Edwin(bug-526001)] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [20:05] EdwinGrubbs: if you have a moment could you look at https://code.edge.launchpad.net/~adeuring/launchpad/bug-513382/+merge/19970 since it involves sprites? === gary-lunch is now known as gary_poster [20:15] adiroiban: i'd prefer to let henninge re-review your changes === bac changed the topic of #launchpad-reviews to: on call: bac || reviewing: leonard || queue [adiroiban(bug-509252-take-2), leonard, Edwin(bug-526001)] || 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: bac || reviewing: leonard || queue [adiroiban(bug-509252-take-2), Edwin(bug-526001)] || 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: bac || reviewing: edwin || queue [adiroiban(bug-509252-take-2)] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [20:21] EdwinGrubbs: you didn't add a test case to show an obsolete productseries [20:22] bac: I can add that. [20:22] EdwinGrubbs: thanks [20:34] rockstar: https://code.edge.launchpad.net/~thumper/launchpad/descriptions-for-merge-proposals/+merge/19564 [20:51] thumper, could you review https://code.launchpad.net/~abentley/launchpad/restricted-diffs/+merge/19999 ? [20:51] oooh 19999 [20:51] abentley: yes [20:52] thumper, thanks. === salgado is now known as salgado-afk === matsubara is now known as matsubara-afk === bac changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [adiroiban(bug-509252-take-2)] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === sinzui changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [adiroiban(bug-509252-take-2), sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [22:12] bac: looking at adeuring's branch, the only info that couldn't be displayed if he switched it to sprites would be the alt tag on the image. The size of the sprite is no problem. [22:13] EdwinGrubbs: would you mind adding that to the MP? [22:13] bac: no problem [23:23] thumper: around to review a branch? [23:27] https://code.edge.launchpad.net/~mwhudson/launchpad/incremental-save-all-revs/+merge/20020 if you or anyone else fancies it [23:59] * thumper looks