[03:13] <achiang> poolie: thanks for the review
[03:14] <poolie> thanks for the patches!
[03:14] <achiang> poolie: np. i'll respond more in the MP's themselves, but the 1st MP, where i stuff the enums into a class...
[03:14] <achiang> poolie: that's because i don't really know python and was just faking it. ;)
[03:14] <poolie> i wondered :)
[03:14] <poolie> it's ok though
[03:14] <achiang> poolie: the only goal was to have a place to hold the enums so i can print them out
[03:15] <achiang> there were no global variables earlier
[03:15] <poolie> python has a somewhat uncommon setup where there is one namespace for 'variables' and 'classes/functions'
[03:15] <poolie> the latter just being variables that point to an object
[03:15] <achiang> or rather, the list of enums wasn't globally accessible
[03:15] <poolie> so
[03:15] <poolie> for static data like 'valid status values' having a global that points to it is really harmless
[03:16] <achiang> poolie: good enough for me. i can just make 3 global lists then... (valid status, valid importance, valid filters) ?
[03:16] <poolie> oh, what would the filters be?
[03:16] <poolie> like 'filter by tags, etc'?
[03:16] <achiang> right
[03:16] <achiang> poolie: i use them in the 2nd MP
[03:17] <achiang> rather, i use that class in the 2nd MP
[03:35] <poolie> ok
[03:35] <poolie> so, yes, i'd probably make an 'enum' class that has that behaviour
[03:35] <poolie> the 'types of filter' is perhaps a concept in its own right
[03:35] <poolie> since it can do more than just match names: also, probably, construct a filter object
[03:36] <poolie> so it's a bit like a factory pattern
[03:36] <achiang> hm, ok
[03:36] <achiang> i'll try and play around with an EnumClass that you describe
[03:37] <achiang> poolie: btw, i just tested, and 'filter tags' does work
[03:38] <poolie> oh i see
[03:38] <poolie> yep, i can see how that works
[03:38] <achiang> http://pastebin.ubuntu.com/705248/
[03:39] <achiang> poolie: also, note how we display current filters
[03:39] <poolie> i thought there should be an 'else' clause at line 98 of the diff but
[03:39] <poolie> in fact you just fall off the end and append it to the list, which is fine
[03:40] <achiang> yeah, that little bit was magic to me
[03:40] <achiang> whatever came out, was simply due to luck
[03:41] <achiang> oh wait, no, that was on purpose. :)
[03:41] <poolie> :)
[03:41] <achiang> it was the actual line 98 that confused me, but somehow still works
[03:42] <achiang> because i thought for sure there might be a KeyError, in the event we got a single word status... then there would be no args[2]
[03:42] <achiang> but somehow, it just works
[03:42] <poolie> [2:] means '2 and anything after it'
[03:42] <poolie> it can be an empty list in which case the del is a nop
[03:43] <achiang> 'filter status new' => there shouldn't be an args[2] in that case?
[03:44] <achiang> so i should get IndexError?
[03:44] <poolie> well, apparently 'filter' has already been consumed
[03:44] <poolie> so you have line='status new', args=['status', 'new']
[03:44] <poolie> args[2:] = []
[03:46] <achiang> poolie: ah! http://pastebin.ubuntu.com/705251/
[03:46] <poolie> right
[03:46] <achiang> that is somewhat surprising at first, but upon further reflection, makes sense
[03:46] <poolie> indeed you can even do foo[1:] = ['hello', 'there']
[03:47] <poolie> you can also say foo[100:] (which will also be empty
[03:49] <achiang> ok, thanks for the help. i'll try and clean it up, probably not tonight
[03:49]  * achiang will have to play with bzr-rebase
[08:51] <czajkowski> Aloha
[08:51] <allenap> Morning.
[08:55] <czajkowski> mrevell: you do know how to make my day start with a smile!
[08:55] <czajkowski> boss walking in and wondering whom I'm talking to less smiley
[09:30] <ubuntu-baltix> Hi, Why Launchpad doesn't build my recipe - error "chroot problem" appears, see https://code.launchpad.net/~stk/+archive/dev/+recipebuild/96097
[09:33] <wgrant> ubuntu-baltix: Hi, we're looking at that issue now. Thanks for letting us know.
[09:41] <ubuntu-baltix> wgrant: please tell me when you fix this "chroot problem"
[10:07] <ubuntu-baltix> wgrant: maybe you already fixed "chroot problem" ( https://code.launchpad.net/~stk/+archive/dev/+recipebuild/96097 )
[10:07] <ubuntu-baltix> ?
[10:08] <wgrant> ubuntu-baltix: No, still working on it.
[10:17] <mrevell> hey, glad to hear it czajkowski :)
[10:18] <czajkowski> I like doing that stuffmakes a change from me doing wire frames last week for someoner else to rip them apart
[10:18] <czajkowski> *someone
[10:43] <wgrant> ubuntu-baltix: That should be sorted out now.
[10:50] <ubuntu-baltix> wgrant: thanks
[11:15] <jml> merge diffs are taking a while
[11:18] <Oleg|percona> guys, I have question about launchpad MP
[11:19] <Oleg|percona> how can I look to history of change?
[11:19] <Oleg|percona> for example, I want see who exactly assign review for this MP: https://code.launchpad.net/~tsarev/percona-server/5.1_fix_bug_716210/+merge/78700
[11:19] <Oleg|percona> (who request review for guy)
[11:19] <czajkowski> mrevell: seen an oneiric nut yet :)
[11:21] <mrevell> czajkowski, Yes :)
[11:21] <czajkowski> good
[11:29] <rvba> Oleg|percona: I don't think that this information is available. allenap?
[11:30] <Oleg|percona> rvba: I got imagine. I think all information about status change are tracking
[11:32] <wgrant> Oleg|percona: It's not visible in the UI, but we store it and it's in the API.
[11:32] <wgrant> https://api.launchpad.net/devel/~tsarev/percona-server/5.1_fix_bug_716210/+merge/78700/votes
[11:32] <wgrant> The "registrant" field.
[11:32] <wgrant> Oh!
[11:32] <wgrant> It is in the UI.
[11:32] <rvba> Really?
[11:32] <wgrant> A tooltip in the "date requested" column.
[11:33] <Oleg|percona> wgrant: I see "when" , but I want know "Who"
[11:33] <rvba> True, good catch!
[11:33] <wgrant> Oleg|percona: Hover over the date.
[11:33] <Oleg|percona> wgrant: who exactly assign concrete guy to review
[11:33] <Oleg|percona> wgrant: guy himself, or project manager, or something else
[11:34] <Oleg|percona> wgrant: nick of the guy who assign another guy for review :)
[11:34] <Oleg|percona> wgrant: bug has button "full activity log"
[11:34] <wgrant> Oleg|percona: Does the date's tooltip not say that?
[11:34] <Oleg|percona> wgrant: aaaa, tooltip
[11:34] <Oleg|percona> wgrant: yes, exactly.. but this is not-intuitive, sorry
[11:35] <Oleg|percona> wgrant: thank you very much :)
[11:35] <wgrant> Not intuitive at all, no :(
[11:54] <njpatel> https://launchpad.net/unity/+series doesn't seem to work
[11:54] <njpatel> oh, hi :)
[12:02] <allenap> njpatel: You've broken it! :) I'll look into it.
[12:02] <njpatel> heh, thank you :)
[12:05] <allenap> njpatel: I've filed bug 871715 and I'll work on it now.
[12:07] <wgrant> allenap: I filed a bug about that on Saturday.
[12:08] <wgrant> It's Critical.
[12:08] <wgrant> I forget the number.
[12:08] <allenap> wgrant: Okay, I'll find it, thanks.
[12:08] <njpatel> allenap, awesome, thank you
[12:08] <wgrant> It's related to the new INCOMPLETE_WITHOUT_RESPONSE status.
[12:12] <njpatel> i say we fix it with PLEASE_RESPOND
[12:35] <chrysn> hi, i'm having trouble with the "release url pattern" feature of series -- can i specify a place to look for links?
[12:36] <chrysn> it seems launchpad just strips the last part off the url, and hopes to find a matching link there, but this doesn't work for example on github (i want tarball downloads here, not git checkouts).
[12:37] <chrysn> in watch files, there is a way to specify this in more detail, but it seems to be incompatible syntax-wise.
[15:16] <smoser> anyone here able to help me with renaming a project?
[15:16] <smoser> allenap, rvba ?
[15:18] <allenap> smoser: Sure.
[15:18] <smoser> https://launchpad.net/pileus -> https://launchpad.net/cirros
[15:19] <smoser> allenap, wait. hold off on that.
[15:19] <allenap> smoser: Okay :)
[15:21] <allenap> smoser: Can you put your request in https://answers.launchpad.net/launchpad/+addquestion so I can more reliably authenticate you? You can ping me again when you've done it.
[15:21] <smoser> allenap, sure. i can do that. thanks.
[16:42] <chrysn> yet another recipe question: can i, like for ppa's, specify a ppa to use for fulfilling a recipe's build dependencies?
[16:45] <bigjools> chrysn: it should work, yes. The recipe is built in the context of the PPA.
[16:51] <chrysn> thanks, i'll try copying the required packages to the target ppa
[16:54] <chrysn> it seems not -- the critical dependency was already in the target ppa, built for oneric (copied and re-built from another ppa), but still the build failed.
[16:54] <chrysn> (the build affected was https://launchpadlibrarian.net/82488465/buildlog.txt.gz)
[16:54] <bigjools> ah darn - well, I wonder if jelmer knows
[16:55] <chrysn> i might be wrong at all and something else failed, but it failed due to "Unable to resolve dependencies", and the only package not yet in ubuntu is said libopencsg-dev
[16:57] <bigjools> I have vague recollections of there being a recipe directive to pull in deps
[16:58] <bigjools> huh
[16:59] <bigjools> grep for "override-sources-list" - it's not setting the PPA up
[17:27] <smoser> allenap, still around ?
[17:28] <smoser> https://answers.launchpad.net/launchpad/+question/173846 if you are (or anyone else can take that).
[17:54] <chrysn> bigjools, i've been wrong -- i expected the build system to pull in the destination ppa, while instead it pulls in the destination ppa's dependencies (not allowing circular dependencies, which i might need here).