=== slank is now known as slank_away === Ursinha-afk is now known as Ursinha === Ursinha is now known as Ursinha-afk [01:16] bigjools: if they paste the error they are having they might get help! [01:17] lifeless: well, quite :) [08:50] good morning [10:57] Morning, all. [12:04] morning deryck [12:04] adeuring: do you have time to do a hangout on this zcml fun please? [12:04] rick_h_: sure [12:59] adeuring: woot! all the product tests passed. So back to my own new tests and making this work. Thanks! [12:59] rick_h_: welcome :) === Ursinha-afk is now known as Ursinha [13:16] jam, mgz: I don't understand the issue with this import to help the user. Can you help? https://answers.launchpad.net/launchpad/+question/215746 === yofel_ is now known as yofel === rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: rick_h | Firefighting: - | Critical bugs: ~170 [14:25] adeuring: can you please review this branch we chatted about then since I'm OCR? https://code.launchpad.net/~rharding/launchpad/changing_maintainer_1077841/+merge/137849 [14:26] rick_h_: sure [14:26] thanks [14:30] ricr=me [14:34] rick_h_, do you have time to review https://code.launchpad.net/~sinzui/launchpad/cleanup-merged-people-1/+merge/137700 [14:42] sinzui: will do after call === slank_away is now known as slank [15:27] sinzui: r=me [15:28] thank you rick_h_ [15:43] rick_h_: could you please review this mp: https://code.launchpad.net/~adeuring/launchpad/bug-1086043/+merge/137882 ? [15:44] sure thing adeuring [15:44] htanks! === matsubara is now known as matsubara-lunch [15:56] I'm having a go at fixing bugs in loggerhead. I'm trying to run serve-branches but all I get from my browser is: The resource could not be found. Anyone able to offer a couple of minutes to help me work out what I'm doing wrong? [15:58] mattyw: when I last ran it I could only get it to work against a single branch and not against a direcotry of them [16:00] rick_h_, from within my loggerhead source directory I do ./serve-branches (no args) correct? I believe that should give me one branch [16:05] ok, what I had to do to get it working was install it as a bazaar plugin, and then use bzr serve --http [16:07] mattyw: https://code.launchpad.net/~rharding/loggerhead/yui3.7.1 is the branch I had going to work on YUI update there and I had to go that plugin route to get it to run/verify [16:09] adeuring: one question and one comment fix on your MP [16:10] adeuring: your tests pass though, so maybe I'm mis-understanding things which is why you didn't have to move the displayname to the new interface as well? [16:10] * adeuring is looking [16:12] rick_h_: I am pretty sure that displayname will be needed to fix bug 1086057. But the bug listings do not show the displyname; the series's name is needed in the call of canonucal_url() [16:12] <_mup_> Bug #1086057: Users with an artifact grant for a bug related to a private product can't view the bug page. < https://launchpad.net/bugs/1086057 > [16:14] rick_h_: but for now, I'd like to keep displayname protected by lp.View -- simply because it is not needed in the bug listing [16:14] adeuring: right, so you moved displayname in the zcml from one block to the other. Then you moved name into IProductSeriesLimitedView, but not displayname [16:14] adeuring: ah ok gotcha === gary_poster is now known as gary_poster|away [16:14] rick_h_, thanks that seems to work [16:15] rick_h_: argh, yes, no I get the oddity. let me check again [16:18] rick_h_: milestone.displayname needs lp.LimitedView for bug listing page, but productseries.displayname is not needed there... As said, I'll probably have to use lp.LimitedView for productseries.dispalyname in my next brnach [16:18] adeuring: right ok. so r=me just wanted to dbl check there [16:19] zcml turning my brain to mush today [16:19] rick_h_: thanks. YOur question made sense -- it was probably not the best idea to change the permission for two different classes in one branch ;) === gary_poster|away is now known as gary_poster [16:25] rick_h_, hi, would you take a look at the extra revisions on https://code.launchpad.net/~james-w/python-oops-tools/recent-oopses/+merge/137055 please? [16:25] they are to fix the tarmac test failure under python2.6 [16:29] rick_h_, do you have time to review https://code.launchpad.net/~sinzui/launchpad/revision-karma-2/+merge/137906 [16:30] * sinzui looks at james_w's branch [16:30] thanks sinzui [16:30] ah, I see. r-=me [16:31] james_w, r=me. [16:31] sinzui, merci [16:36] sinzui: sure thing looking [16:44] sinzui: r=me [16:44] thank you rick_h_ === matsubara-lunch is now known as matsubara === deryck is now known as deryck[lunch] === almaisan-away is now known as al-maisan [17:26] rick_h_, me again, I thought I'd look into this bug https://bugs.launchpad.net/loggerhead/+bug/358293. But I can't seem to even find the search box displayed anywhere. [17:26] <_mup_> Bug #358293: Search field is too narrow < https://launchpad.net/bugs/358293 > [17:28] mattyw: hmm, wonder if it's not displayed in a single branch view === al-maisan is now known as almaisan-away [17:33] rick_h_, hmmm, looks like this is what I see https://bugs.launchpad.net/loggerhead/+bug/1075907 [17:33] <_mup_> Bug #1075907: Loggerhead - serving two directories from parent < https://launchpad.net/bugs/1075907 > [17:34] mattyw: right [17:37] sinzui: forced another build-not build [17:42] oh, so did I [17:42] heh, crush the buildbot! === deryck[lunch] is now known as deryck === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [20:09] deryck: Hi, did the definitional issue that I raised last night get sorted out before we announced private projects were out of beta? [20:09] I suspect we've released a definition of privacy that we can't support. [20:09] Specifically, AIUI we say that series and milestones are private from people who can only see a bug, when that's impossible. === rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~170 [20:29] hi wgrant. It's not landed yet, but abel has a branch nearly done. Should be landing during his tomorrow. [20:29] Ah, great. [20:30] wgrant, thanks for catching that and letting us know. [20:30] It just reveals some details of milestones and series? [20:30] yeah [20:30] well, and a list 403s [20:30] I mean, how does the fix fix it? [20:31] wgrant, that was the yeah, and then I thought you meant something else. sorry. :) [20:31] Right :) [20:31] Thanks [20:31] wgrant, we fix it with limited view to reveal the name of the milestones and series. === gary_poster|away is now known as gary_poster