[03:25] lifeless: want to eyeball a boring as crap branch which just moves errors from canonical.launchpad.webapp.interfaces into lp.app.errors? [03:25] no [03:25] +1 [03:26] rs? [03:26] yes [03:26] you break it you bought it [03:26] ack [03:32] lifeless: 2542 lines (+256/-235) 147 files modified :-) [03:32] not that app.errors is really any better, but I get the psych value of it ;) [03:37] lifeless: well, lp.app is where much of canonical.launchpad.webapp is going [03:37] yeah, I know [03:37] just saying that A.bigbag == B.bigbag :) [03:37] actually... [03:37] we are breaking up a general interfaces big bag into two [03:37] errors and interfaces [03:37] so in a way, it is being a lot broken up [03:38] \o/ [03:38] http://bazaar.launchpad.net/~lifeless/launchpad/malone/revision/11225 [03:38] thats an incremental patch on my make-search-faster branch [03:38] anyone care to ok it ? [03:39] lifeless: where is the merge proposal? [03:39] https://code.edge.launchpad.net/~lifeless/launchpad/malone/+merge/30904 [03:39] the proposal is approved [03:39] but ec2land told me I was foolishly hopeful [03:42] lifeless: why tweak sample data instead of adding a bug with the factory [03:42] ? [03:42] it raises my ick factor [03:43] thumper: minimal change to work in with the existing test [03:43] thumper: adding a new bug causes the rest of the entire doctest to go boom [03:43] :( [03:43] indeed [03:44] lifeless: commented. [03:45] thanks [05:59] lifeless: Nice easy DB review for you: https://code.edge.launchpad.net/~wgrant/launchpad/per-archive-build-debug-symbols/+merge/29671 [06:03] mmm [06:03] is archive really the right scope for this ? [06:04] its a bit of a shame all our config knobs are spread all over [06:04] * lifeless tries not to use flags for everything [06:06] lifeless: What would be a better scope? [06:07] packageset-in-archive ? [06:07] I dunno, it just seems rather coarse, or something. [06:07] It is coarse, yes. [06:07] Packagesets are an interesting idea, but they're not workable yet. [06:09] wgrant: dbgsyms is an unnecessary abbreviation in at least one place in the patch [06:10] its also a redundant comment in that place too [06:10] True. [06:10] given the variable name is pretty clear [06:10] This branch is a year old, and the terminology throughout the rest of the branches has changed since then. [06:10] It is probably removable indeed. [06:11] please don't use the string literal there [06:11] for commercial-admins [06:11] rather use the sampledata module [06:12] Hm. Does it really only have one value at the moment? [06:12] Or is there another one I'm missing somewhere? [06:12] its pretty small right now === wgrant changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [wgrant(http://bit.ly/dvBcHk)] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === stub1 is now known as stub [12:30] I'm reviewing james_w's sampledata-0 branch btw, === mrevell is now known as mrevell-lunch === matsubara-afk is now known as matsubara [13:32] bigjools, getting to your branch now. sorry about the delay. [14:14] bigjools, done. [14:14] jml: you rock, thanks [14:15] bigjools, my pleasure. [14:15] jml: the next branch will clean up these damn tests I hope [14:16] the "if in doubt return a Deferred" approach is a good one [14:18] jml: BTW there's nowhere in the guidelines that says we use US spelling in code === mrevell-lunch is now known as mrevell [14:18] :( [14:18] you mean: :) [14:18] bigjools, no, I don't. [14:18] lol [14:19] I hate having to think unnecessarily when reading code [14:19] I prefer "international English" [14:19] me too [14:19] color [14:19] that's why I go out of my way to make the English agnostic [14:19] color, never ever ever colour [14:19] the mighty British Empire says otherwise [14:20] what a pity they didn't invent C [14:20] ha [14:20] FWIW "dialog" to me means a popup. [14:20] a dialogue is a conversation [14:21] if you spell it slaveDialogueEnded in the code it's going to look weird to lots of people and eventually someone is going to change it to be slaveDialogEnded [14:22] * bigjools sighs heavily [14:23] feel sorry for the French [14:24] I bet they have discussions about whether it should be self.ordinateur or self.la_ordinateur [14:27] jml: ok, so slavePowowEnded it is [14:29] thanks jml. Just a couple of questions in https://code.edge.launchpad.net/~james-w/launchpad/no-more-sampledata-0/+merge/31470 [14:29] jml: plus there's a nice juicy Collection branch sitting there :-) [14:40] bigjools, "prefer the standard to the offbeat", also, the OED spells it powwow or pow-wow :P [14:43] jml: grumble...smartarse :) [14:44] does the OED carry US spelling? [14:44] I need a review of a small patch to lazr.restful: https://code.edge.launchpad.net/~benji/lazr.restful/nail-versions/+merge/31421 [14:44] it just nails some versions so the tests pass again [14:45] bigjools, yes. main articles are in real English, but it is always explicit when there are regional spelling variations. [14:46] jml: so if I use slaveConversationEnded will that satisfy your reviewing juices? [14:46] bigjools, yes. [14:46] hurray === abentley changed the topic of #launchpad-reviews to: On call: abentley || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [14:56] james_w: replied. I might look at the archive-collection thing. [14:57] jml: no rush [15:26] jml: reply incoming [15:27] jml: btw thanks for the collections explanation. I think I get it, but since I'm more of a JFDI than a theoretical kind of man I think more concrete examples will help. In due course. [15:30] bigjools, np. [15:30] bigjools: have you seen https://code.edge.launchpad.net/~james-w/launchpad/archive-collection/+merge/31499 ? [15:32] I did it because I wanted something very similar to getArchivesForDistribution in the deletion branch, but with one more collection. While you can continue to add more methods to ArchiveSet with the requirements of every caller, letting them construct their query is more flexible. [15:43] james_w: yes, right. Does it build up the same query or issue multiple queries? [15:43] one query [15:44] the shape of that query may not be optimal, but that's the case with any new or modified queries === deryck is now known as deryck[lunch] === Ursinha is now known as Ursinha-lunch [16:15] jml: I don't understand why you want IBuilder.getActiveBuild [16:16] oh scratch that [16:16] I can't fucking read [16:16] bigjools, np. otp right now :) [16:20] jml: FWIW it also returns a proxied object and that is force of habit for me :) anyway ... [16:38] jml: just to clarify, when you say "test_returnsAll... style rather than testReturnsAll style" do you mean "test_returnsAll_does_something" or "test_returnsAllDoesSomething" ? [16:39] james_w, test_properName_does_a_thing [16:39] jml: ok [17:01] james_w, archivecollection reviewed. === salgado is now known as salgado-lunch === matsubara is now known as matsubara-lunch [17:20] I need a review of a small patch to lazr.restful: https://code.edge.launchpad.net/~benji/lazr.restful/nail-versions/+merge/31421 === Ursinha-lunch is now known as Ursinha === matsubara-lunch is now known as matsubara === salgado-lunch is now known as salgado === matsubara is now known as matsubara-afk === mwhudson_ is now known as mwhudson === abentley changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === salgado is now known as salgado-afk === Ursinha is now known as Ursinha-afk