=== 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!