| === jincreator1 is now known as jincreator | ||
| === Ursinha is now known as Ursula__ | ||
| === micahg_ is now known as micahg | ||
| achiang | poolie: thanks for the review | 03:13 |
|---|---|---|
| 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:14 |
| 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:15 |
| 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:16 |
| achiang | rather, i use that class in the 2nd MP | 03:17 |
| 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:35 |
| 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:36 |
| achiang | poolie: btw, i just tested, and 'filter tags' does work | 03:37 |
| poolie | oh i see | 03:38 |
| poolie | yep, i can see how that works | 03:38 |
| achiang | http://pastebin.ubuntu.com/705248/ | 03:38 |
| 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:39 |
| achiang | yeah, that little bit was magic to me | 03:40 |
| achiang | whatever came out, was simply due to luck | 03:40 |
| 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:41 |
| 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:42 |
| achiang | 'filter status new' => there shouldn't be an args[2] in that case? | 03:43 |
| 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:44 |
| 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:46 |
| poolie | you can also say foo[100:] (which will also be empty | 03:47 |
| 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 | 03:49 | |
| czajkowski | Aloha | 08:51 |
| === allenap changed the topic of #launchpad to: https://launchpad.net/ | Help contact: allenap, rvba | Launchpad is an open source project: https://dev.launchpad.net/ | This channel is logged: http://irclogs.ubuntu.com/ | User Guide: https://help.launchpad.net/ | Support: https://answers.launchpad.net/launchpad | ||
| allenap | Morning. | 08:51 |
| 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 | 08:55 |
| 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:30 |
| wgrant | ubuntu-baltix: Hi, we're looking at that issue now. Thanks for letting us know. | 09:33 |
| ubuntu-baltix | wgrant: please tell me when you fix this "chroot problem" | 09:41 |
| ubuntu-baltix | wgrant: maybe you already fixed "chroot problem" ( https://code.launchpad.net/~stk/+archive/dev/+recipebuild/96097 ) | 10:07 |
| ubuntu-baltix | ? | 10:07 |
| wgrant | ubuntu-baltix: No, still working on it. | 10:08 |
| mrevell | hey, glad to hear it czajkowski :) | 10:17 |
| 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:18 |
| === daker_ is now known as daker | ||
| wgrant | ubuntu-baltix: That should be sorted out now. | 10:43 |
| ubuntu-baltix | wgrant: thanks | 10:50 |
| jml | merge diffs are taking a while | 11:15 |
| Oleg|percona | guys, I have question about launchpad MP | 11:18 |
| 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:19 |
| mrevell | czajkowski, Yes :) | 11:21 |
| czajkowski | good | 11:21 |
| rvba | Oleg|percona: I don't think that this information is available. allenap? | 11:29 |
| Oleg|percona | rvba: I got imagine. I think all information about status change are tracking | 11:30 |
| 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:32 |
| 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:33 |
| 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:34 |
| Oleg|percona | wgrant: thank you very much :) | 11:35 |
| wgrant | Not intuitive at all, no :( | 11:35 |
| njpatel | https://launchpad.net/unity/+series doesn't seem to work | 11:54 |
| njpatel | oh, hi :) | 11:54 |
| allenap | njpatel: You've broken it! :) I'll look into it. | 12:02 |
| njpatel | heh, thank you :) | 12:02 |
| allenap | njpatel: I've filed bug 871715 and I'll work on it now. | 12:05 |
| ubot5 | Launchpad bug 871715 in Launchpad itself "KeyError in view/bugtask_status_counts on Product:+series" [Critical,In progress] https://launchpad.net/bugs/871715 | 12:05 |
| wgrant | allenap: I filed a bug about that on Saturday. | 12:07 |
| 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:08 |
| njpatel | i say we fix it with PLEASE_RESPOND | 12:12 |
| chrysn | hi, i'm having trouble with the "release url pattern" feature of series -- can i specify a place to look for links? | 12:35 |
| 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:36 |
| chrysn | in watch files, there is a way to specify this in more detail, but it seems to be incompatible syntax-wise. | 12:37 |
| === medberry is now known as med_out | ||
| smoser | anyone here able to help me with renaming a project? | 15:16 |
| smoser | allenap, rvba ? | 15:16 |
| allenap | smoser: Sure. | 15:18 |
| smoser | https://launchpad.net/pileus -> https://launchpad.net/cirros | 15:18 |
| smoser | allenap, wait. hold off on that. | 15:19 |
| allenap | smoser: Okay :) | 15:19 |
| 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. | 15:21 |
| === micahg_ is now known as micahg | ||
| === zyga is now known as zyga-afk | ||
| === matsubara is now known as matsubara-lunch | ||
| === rvba changed the topic of #launchpad to: https://launchpad.net/ | Help contact: allenap | Launchpad is an open source project: https://dev.launchpad.net/ | This channel is logged: http://irclogs.ubuntu.com/ | User Guide: https://help.launchpad.net/ | Support: https://answers.launchpad.net/launchpad | ||
| chrysn | yet another recipe question: can i, like for ppa's, specify a ppa to use for fulfilling a recipe's build dependencies? | 16:42 |
| bigjools | chrysn: it should work, yes. The recipe is built in the context of the PPA. | 16:45 |
| chrysn | thanks, i'll try copying the required packages to the target ppa | 16:51 |
| 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:54 |
| 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:55 |
| bigjools | I have vague recollections of there being a recipe directive to pull in deps | 16:57 |
| bigjools | huh | 16:58 |
| bigjools | grep for "override-sources-list" - it's not setting the PPA up | 16:59 |
| === deryck is now known as deryck[lunch] | ||
| smoser | allenap, still around ? | 17:27 |
| === matsubara-lunch is now known as matsubara | ||
| smoser | https://answers.launchpad.net/launchpad/+question/173846 if you are (or anyone else can take that). | 17:28 |
| === zyga-afk is now known as zyga | ||
| 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). | 17:54 |
| === deryck[lunch] is now known as deryck | ||
| === matsubara is now known as matsubara-afk | ||
| === yofel_ is now known as yofel | ||
| === epsy is now known as \u03b5 | ||
| === jpds_ is now known as jpds | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!