=== maclin1 is now known as maclin | ||
pitti | Bonjour | 02:52 |
---|---|---|
sarnold | hey pitti :) | 03:17 |
pitti | hey sarnold, how are you? | 03:19 |
sarnold | pitti: pretty good, just got back from dinner :) | 03:20 |
sarnold | pitti: how's your day going so far? | 03:21 |
pitti | sarnold: it shouldn't even have begun yet, but I can't sleep any more :) | 03:21 |
sarnold | pitti: completely agreed :) | 03:21 |
pitti | started as usual, wading through -proposed mess; due to the uninstallable kernel we got tons of failures at night | 03:21 |
didrocks | good morning | 05:39 |
=== nudtrobert1 is now known as nudtrobert | ||
larsu | good morning! | 07:22 |
pitti | hey larsu | 07:25 |
larsu | hi pitti! Wie gehts? | 07:25 |
pitti | larsu: ganz okay, war nur viel zu frueh wach | 07:26 |
pitti | larsu: how is your new flat? | 07:26 |
larsu | pitti, amazing! Loved waking up there this morning :) | 07:26 |
larsu | but no internet yet | 07:26 |
larsu | working at dholbach's office with him and desrt | 07:26 |
desrt | hello | 07:27 |
didrocks | hey larsu! desrt | 07:31 |
didrocks | hey pitti, /me se sentait seul ce matin :) | 07:31 |
larsu | bonjour didrocks! ça va? | 07:31 |
didrocks | larsu: ça va, and you? | 07:31 |
didrocks | slept for the night in your new accomodation? | 07:31 |
larsu | ça va tres bien | 07:31 |
larsu | yep | 07:31 |
desrt | didrocks: yup. it's sleepy. | 07:31 |
didrocks | nice! | 07:31 |
desrt | larsu tried to wake me at 7... then at 7:30, then at 8.... | 07:32 |
didrocks | ahah | 07:32 |
pitti | didrocks: ah, désolé -- j'ai me léve déjà à 5h, mais j'ai perdu à dire bonjour | 07:32 |
larsu | then dholbach came to pick us up :) | 07:32 |
didrocks | larsu has the repeat function :p | 07:32 |
larsu | hehe | 07:32 |
didrocks | handy feature | 07:32 |
desrt | the "get up now, daniel is here" worked | 07:32 |
didrocks | heh, that was like "no other way now" | 07:32 |
didrocks | pitti: pas de problème ;) | 07:32 |
desrt | there was also promise of coffee and a chocolatine | 07:32 |
pitti | hey desrt, kiel vi fartas? | 07:33 |
desrt | pitti: bonege, dankon :) | 07:33 |
desrt | ni estas ĉe daniel :) | 07:33 |
dholbach | hi guys | 07:34 |
desrt | ya. this one :) | 07:34 |
dholbach | can somebody take a look at https://code.launchpad.net/~attente/unity-control-center/non-gnome-unity-ibus-support/+merge/238293 and https://code.launchpad.net/~brunonova/software-properties/lp1381050_2/+merge/264634 and https://code.launchpad.net/~binli/unity-settings-daemon/dont-scale-out-of-screen/+merge/268881? | 07:34 |
didrocks | dholbach: even if that's not really my role, but I'll patch pilot this afternoon, so I can have a look | 07:36 |
* dholbach hugs didrocks | 07:36 | |
* didrocks hugs dholbach back | 07:36 | |
* didrocks would have like a "pain au chocolat", but it seems it's only for people in Berlin :p | 07:37 | |
dholbach | didrocks, it's a "Schokoladencroissant" over here :-P | 07:39 |
dholbach | didrocks, or "Schokocroissant" rather | 07:39 |
desrt | dholbach: we don't need to add more friction to this particular argument... | 07:39 |
desrt | also, disappointed that there is no ¨ in there somewhere | 07:39 |
pitti | desrt: so you want an Überraschungsei ? | 07:40 |
* didrocks lalalalala, "croissant" and "chocolate" in the same line | 07:41 | |
didrocks | larsu: https://code.launchpad.net/~binli/unity-settings-daemon/dont-scale-out-of-screen/+merge/268881 sounds like a hack, do you know if there is a better fix upstream? | 07:41 |
desrt | didrocks: love you too :) | 07:42 |
larsu | didrocks, this is ridiculous... | 07:43 |
didrocks | larsu: mind commenting? :) | 07:44 |
didrocks | larsu: of course, using same exact words :p | 07:44 |
willcooke | morning all | 07:58 |
didrocks | hey willcooke | 08:00 |
desrt | didrocks: hey. i just met someone who looks just like you | 08:02 |
desrt | maybe a brother? | 08:02 |
Laney | morning! | 08:03 |
desrt | Laney: hi! | 08:03 |
Sweet5hark | Morning all, today is Setting Orange, the 26th day of Bureaucracy in the YOLD 3181! | 08:04 |
Laney | hey desrt | 08:06 |
Laney | good moving yesterday? | 08:07 |
desrt | yes | 08:07 |
desrt | slept well :) | 08:08 |
desrt | (although i didn't do that much, really) | 08:08 |
Sweet5hark | desrt: arent you canadians supposed to only move on July, 1st? (https://en.wikipedia.org/wiki/Moving_Day_%28Quebec%29) | 08:12 |
desrt | Sweet5hark: canada exists outside of quebec | 08:12 |
desrt | Sweet5hark: furthermore, i'm moving in berlin... | 08:12 |
Sweet5hark | desrt: oh, cool! | 08:12 |
desrt | not me. | 08:12 |
desrt | helping larsu | 08:12 |
desrt | he needed some extra hands, so i flew over | 08:13 |
Laney | he had to leave the country to move outside of the designated day | 08:14 |
larsu | didrocks, finally commented | 08:14 |
Sweet5hark | nice | 08:14 |
Sweet5hark | Laney: so all Canada is just Quebec? Thought so! | 08:14 |
* Sweet5hark should be careful not to troll _everyone_ today who has rights to sponsor his package. | 08:18 | |
willcooke | :D | 08:18 |
larsu | didrocks, commented on the bug as well. Probably we want to raise the dpi limit instead of hard coding a minimal resolution | 08:20 |
larsu | 192 is *not* hidpi imo | 08:20 |
larsu | but that's what u-s-d uses as a limit now | 08:20 |
* larsu can prepare a patch if needed | 08:21 | |
larsu | Laney, is there something like `apt-get build-dep` for `apt`? | 08:21 |
Laney | don't think so | 08:22 |
* dholbach hugs didrocks | 08:23 | |
larsu | Laney, I don't understand your comment about blocking the constructor | 08:23 |
* didrocks hugs dholbach back | 08:23 | |
Laney | what comment? | 08:23 |
Laney | oh | 08:23 |
didrocks | larsu: no harcoding? so sad :p | 08:24 |
Laney | more context please :P | 08:24 |
didrocks | larsu: agreed btw :) | 08:24 |
didrocks | Laney: hey! | 08:24 |
didrocks | Laney: //code.launchpad.net/~binli/unity-settings-daemon/dont-scale-out-of-screen/+merge/268881? | 08:24 |
Laney | no | 08:24 |
didrocks | ah sorry, didn't backlog fully | 08:24 |
Laney | larsu: we need to know the initial value for the property before giving an object | 08:24 |
larsu | Laney, indicator-datetime MR | 08:24 |
larsu | you block monitor_timezone_property(), not the constructor, right | 08:25 |
Laney | yeah, that's only called from the constructor | 08:25 |
Laney | so it essentially is it | 08:25 |
larsu | ah, right, sorry | 08:25 |
didrocks | larsu: I'm going to smooth a little bit your answer :) | 08:25 |
Laney | I kept the structure of the original code | 08:25 |
larsu | yeah this is very weird | 08:26 |
larsu | don't do that | 08:26 |
larsu | there is another main loop still running as well, right? | 08:26 |
Laney | ya | 08:26 |
larsu | desrt, did you ever write a blog post "recursive main loops considered evil"? | 08:27 |
Laney | haha | 08:27 |
larsu | if not, please do and give Laney the link :D | 08:27 |
larsu | Laney, gotta run for ~30 mins. Delivery to the new place | 08:27 |
larsu | will have a further look after | 08:28 |
Laney | tell me how to do it better | 08:28 |
didrocks | Laney: ah, if you piloting before me (I'll probably do tomorrow then, as I'm sure it will empty), do you mind looking at Daniel's request ~1h ago? (larsu commented on one already) | 08:32 |
didrocks | if not, I'll handle them in my shift | 08:32 |
Laney | didrocks: I looked at the attente one and the other seems to have been reviewed by bdmurray 8 hours ago? | 08:45 |
Laney | ah, my new keyboard just came | 08:54 |
Laney | blank keycaps | 08:54 |
willcooke | Laney, wot? On purpose? | 08:55 |
Laney | hahaha | 08:55 |
Laney | yep | 08:55 |
Laney | I thought it would look cool | 08:55 |
Laney | and actually it does | 08:56 |
Laney | also it's got clicky keys \o/ | 08:56 |
Laney | WFH benefits | 08:56 |
didrocks | I guess not that WFH: http://www.wfh.org/ | 08:57 |
Laney | sorry about your blood but have this neat keyboard | 08:57 |
didrocks | ;) | 08:58 |
didrocks | http://www.urbandictionary.com/define.php?term=WFH | 08:58 |
* didrocks likes the definition | 08:58 | |
didrocks | "Working from home - when your slacking off and not wanting to do any work." | 08:58 |
Laney | sounds right | 08:58 |
Laney | might have to move IRC onto the desktop now so that I can click clack more | 08:59 |
Laney | or set up synergy | 09:00 |
didrocks | or develop a robot answering for you | 09:00 |
Laney | schnap it | 09:00 |
didrocks | heh | 09:00 |
=== greyback__ is now known as greyback | ||
=== alan_g is now known as alan_g|llunch | ||
larsu | Laney, do you need to fetch the value on startup? Can't you signal that it's ready after the result comes back? | 09:56 |
larsu | it was ok before because local file io is generally considered ok to do blocking | 09:56 |
larsu | not so for d-bus calls (imo, desrt thinks otherwise) | 09:56 |
larsu | otherwise it looks good to me | 09:57 |
larsu | will give it a spin now and then comment. Thanks for working on this! | 09:57 |
Laney | larsu: I think the object should be ready from the start (also not having it so makes one of the old testcases fail) | 10:04 |
larsu | Laney, yeah ideally it would be, but it's slowing down the rest of the program | 10:05 |
larsu | and might cause issues with the other main loop (which uses the same main context) | 10:05 |
Laney | it's weird to start up in (say) UTC and then reconfigure later | 10:06 |
Laney | could I make a new main context? | 10:06 |
larsu | in a new thread... | 10:07 |
larsu | and it's a bit overkill... | 10:08 |
larsu | is it just a test case that fails or does the real program depend on this too? | 10:08 |
desrt | OH MY GOD STOP | 10:09 |
Laney | dude | 10:09 |
desrt | (sorry -- larsu just told me) :) | 10:09 |
larsu | I did not! | 10:09 |
larsu | I simply explained the problem to you | 10:09 |
desrt | i'm actually okay with the sync dbus call | 10:10 |
desrt | just don't recurse the global mainloop -- ever | 10:10 |
Laney | it's just slightly more annoying to write all that shit again | 10:10 |
desrt | it's not optimal | 10:10 |
desrt | but in your situation it's probably something like the optimal solution | 10:10 |
Laney | when I just want to say hey, wait here until you get the value | 10:10 |
* larsu gives in becasue it's probably not that big of a deal | 10:11 | |
desrt | but keep in mind that datetimed is not normally running | 10:11 |
desrt | so this sync call is going to be blocking on a dbus activation | 10:11 |
larsu | ugh | 10:11 |
desrt | which is sort of a good point | 10:11 |
desrt | why are you using dbus for this? | 10:11 |
Laney | instead of what? | 10:11 |
desrt | read /etc/localtime? | 10:11 |
Laney | someone broke gfilemonitor | 10:12 |
larsu | this is what he was trying to get rid of | 10:12 |
larsu | ya... | 10:12 |
desrt | so use dbus to watch for the change signal | 10:12 |
Laney | and then someone else said you should use timedated instead | 10:12 |
desrt | but read the file | 10:12 |
desrt | yes | 10:12 |
desrt | use timedated but only for the signal | 10:12 |
larsu | s/someone/larsu? | 10:12 |
desrt | do the reading locally | 10:12 |
Laney | jesus christ | 10:12 |
desrt | seriously | 10:12 |
desrt | this is how it's "supposed" to work with dconf-like services | 10:12 |
desrt | which is more or less everything these days | 10:12 |
desrt | writing and change monitoring = dbus | 10:12 |
desrt | reading = do it directly yourself | 10:12 |
larsu | meh, this is overkill | 10:13 |
Laney | what is propertieschanged for then | 10:13 |
larsu | you'd still use that | 10:13 |
desrt | there are all kinds of dbus services.... | 10:13 |
Laney | but then ignore the value that it gives you | 10:13 |
Laney | and go read it off the FS | 10:13 |
Laney | that is some kind of bullshit | 10:13 |
* desrt shrugs | 10:13 | |
* larsu shrugs harder and is hungry | 10:13 | |
desrt | activating a dbus service for no good reason is some kind of bullshit | 10:13 |
desrt | use the filesystem | 10:14 |
desrt | adding a matchrule doesn't activate the service | 10:14 |
* ricotz isn't really following, but gfilemonitor is really broken currently | 10:16 | |
desrt | Laney: do you have a specific bug for this case i can look at? | 10:16 |
desrt | iirc you only sent trash ones yesterday | 10:16 |
Laney | https://bugs.launchpad.net/ubuntu/+source/indicator-datetime/+bug/1485659 | 10:16 |
ubot5 | Ubuntu bug 1485659 in indicator-datetime (Ubuntu) "Date/time indicator doesn't update after changing time zone" [Low,In progress] | 10:16 |
desrt | we're going to lunch now. i'll look at this as soon as i'm back | 10:17 |
Laney | I could maybe get behind reading for the initial value | 10:17 |
desrt | but ya in general -- i agree with larsu about watching dbus for the change | 10:17 |
desrt | but please use the disk for reading | 10:17 |
desrt | since activating the service is really expensive | 10:17 |
Laney | but if we get a new one in PropertiesChanged | 10:17 |
Laney | then.... | 10:17 |
desrt | ya | 10:17 |
desrt | this does make sense | 10:17 |
desrt | i mean, why read the disk again if you just got told, right? | 10:17 |
Laney | yes | 10:17 |
Laney | that is the "bullshit" I mentioned earlier | 10:17 |
desrt | i'm 100% neutral on this point | 10:18 |
desrt | i'd probably read the disk just because this tends to be how i structure my code | 10:18 |
desrt | (and since the read is almost free) | 10:18 |
desrt | but no harm in using the info from the signal, either | 10:18 |
Laney | and luckily the original code had a function to read this /etc/timezone thing | 10:18 |
desrt | but on startup.... ya... please don't activate the service | 10:18 |
Laney | so can probably just bring that back | 10:18 |
Laney | this makes sense to me | 10:18 |
desrt | also: watch for the obnoxious race | 10:18 |
desrt | gdbus doesn't make this easy :( | 10:19 |
desrt | (honestly, it's probably safe to ignore it) | 10:19 |
Laney | which is? | 10:19 |
desrt | 1) send AddMatch via worker thread | 10:20 |
desrt | 2) read from disk | 10:20 |
desrt | 3) timezone is changed, signal is sent, match rule wasn't handled by dbus-daemon yet, you miss the change | 10:20 |
desrt | 4) AddMatch is delivered and processed | 10:20 |
desrt | gdbus actually kinda makes this situation shitty | 10:21 |
desrt | but again, it's unlikely to hit you | 10:21 |
desrt | you can work around it if you must by pinging the daemon | 10:21 |
desrt | as a sync call, after adding the matchrule | 10:21 |
Laney | I need to do what I am supposed to be doing instead of this bug | 10:22 |
Laney | will look later | 10:22 |
desrt | G_DBUS_SIGNAL_FLAGS_NO_MATCH_RULE got added for exactly this reason | 10:22 |
desrt | since dconf wants to send its own match rules so it knows when the match is actually active | 10:22 |
desrt | → food | 10:22 |
Laney | you can fix GFileMonitor too if you want :) | 10:23 |
desrt | that comes after food :) | 10:23 |
* desrt kicks larsu away from irc | 10:23 | |
Laney | BinLi: hi, I hope you're the right BinLi. Can you look to fix wily for https://code.launchpad.net/~binli/ubuntu/trusty/modemmanager/lp1441095/+merge/255935 per seb128's comment please? | 10:56 |
Laney | If it's cherry-picks from upstream this can probably go via Debian and maybe cyphermox can help you there | 10:57 |
desrt | Laney: i think this bug was the atomic replacement bug, already fixed | 11:38 |
Laney | HI | 11:39 |
Laney | fixed where? | 11:39 |
Laney | master? | 11:39 |
desrt | gvfs-monitor-file /etc/localtime out of jhbuild works | 11:39 |
desrt | can you test it with the version you have? | 11:39 |
Laney | oh yeah, this works now | 11:48 |
Laney | larsu: https://code.launchpad.net/~octoquad/software-center/899878-hardcoded-colors-dark-theme/+merge/268849 is this sane? | 11:50 |
larsu | Laney, why does software center care at all about the theme it's running under | 11:51 |
didrocks | larsu: it uses css (and it's gtk2, so no upstream support), no? | 11:52 |
larsu | didrocks, it can't be gtk2 and use css | 11:52 |
larsu | or is that for the web view? | 11:52 |
didrocks | web view | 11:52 |
Laney | nah it is gtk3 | 11:52 |
didrocks | (some parts are) | 11:52 |
=== alan_g|llunch is now known as alan_g | ||
didrocks | but yeah, Laney is right, it's gtk3 now | 11:53 |
larsu | clearly this is gtk css | 11:53 |
Laney | it does have its own css though | 11:53 |
larsu | weird | 11:53 |
larsu | so if I run my desktop in a theme that software-center doesn't know about, it looks like shit? | 11:53 |
larsu | anyway: fine by me if that's what it's doing. Lacking rationale for the transparency adjustment and includes whitespace fixes which should be a separate commit | 11:54 |
larsu | (or just remove them, it's not that important) | 11:54 |
didrocks | larsu: I think I know looking at the code | 11:57 |
didrocks | as it has its own widgets, using cairo surfaces to render | 11:57 |
larsu | ah. weird | 11:57 |
didrocks | I guess it tries to grab the value from the theme + additional properties | 11:57 |
didrocks | like in softwarecenter/ui/gtk3/widgets/buttons.py | 11:57 |
larsu | makes sense | 11:58 |
Laney | didrocks: you want to handle the upload? :) :) | 11:59 |
didrocks | Laney: that would be tomorrow for me, but sure | 11:59 |
Laney | aren't you PPing? :P | 11:59 |
didrocks | Laney: when I saw your name this morning, I think I'll be better to handle that tomorrow, as there are too many times there is no-one | 12:00 |
larsu | dholbach's mail really helped, eh? Now everyones PPing | 12:00 |
Laney | I always pee pee | 12:00 |
didrocks | so better at least to have at least one dude every day than 2 one day and then no-one | 12:00 |
Laney | is it? | 12:00 |
Laney | there's still 99999 items | 12:00 |
didrocks | I guess it's better for online presence | 12:01 |
* Laney shrugs | 12:03 | |
didrocks | dude! | 12:03 |
Laney | duuuuuuuuuuuuuude! | 12:04 |
Laney | up to you man, you do it when you think is best | 12:05 |
didrocks | yep, hence tomorrow ;) | 12:07 |
* didrocks opens moaarrrr sdk bugs meanwhile | 12:07 | |
didrocks | then, I'll consider my example app "done" | 12:07 |
didrocks | with nice margin for Laney :) | 12:07 |
Laney | \o/ | 12:15 |
=== MacSlow is now known as MacSlow|lunch | ||
desrt | MacSlow|lunch: enjoying your meal? | 13:26 |
attente | Laney: the conflicts on that branch should be resolved now | 13:52 |
=== MacSlow|lunch is now known as MacSlow | ||
* qengho afk a bit. | 14:25 | |
Laney | attente: nice | 14:26 |
Laney | you're my favourite! | 14:26 |
attente | :* | 14:28 |
attente | Laney: wait a second. can i check something before you approve | 14:30 |
Laney | ummmmmMMMMMMmmmmmmmmmmmmmmmmmmmmM | 14:30 |
Laney | this is going to take a bit of getting used to | 14:35 |
* Laney got a trackball maus | 14:35 | |
larsu | git bzr push lp:~/notify-osd/crisper-on-hidpi | 14:37 |
larsu | phew | 14:37 |
Laney | mmm crisps | 14:37 |
Laney | also ++ for using bzr-git-bzr-git | 14:37 |
larsu | :) | 14:38 |
Laney | get someone who knows this shit to review that | 14:38 |
larsu | Laney, who would that be? | 14:39 |
larsu | can you give it a try? | 14:39 |
larsu | not reviewing, just seeing how it works for you | 14:39 |
Laney | dezrt? :) | 14:39 |
Laney | or MacZlow himzelf? :P | 14:40 |
larsu | she's spelled with an s | 14:40 |
larsu | like so: desrt | 14:40 |
larsu | :P | 14:40 |
* Laney iz avoiding highlighting people | 14:40 | |
* larsu is destroying Laney's plan | 14:40 | |
Laney | at least I avoid the blame | 14:40 |
Laney | :) | 14:40 |
* didrocks is destroying larsu's plan to destroy Laney's one | 14:40 | |
Laney | yes I will try this | 14:40 |
larsu | uh oh | 14:40 |
larsu | this is getting meta | 14:40 |
larsu | Laney, thanks :) | 14:41 |
larsu | MR is up https://code.launchpad.net/~larsu/notify-osd/crisper-on-hidpi/+merge/269930 | 14:41 |
didrocks | and with this last bug opened, I consider my example app done! | 14:43 |
attente | Laney: yeah... should be ok :o | 14:43 |
Laney | didrocks: get it in ze store! | 14:47 |
didrocks | and make $$$$$ | 14:51 |
Laney | larsu: it sure looks crisp to me | 14:51 |
larsu | glad to hear :) | 14:52 |
larsu | thanks for testing! | 14:52 |
Laney | thanks for fixing! | 14:53 |
larsu | was a bit annoying indeed | 14:53 |
* larsu sat on a particular issue all morning | 14:53 | |
larsu | we should redo this code to use proper widgets and gtk css (which wasn't available back then) | 14:53 |
ogra_ | and i thought you sat next to dh | 14:53 |
ogra_ | *dholbach | 14:54 |
larsu | I did, and still do | 14:54 |
* larsu hugs dholbach | 14:54 | |
larsu | man, he's not even in here anymore | 14:54 |
ogra_ | yeah | 14:54 |
ogra_ | snappy ate him | 14:54 |
ogra_ | </french accent> | 14:54 |
larsu | haha | 14:54 |
* Laney puts away the second keyboard | 14:57 | |
* Laney is getting into good habits | 14:57 | |
Laney | this file thing fucks up the testsuite | 15:43 |
* qengho back | 15:44 | |
Laney | bah | 15:44 |
larsu | be strong Laney | 15:44 |
Laney | it calls a mocked SetTimezone | 15:44 |
Laney | which doesn't write to /etc/timezone of course | 15:44 |
Laney | :| | 15:44 |
larsu | but should emit the signal now? | 15:45 |
Laney | we don't wait for that any more | 15:45 |
larsu | right | 15:45 |
Laney | it's like oh hi I got an object | 15:45 |
Laney | oh it's Europe/London not the thing I just set it to | 15:45 |
Laney | this is annoying | 15:45 |
Laney | good thing is that the old testsuite had some shit for this | 15:46 |
Laney | I'm basicaly undoing half of my work | 15:46 |
larsu | :( | 15:46 |
Laney | bye! | 17:04 |
Laney | I got the tests to pass again | 17:04 |
Laney | not sure if some shit is weird or not | 17:04 |
Laney | ready for more abuse tomorrow :) | 17:05 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!