[00:41] Greetings! Is it possible to blacklist a package in the live session? [00:43] bluesabre: What do you mean by blacklist? [00:43] in xubuntu we ship light-locker as a means to lock one's session [00:44] it would be beneficial to not have that package installed in the live session [00:44] (because the timed locking would lock a user out of the live session) [00:47] Um, I thought settings were twekaed on live boot such that the lock screen isn't enabled... I know that happens for Ubuntu... [00:47] tweaked* [00:49] Maybe xubuntu needs a script that gets loaded by casper to do this for a live session. [00:49] ok, I'll take a look at that [00:49] thanks for the suggestion [00:51] np === m_conley is now known as m_conley_away === duflu_ is now known as duflu [08:37] hi :) [08:44] how can I see the current version of a trunk package to add it as a dependency on another pakage's control? [08:45] most packages in debian control have a version (>= version_number) [09:00] good morning desktopers! [09:01] hikiko, there is no "current version of trunk", you either use ">> current_archive_version", assuming the next upload is going to have your change or you wait for the upload to tweak the version [09:01] hikiko, did you get any designer feedback on the control center UI? I read your comment, the backend requirements shouldn't dictate the UI though... [09:02] mpt, hey, do you think you could have a look/comment on the UI/design of https://code.launchpad.net/~hikiko/unity-control-center/unity-control-center.per-monitor-fonts-scale-factor-slider/+merge/205227 (http://imgur.com/cVlLyV0) [09:07] seb128, I think bregma asked some designer at some point but I will discuss with him again for sure, maybe I have to move the slider on the left [09:08] hikiko, is that setting only about fonts? [09:08] morning chaps [09:09] Laney, hey, how are you? [09:09] seb128: good thanks, although my arms are feeling the pain from climbing :P [09:10] seb128, no, it's for all ui-components in Unity8/QML apps (Kaleo will know for sure) in Unity7 it's for fonts but since everything is rendered according to the EM size it's for everything... [09:10] might have gone a few too many times lately ... [09:10] hikiko, the label is misleading then [09:10] Laney, hehe, I know the feeling ;-) [09:13] I've been 4 times in 5 days :( [09:13] how are you doing? [09:13] Laney, what do you think about adding ubuntu-schemas to the ubuntu-settings source? [09:13] you mean merging gsettings-...? [09:14] Laney, I'm good thanks [09:14] Laney, yeah, "merging/superseeding" [09:15] yeah if you like, that should be easy to do [09:15] also I don't remember know why you made gsettings-ubuntu-touch-schemas depends on ubuntu-touch-sounds but that's a depends we don't want on the desktop [09:15] I did not [09:15] Laney, yeah, doability is fine, I was wondering if you think it's a good idea ;-) [09:16] https://code.launchpad.net/~laney/gsettings-ubuntu-touch-schemas/no-sounds-dep/+merge/192314 [09:16] Laney, hum, sorry, seems like I got that backward, I remember we discussed if the default value should be pulled in [09:17] well, I can think that it mixes up the idea of 'upstream' and 'downstream' [09:17] did I argue in favor of the depends by then?! [09:17] yeah but I think you were convinced in the end [09:18] seb128, you are right, I didn't replace it when I renamed the fonts_scale_factor to ui_scale.. [09:18] thanks! [09:18] hikiko, yw [09:19] Laney, I sure am convinced :p [09:19] ;-) [09:19] seb128, done [09:19] Laney, well, with daily landing, CI, etc, there is no much downstream/upstream difference nowadays [09:19] mpt, thanks! [09:20] hmm [09:20] It's like ubuntu-settings overrides GNOME stuff but then we also randomly provide schemas for things in there too [09:20] 'this package is vaguely to do with gsettings' [09:20] hikiko, we got design input on the mp, thanks to mpt, please address his comment as well (I'm going to handle the schemas thing) [09:23] let me see :) [09:24] Laney, ideally regrouping things (like we regrouped themes in icon themes in ubuntu-themes) would be nice [09:24] less sources/projects to handle [09:29] ok let's move the slider to the left :) [09:29] sil2100, didrocks: hey, would it be easy to put unity-system-settings and unity-control-center under CI train? they have auto commits atm so I guess they are set up in the old autolanding way [09:31] seb128: should be easy, I guess you just need to tag latest release to avoid having a crazy changelog [09:31] seb128: it should be relatively fast and easy, we just need to disable the auto-landing and then one tag to have all ok [09:31] didrocks, sil2100: thanks, what sort of tag? [09:32] can you do it/let me know what to do? [09:32] * didrocks looks at the code before telling wrong things :) [09:32] one sec [09:32] seb128: ok, it's just the package version [09:33] didrocks, sil2100: you have your daily morning call now? feel free to get back to me after it, no hurry there [09:33] (including the -0ubuntu…) [09:33] yeah [09:33] just put that and the bot will only collect after that tag [09:33] ok [09:33] I need a manual commit to push a tag? [09:34] I think you can push it with --overwrite without a commit, but not garantee [09:34] no* [09:34] let me try [09:34] I just need to fresh checkout and look at the tags to verify [09:49] didrocks, in fact debcommit -r tagged things already, we should be fine for unity-control-center I think [09:49] $ bzr tags | grep 14.04.3 [09:49] 14.04.3-0ubuntu1 12707 [09:50] sil2100, ^ [09:50] seb128: great! at worst, it's a big (incorrect) changelog anyway… [09:50] sil2100: mind putting that one? ^ [09:50] didrocks: you mean, like, disabling from cu2d? ;) [09:51] (and adding seb as the lander) [09:52] didrocks, sil2100: unity-settings-daemon should good to go as well [09:55] sil2100: yeah, seb as lander, disabling with the usual parameters [09:55] seb128: do you mind adding it to the bootcamp spreadsheet? [09:56] (trying to have a list of our projects) [09:56] so that later on, you can subscribe, and so on [09:59] didrocks, sil2100: done, l80&81 (inserted bellow g-c-c-unity to keep logical units) [09:59] thanks! [09:59] yw ;-) [10:00] sil2100: tell us once done [10:01] seb128: where can I find unity-system-settings ? [10:02] sil2100, unity-settings-daemon you mean? [10:02] sil2100, https://code.launchpad.net/unity-settings-daemon [10:03] https://code.launchpad.net/~unity-settings-daemon-team/unity-settings-daemon/trunk [10:04] seb128: ah, those I know already ;) You mentioned something about unity-system-settings in the backlog and I was thinking if I missed anything [10:04] sil2100, my fingers typoed, those projects are too close :p [10:04] sil2100, I mean unity-control-center and unity-settings-daemon [10:12] seb128, didrocks: reconfigured ;) All should be ok now [10:12] sil2100, meaning I can go through CI train for those now? [10:14] Laney, I was just pondering taking that new gtk-doc or not, quite some changes, good that you went for it, spare me the extra thinking ;-) [10:14] glib depends on it :P [10:15] seb128: yes [10:15] Laney, yeah, new gdk-pixbuf as well (was looking at this one) [10:15] nod [10:16] Laney, can you +1 https://code.launchpad.net/~seb128/unity-settings-daemon/hidpi-screen-support/+merge/207125 for me? I want to try CI train land u-s-d [10:17] Laney, that patch is already in g-s-d so it should be fine [10:18] seb128: g-s-d in ubuntu? [10:18] Laney, yes, https://launchpad.net/ubuntu/+source/gnome-settings-daemon/3.8.6.1-0ubuntu3 [10:18] ack [10:18] thanks [10:20] done [10:23] thanks [10:27] sil2100, ok, put requests for the 2 new components, let's see how that goes [10:49] didrocks, https://launchpad.net/~ci-train-ppa-service/+archive/landing-005/+sourcepub/3923487/+listing-archive-extra [10:49] didrocks, changelog worked ;-) [10:49] excellent! :) [10:49] u-c-c railing the train! [10:50] tjaalton, ^ btw, I think you were interested in new wacom? [10:50] didrocks, and u-s-d! https://launchpad.net/~ci-train-ppa-service/+archive/landing-006/+sourcepub/3923493/+listing-archive-extra [10:50] hehe "niceeeeee" ;) [10:50] ;-) [10:50] seems people try to challenge the ci train wrapping :) [10:50] change the scaling value by writting the [10:50] "org.gnome.desktop.interface scaling-factor" gsettings key [10:51] :) [10:51] ;-) === tkamppeter_ is now known as tkamppeter === vrruiz_ is now known as rvr [11:22] seb128 and Laney could you please get one more look at this: [11:22] https://code.launchpad.net/~hikiko/gsettings-ubuntu-touch-schemas/scale-factor-for-unity7-unity8/+merge/206240 [11:22] (thanks!) [11:23] hikiko, sure [11:23] though I'm a bit confused at what Laney wants with the packages sources/names atm [11:23] oh sorry [11:24] I thought Lane and Laney is the same person :) [11:24] why is picking names always so annoying :p [11:24] * Laney hulks out [11:24] sorry sorry :) [11:24] that is me [11:24] you weren't wrong :P [11:24] hikiko, he is [11:24] then because he reviewed it already [11:25] seb128: umm I don't want to block you doing what you think is right, it just feels a bit weird is all [11:25] renaming the binary is A-OK as far as I'm concerned [11:25] 'it' being merging the sources [11:26] Laney, I've no strong opinion either way, that's the sort of details I don't care much about [11:26] then my preference is to rename the binary package and ideally the source but probably do that later on [11:26] robert_ancell started on merging sessions, settings, etc in http://code.launchpad.net/~robert-ancell/+junk/ubuntu-session [11:27] but I though moving the gnome-session .session files in there were touch much [11:27] ok [11:27] let's do it easy then [11:27] renaming the binary to "ubuntu-schemas" [11:27] ? [11:28] gsettings-ubuntu-schemas I think [11:28] and the AS one too [11:29] right [11:29] do you want to do it maybe? ;-) [11:29] (if not that's fine, I'm going to have a look next, after landing the new u-c-c and u-s-d) [11:29] could do in a bit perhaps [11:29] just testing glib and dconf [11:29] k, let me know if you start on it [11:30] then I will see about webkit [11:30] I've a few items/updates I started on [11:30] but I'm going to have a look in the afternoon if you don't beat me to it [11:30] hopefully upload that today [11:30] ok [11:30] hikiko: looks ok to me now! [11:31] :D [11:31] I wouldn't have been so verbose in the description (the bit where you justify int vs float) but it's okay [11:32] also I'm interested in how this will interact with what gnome does [11:33] I suppose config tools will have to write two keys at some point [11:33] ops sorry [11:33] no worries, it's fine [11:35] well in unity7 you either change the text-scaling-factor and see the same size in each monitor [11:35] or you use the ui-scale and you change the scaling per monitor [11:35] if you only have 1 monitor [11:35] then I guess it's almost the same [11:35] for unity7 at least [11:42] bregma, Trevinho_, the new unity with hidpi support is buggy for me, on my external monitors in 1920 resolution the launcher/panel are bigger after the update [11:42] where they should not, that's not an hidpi screen and the old default was fine, the new factor seems wrong [11:52] seb128: what's your dpi? xdpyinfo | grep dots ? [11:52] xnox, resolution: 96x96 dots per inch [11:53] seb128: that should factor 1... [11:54] seb128: where are these hidpi merge proposals / ppas that I can try? [12:08] xnox, sorry, was dealing with lunch getting ready ;-) [12:09] xnox, https://launchpad.net/~ci-train-ppa-service/+archive/landing-003/ [12:09] seb128: burned much?! =))) [12:09] lol, no, it was alright and I'm done eating ;-) [12:09] xnox, https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc#gid=21 has the list of vcses [12:10] if you want the code details [12:10] seb128: thanks. [12:11] yw [12:11] looking to https://code.launchpad.net/~brandontschaefer/unity/launcher-high-dpi-dynamic-resize/+merge/203176 they changed the units to be _em values [12:12] but I guess that's already in the code, was just not used before, so the change to review might not be in that list [12:13] seb128: i'm yet to figure out how to do dpi in e.g. plymouth and lightdm. [12:13] seb128: i guess system-wide setting for lightdm should be to autoscale. [12:14] xnox, lightdm runs u-s-d, that sets the autoscale factor for gtk, which should took care of part of the issue [12:14] well, unity-greeter rather than lightdm [12:15] seb128: yes and no, as patterns and logos are rendered at fixed px based offsets and sizes... [12:15] right [12:16] seb128: the grid dots, the bottom left logo, and centered (other monitor logo), frames around text input - should all dpi scale. [12:16] seb128: unless cairo changes it's layout to be dpi-wise rather then pixel wise? [12:18] no, I don't think so [12:28] seb128: btw, I came back from NUDT this afternoon, will continue ibus work tomorrow [12:29] happyaron, ok, thanks [12:29] happyaron, did the "disable ctrl-space by default" looks fine to you? [12:29] the patch looks fine, but I'm not able to test it when at NUDT [12:30] happyaron, ok [12:31] happyaron, how is Kylin work looking for feature freeze? [12:32] seb128: they are mostly focused on cooperation projects with commercial companies, no big action regarding the distro itself. [12:32] ok [12:32] shrug, I could use with bugzilla being up [12:33] FJ and I will go to a partner's office maybe in next week, for the cooperation project of input method. [12:33] ok [12:34] and getting a station, which we don't have at our office, :P [12:34] station? [12:35] I mean a seat in office, dunno which word is proper to describe. [12:39] oh, I see [12:42] * desrt yawns [12:42] desrt, good morning! [12:43] gähn [12:43] greetings desrt [12:43] seb128: i'm doing the renames now [12:43] Laney, thanks [12:43] good morning, peeps [12:43] Laney, then we can CI train landing it since u-c-c is there as well now [12:43] want to get the depends there fixed to gsettings-ubuntu-schemas? [12:44] yes please, I should approve that mp you pointed earlier [12:44] seb128: yes, updated wacom backend for u-c-c/u-s-d? [12:44] tjaalton, robert_ancell did https://code.launchpad.net/~robert-ancell/unity-control-center/wacom-panel-3.8/+merge/207082 [12:45] tjaalton, I guess we need to update the unity-settings-daemon side as well [12:47] seb128: yeah i think so. i'll be able to test these next week [12:47] tjaalton, great, thanks, let us know how it goes === MacSlow is now known as MacSlow|lunch === alan_g is now known as alan_g|lunch === MacSlow is now known as MacSlow|lunch === alan_g|lunch is now known as alan_g [13:46] merging acpid and adding unity-settings-daemon support to it... [13:47] xnox, that is using g-s-d? [13:48] seb128: it does nothing if g-s-d is running... [13:48] hum, k [13:49] seb128: it manages powerbutton policy (e.g. what to do) but it should do something on e.g. headless servers, not when there is u-s-d and unity pooping up a dialog "sleep shutdown restart" [13:49] right, I wonder if that's what makes my system shutdown when pressing the power button (rather than opening the dialog) [13:50] (I'm running u-s-d) === Trevinho_ is now known as Trevinho [13:50] I noticed that some days ago and have it on my "to debug" list [13:52] seb128: upgrade to new acpid, when that builts and let me know. [13:53] xnox, sure, thanks for the letting us know about the issue/update! [13:54] seb128: i'm just franticly going though my outstanding merges list, trying to fix up everything what will be useful in an lts at the moment. [13:56] xnox, yeah, makes sense [13:56] Trevinho, let's move here, the other channel is a bit busy [13:56] Trevinho, I didn't realize you would use the text scaling as an UI scaling [13:56] ok [13:57] Trevinho, the key for gtk scalling upstream is "org.gnome.desktop.interface scaling-factor" [13:57] not text-scaling [13:57] seb128: that was a late thing we added both for testing and because I thought it's what we wanted as well... I.e. using that value as minmum scaling factor anyway [13:57] ouch [13:57] ok... I can fix it in a small branch [13:57] thanks [13:57] can still we add this to the queue? [13:57] sure [13:58] you just need to edit the gdoc and ask for the silo to be reconfigured [13:58] but I'm fine if that happens in another round, I guess not a lot of people use the text-scaling [13:58] btw that key is set by the a11y -> big text option [13:58] yes [13:59] in fact that's why I wanted to supprot this as well [13:59] well, the setting says "text" [13:59] because for a11y issues having everything scaled up was something better I thought [13:59] not UI :p [14:00] Trevinho, well, personally I've no a11y issue, the default font is just a bit small and I get tired at the end of the day, and since we have no UI for font size I went for the big text one [14:01] but unity really looks not so nice with that scaling factor applid [14:02] seb128: yeah, the fact is that things won't scale, at least not everything... SO... I guess we still need to use the text-scaling factor as our base scaling factor if defined, don't you agree? [14:02] Trevinho, it feels wrong to have an option called "big *text*" which increases your launcher and panel [14:02] seb128: seems new compiz is placing new windows across boundaries...is that a known issue? [14:02] mdeslaur, define "across boundaries"? [14:03] I think Trevinho has a fix submitted for decoration issues [14:03] seb128: when I open a terminal on workspace 1, the window gets put at the bottom of the screen with part of it in workspace 3 [14:03] mdeslaur: on the bottom it happens yes... known issue, I've on my list [14:03] mdeslaur, what Trevinho says then [14:03] Trevinho: ok, cool, thanks [14:04] mdeslaur, you are picky about your desktop, that's good ;-) [14:05] seb128: hehe, yes, I do enjoy it when it works properly :) [14:05] Trevinho: FYI, it also places new windows with the title bar hidden behind the global menu [14:06] probably the same issue [14:06] Trevinho, ok, so those text-scaling scaling-factor confuse me [14:06] Trevinho, I've no real opinion on how things should behave but that seems like bug fix material for after ff if the behaviour is wrong [14:07] mdeslaur: mh, that's weird... [14:08] Trevinho: only seeing it with virt-manager though, not sure why [14:09] seb128: well, the main thing was to get unity scale up (and down) keeping it working with no regression, and that's what we are doing... But some details still need to be improved. [14:09] Trevinho: ie: switch to a new workspace, open terminal, terminal is correctly placed lower than global menu. Open virt-manager, virt-manager goes beside the terminal, but with titlebar hidden underneath global menu === MacSlow|lunch is now known as MacSlow [14:11] seb128: mh that scaling factor value gnome has is quite hard to use since it's an integer [0...4294967295]... but... 2 is already scaling of factor 2, so it's too much [14:11] seb128: we want scaling factor steps in float [14:12] seb128: so, for now I'll just disable the usage of text-scaling-factor [14:12] Trevinho, well, you guys are adding a new setting no? [14:12] seb128: yes, but I also would have loved to have gtk apps to scale with us [14:14] Trevinho, well, gtk apps are going to scale by 2 [14:15] through u-s-d/scaling-factor [14:15] seb128: I guess we'd need to patch gtk then to adapt to our scaling... as we support also scaling 1.5.. or less [14:15] and cairo can do that... so it shouldn't be hard (/me hopes) [14:15] Trevinho, there is a reason gtk did int values, it's not going to be that easy [14:16] you can't do half pixels [14:16] no, sure... [14:16] 3*1.5=4.5 bog [14:16] Trevinho, well, let's already land the new stuff for ff and then see where we stand/what we can fix/improve [14:17] ok [14:17] Trevinho, I'm going to test again new unity with text-scaling to 1, feel free to keep using that key for easy testing until the new settings land [14:18] seb128: ok, ah decorations scaling is to-be-fixed in these branches... I already have the code, but I'm pushing that later today [14:18] I mean, it's working but window buttons are not so nice... [14:18] (another good thing of new decos btw) [14:22] seb128: there's one thing that gnome scaling factor seem to cause: gdk display at that point rerturns has wrong geometries... [14:23] I mean, they're scaled as well I think [14:23] so the launchers, panels have wrong size [14:23] should I scale them as well?!? :o [14:23] no idea about that [14:28] hm, where is mardy ? [14:29] seb128, did you see https://01.org/gsso/ ? is that a fork of ubuntu's SSO ? [14:29] xclaesse, kenvandine mentioned it before [14:29] that's not new [14:30] that's based on the same stack, they "just" replace the qt part by a glib based one [14:30] they keep it mostly compatible [14:31] oh, that's the rewrite of the qt part for tizen? [14:31] yup [14:31] I remember mardy told me about that a long time ago [14:44] seb128: so yes I've to scale also the monitor sizes to get that scaling factor to work... I don't know if it's something wanted by gdk... [14:45] seb128: however, I've moved to use that setting and it works ok [14:45] k [14:45] seb128: decorations dragging/resizing is supported in a different branch though, but that can be landed in fe hours [14:46] Trevinho, ok, well up to you guys to handle the landing, bregma has access now I think [14:46] Trevinho, but let me know if I can help [14:47] imho you guys should land that first round of hidpi work [14:47] seb128: ok [14:47] so you can move on on landing other features [14:47] seb128: this is the MR fyi [14:47] https://code.launchpad.net/~3v1n0/unity/hdpi-gnome-scaling-factor/+merge/207217 [14:47] like the lock screen [14:47] Trevinho, thanks [14:47] Trevinho, do you ever actually sleep? [14:47] bregma: yes... like 6 hours... :) [14:47] no, almost 7 actually :D [14:48] FFs are always a physical exercise to me... BUt this one is quite relaxed compared to the one in the past... As I've mostly had to help with other's stuff :) [14:49] seb128: ops, the MR was wrong.. I didn't set the prereq ;:D [14:50] bregma: i presume you're too busy today, but in case you'd wanna sponsor light-locker-settings, that'd be awesome! https://bugs.launchpad.net/light-locker-settings/+bug/1281536 [14:50] Launchpad bug 1281536 in light-locker-settings "[needs-packaging] light-locker-settings" [Wishlist,In progress] [14:50] bregma: so, we'd need to get this in the train of Brandon's branches https://code.launchpad.net/~3v1n0/unity/hdpi-gnome-scaling-factor/+merge/207219 [14:54] Trevinho, we'd need bschaefer's review of that first, there was a reason we were not using the GTK scaling factor like that and he has the background and information to explain [14:55] bregma: he told me the problem was that it's an integer... so we couldn't scale of 1.4 for example [14:55] bregma: now, this change doesn' actually does anything but using that integer value adn allowing scaling by 2, 3.... [14:56] bregma: anyway that's something that we're going to ignore afterwards [14:56] bregma: I put that only to have a setting to play with, but... I could get rid of it [14:56] the fact is that I would like to land all this stuff asap, to have still time for the next stuff [14:57] Trevinho, we might just have to include it in the next landing, I don't want to restart the AP test run because it takes 3 hours [14:58] bregma: ah ok sure [14:58] Trevinho, I notice when I unhide the Launcher now it moved the whole desktop to the right... have you seen that? [14:59] bregma: you mean, when you change the option? [15:00] bregma: if yes, that's a feature... actually a feature of nautilus itself as there isn't a dock on the left anymore [15:00] I have the Launcher set to autohide, and when I unhide it, the icons I have lying around on the desktop jump to the right, like the whole desktop is moving [15:01] then they jump back when the Launcher hides again [15:01] the indicator ordering is all messed up, too [15:01] bregma: nvm, Laney seems to be looking at it already (thanks a lot Laney !) [15:01] np [15:02] oh, no [15:02] bregma: that's with last branches or trunk? [15:02] seb128: new acpid is in release pocket. [15:02] xnox, thanks [15:02] attente_, hey [15:02] * Trevinho has not it here [15:02] ochosi: I'm just going to make some minor fixes while uploading, hope that's ok [15:02] seb128, hi [15:02] Laney: absolutely [15:02] Trevinho, latest landing PPA, but the system may very well be messed up, it's a development machine taht gets that way sometimes [15:03] attente_, I'm having issues with indicator-keyboard tests, maybe you can help me [15:03] attente_, public void test_activate_text_entry_settings () { [15:03] attente_, stderr.printf ("_service.command = \"%s\"\n", (!) ((!) _service).command); [15:03] attente_, where is that command coming from? [15:03] Trevinho: bregma: I have messed up indicator icon ordering with latest landing PPA. I did not have this with Unity from main. [15:03] bregma: yeah, but well.. I'm running that code now and I don't have it here... but, latest merges/remerges might have caused a regression [15:04] seb128, the command is the property in the Service class in the same file [15:04] ChrisTownsend, Trevinho, bregma: oh, I had that as well, tomboy was top right for me, instead of session [15:04] bregma: However, I don't see the Launcher unhide issue you describe. [15:04] :o [15:04] seb128, when the command is executed, the only thing the service does is set that string and trigger a notify [15:04] what it landed? [15:05] Trevinho: What do you mean? [15:05] seb128, so during the test, when we spawn something like gucharmap, the test can know, "ok, it was executed", without having to run the actual gucharmap [15:05] ChrisTownsend: I dont' know what might have been caused that [15:06] Trevinho: Not sure. I'll purge the landing PPA and see if it goes back to normal. [15:06] attente_, ok, I'm overlooking something then, but http://bazaar.launchpad.net/~seb128/indicator-keyboard/use-unity-control-center/revision/272 makes the test unhappy and I don't understand why [15:07] seb128, oh... probably because it's missing the symlink that would call the dbus method [15:07] attente_, oh, in tests/execute [15:07] attente_, thanks [15:08] ChrisTownsend: I made a change in libindicator but no ABI/API changed [15:08] attente_, I greped the source, I didn't think there were symlinks for it [15:08] attente_, the mp is coming your way in a few minutes ;-) [15:09] seb128, ok, thanks! [15:09] Trevinho: Yeah, with just the landing PPA purged, indicators are back in the correct order, so it's something in one of the MP's. [15:09] Trevinho, it's not libindicator, that landed yesterday [15:09] but I confirm what ChrisTownsend says [15:09] oh, wait I see it [15:10] it's ups there [15:10] let me check what changed [15:10] bregma, ^ you are going to need a new round to land that ppa, you can start adding vcses to the request and ask for a silo reconfigure [15:11] * ChrisTownsend Hears bregma groan about having to stop his Autopilot run [15:11] uff, yeah I told Brandon to fix an issue last night, but had not the current trunk... so my diff was not the same, and the criss-cross merge made the rest I guess [15:11] ah, Brandon's famous criss-cross merge [15:11] ochosi: You could avoid some of those things in rules by fixing the makefile [15:12] let me see [15:12] like `install' lets you specify the permissions directly & I wouldn't install INSTALL and COPYING personally [15:12] Laney: hmm, sorry, i only helped write the app, the packaging was done by someone else... :/ [15:12] other than that, looks good -- I'll sponsor [15:12] sure [15:12] you can pass the message on ;-) [15:12] thank you very much! [15:12] i will :) [15:14] Trevinho, if you can get something ready to fix those indicators, I'll add the other stuff to the silo, I don't think we have time for two separate landings [15:14] bregma: it's alerady done... [15:16] attente_, ok, that worked, thanks! https://code.launchpad.net/~seb128/indicator-keyboard/use-unity-control-center/+merge/207225 [15:35] ochosi: ok, done [15:36] Laney: you're the best! a big thank-you from the xubuntu community [15:36] yw [15:36] Laney: very much appreciated, even more so one day before FF... [15:37] I think it'd have been alright anyway as you got the request in in time [15:37] but still === alan_g is now known as alan_g|tea [15:46] Laney: in fact we're having a tough time this cycle, as our uploaders are MIA or really busy with RL. still hoping to get some default-settings stuff and seed-changes merged :/ [15:46] ochosi: yeah, it's hard [15:46] best thing is to get the stuff in the queue [15:46] yeah, but the problem is that we have nobody with even merge-rights to some of our branches [15:47] so we have like 10 merge-requests pending [15:47] (all approved by the xubuntu project lead) [15:47] anyway, i don't wanna whine here, thanks for your help so far :) [15:48] if you think that sponsors can also be asked to merge stuff, then i'd ask around [15:48] yeah, they should be able to [15:48] just put it all in a bug for ubuntu-sponsors [15:49] oh, ok [15:49] will try that then [15:49] although it's quite optimistic, considering how "late" it is [15:51] that kind of stuff is okay after FF [15:51] * Laney beams positivity to ochosi [15:51] thanks Laney :) [15:51] much appreciated, really === alan_g|tea is now known as alan_g [16:16] Laney, do you want to do the CI train landing for landing the schemas rename? (or do you think we should batch that with the new key/u-c-c changes in a silo later in the week?) [16:18] seb128: either would be fine, could take the rename/dep drop now [16:18] Laney, let me test https://code.launchpad.net/~hikiko/unity-control-center/unity-control-center.per-monitor-fonts-scale-factor-slider/+merge/205227 to see if it's ready for landing [16:21] ok uploading mesa 10.1 [16:22] seb128: yeah, getting it all in sooner avoids the need for FFe [16:23] Laney, story of my week :p [16:23] ;-) [16:23] but I'm getting there [16:23] I've a couple of "update panels" for u-c-c I want to do [16:23] but I think otherwise we are good [16:23] well "we" doesn't include unity [16:23] those guys are screwed for ff [16:24] all aboard the ci train :> [16:25] is it too late for me to merge this for FF? https://code.launchpad.net/~attente/indicator-keyboard/ibus-panel [16:25] FF is tomorrow 2100UTC ish [16:26] so no, but ideally get someone to review for you today if possible [16:26] otherwise exceptions can happen ... [16:27] ok, i'll propose it then... [16:27] thanks Laney [16:28] attente_, I can review that tomorrow, so keep hacking on it today if needed [16:28] seb128, ok, it's just that there are two bugs that are haunting it... [16:28] attente_, what impact do they have? [16:28] today allows for iterations [16:28] if it's like "use 100% cpu while it's running" we shouldn't land [16:28] * Laney puts the curling on [16:28] lol [16:29] nothing that severe... [16:29] if they are "the menu doesn't always refresh" that's ok [16:29] attente_, I'm asking because desrt mentioned "50 dbus message by keypress" yesterday :p [16:29] or something around those lines [16:29] attente_, well, anyway, your call ;-) [16:30] attente_, I'm giving it a test in a bit [16:30] haha.. yeah.. [16:30] joke aside, what are the bugs? [16:31] the menu doesn't always refresh correctly and the radio dot is missing === Sweetsha1k is now known as Sweetshark === gatox is now known as gatox_lunch === cyphermox_ is now known as cyphermox === mjohnson15_2 is now known as mjohnson15 [16:40] seb128, fixed :) [16:40] hikiko, thanks [16:42] hikiko, the slider gives me a "0.5" value, is that normal? [16:42] hikiko, shouldn't the default be "1"? [16:42] Good morning [16:43] let me see seb128 because the value we show is not the value we store [16:44] sec [16:45] pitti, hey, how are you? [16:46] pitti, https://bugs.launchpad.net/ubuntu/+source/udisks/+bug/711549 seems to be back, do you know if that's a known issue/regression (the bug was fixed in udisk maybe udisks2 has the same issue?) [16:46] Launchpad bug 711549 in udisks "pressing physical eject button on CD/DVD drive causes icon to remain on desktop after eject" [High,Fix released] [16:47] seb128, could you pull and check again? [16:47] pitti, I tried to put a DVD in my laptop earlier to debug something else, I've put it back in the box but the mount icon is still there in unity/nautilus since this morning [16:48] seb128: hmm, it hasn't really changed in ages [16:49] pitti, well, I didn't test that in ages, the fix in udisks was working for precise, maybe it never worked for udisks2? [16:49] seb128: that's ok, no new notebooks come with a dvd drive anyway ;-) [16:49] I don't even rememver the fix [16:49] but it's been several years since I actually had a CD drive, but it perhaps also fails with a simulated CD [16:50] pitti, http://launchpadlibrarian.net/68907029/udisks_1.0.2-4_1.0.2-4ubuntu1.diff.gz was the fix [16:52] seb128: ah, I saw the bug mail [16:54] seb128: oh, I see [16:55] seb128: well, it's not really supposed to work that way [16:55] seb128: in quantal or even earlier we enabled kernel-side polling [16:55] so perhaps that broke [16:56] seb128: i. e. if you insert or eject a CD, you should get an event in udevadm monitor [16:56] pitti, ok, all I know is that udisksctl dump lists a DVD that has been ejected since this morning ... but that seems like a debug topic for after ff [16:57] seb128: ok, then it seems kernel polling for eject has been broken [16:58] pitti, do you know how to test that? [16:58] pitti, btw we don't need to discuss/debug it today, I'm happy to open a bug report and follow up in async way/later there, you probably have other things to do at the sprint [16:58] we have that bug report, right? [16:59] which one? [16:59] I thought you meant bug 1281110 [16:59] Launchpad bug 1281110 in udisks2 (Ubuntu) "cds no longer appear in the device notifier" [Undecided,New] https://launchpad.net/bugs/1281110 [16:59] I debugged one earlier where the submitter was saying that nothing happened when inserting a CD [16:59] that's pretty much the same issue [16:59] right [17:00] kernel poling for new/removed media [17:00] I just realized [17:00] I got the prompting this morning [17:00] I suppose if you do that, nothing happens in udevadm monitor -e --udev? [17:00] so I though that was working [17:00] seb128: proper scaling using that gnome key is coming to the ppa... [17:00] Trevinho, nice! [17:01] pitti, http://paste.ubuntu.com/6960826/ when inserting the CD [17:01] 7~DISK_MEDIA_CHANGE=1 [17:02] oh, that's good [17:02] seb128: and if you press the eject button, do you get an event? [17:02] pitti, http://paste.ubuntu.com/6960832/ [17:03] oh, good [17:03] so the kernel/udev parts are working still === bschaefer_ is now known as bschaefer [17:08] hikiko, thanks, that commit makes the default be 1 [17:09] hikiko, speaking about the UI, I guess the "UI Scale:" shouln't have a ":" and be right aligned with the other labels [17:09] hikiko, and the slider should take the same space as the combos [17:11] mmm I was about to leave, let me fix it quickly and push :) [17:11] hikiko, thanks, and sorry for that [17:16] fixed :) [17:16] hikiko, thanks! [17:17] no prob there was a trick I hadnt noticed [17:17] hikiko, have a nice evening! [17:17] if you check homogeneous it gets automatically the size/padding of the container :D [17:17] have a good evening everyone + seb128 thanks for the review! [17:17] attente_, ok, https://code.launchpad.net/~attente/indicator-keyboard/ibus-panel/+merge/207240 works fine for me (I've no clue about those IM option so I can only say they look like (and that the normal switching doesnt regress) [17:18] happyaron, ^ can you give an user testing round to that one, as you use IMs ;-) [17:18] attente_, but +1 from me for landing, feel free to push to the branch today, I'm going to do a landing tomorrow my morning with was is in there, if that works for you [17:20] seb128, ok, works for me, i'll try to resolve those issues today though [17:20] attente_, ok, good luck ;-) [17:20] seb128, thanks! [17:20] attente_, we can still do other landings with fixes after ff otherwise, so don't overstress about it [17:20] yw! [17:35] didrocks, sil2100: how can I request a CI train request to be dropped? [17:35] or superseeded [17:35] seb128: if it's yours, of course :) [17:35] or one you are the owner [17:36] didrocks, "how"? :p [17:36] yeah, I'm owning that one [17:36] seb128: you can run merge and clean with "only free silo" [17:36] thanks [17:36] seb128, are you going to rebuild all -doc packages to get the new gtkdoc styling? :P [17:37] xclaesse, no way :p [17:37] seb128: then, for clarity, if you are not going to reuse that line, you should put something in the status field [17:37] (and maybe some red or green) [17:38] didrocks, "status", the column that says "auto-updated, do not touch"? ;-) [17:38] seb128: yeah, if you are not going to use that line anymore, you can overwrite it [17:38] then, you won't be able to reuse it [17:42] didrocks, ok, superseeding l18 by l39 [17:42] robru: mind assigning it? ^ [17:42] sure [17:42] thx! [17:43] thanks [17:44] seb128, ok, silo 8, please build [17:44] hmmm wait [17:44] didrocks, prepare job told silo 8, but spreadsheet wont' accept it [17:45] robru: oh? [17:45] robru: backend is always right [17:45] robru: ah, I know, what happened [17:45] wait for 5 minutes [17:45] and retry [17:46] ok [17:46] robru: as the silo has just been freed, it's not synced back [17:46] robru: and I changed the sync from 1 min to 5 minutes [17:46] (to pull for backend update) [17:46] to see if it relaxes the spreadsheet issues [17:46] 8 is the one I just cleaned [17:46] in addition to my optimization [17:46] seb128, yeah, because its clean, the backend assigned it again ;-) [17:46] robru: tomorrow, I'll put it back to 1 min [17:47] we'll see if the spreadsheet goes crazy again [17:47] didrocks, ugh, now it just says line 18 is silo ready [17:48] argh [17:48] yeah, it reassign [17:48] so we have a race in that case :) [17:48] didrocks, so i have to manually switch that? [17:48] yeah, using the force free silo (admin only) [17:48] in the spreadsheet menu [17:48] then assign the other line [17:48] didrocks, i just moved the info down to line 39 ;-) === gatox_lunch is now known as gatox [17:49] robru: works as well :) [17:49] the hidden column? :p [17:49] seb128, ok, go for build now [17:49] didrocks, yeah [17:49] cheating! [17:49] :) [17:49] robru, didrocks: thanks [17:49] robru: seb128: just put something in the status column of line 18 and change the background color please [17:50] didrocks, on it [17:50] like "superseeded" and green or red :) [17:50] thanks! [17:59] new mesa landed in trusty-proposed breaks unity builds [17:59] we can't build unity now due to [17:59] Package xcb-dri3 was not found in the pkg-config search path. [17:59] Perhaps you should add the directory containing `xcb-dri3.pc' [18:00] not today... not today! :/ [18:00] Trevinho: do you have some time to help me with my decorator bug? :) [18:00] mlankhorst, ^ [18:00] didrocks: mh, not sure... let me know though :) [18:00] Trevinho: so, as seb128 told you, I have different close/minimize/maximize icons on the decoration than when it's maximized [18:01] ? [18:01] I reseted /org/gnome/desktop/interface/icon-theme and /org/gnome/desktop/interface/cursor-theme [18:01] mlankhorst: see failures at https://launchpadlibrarian.net/166779034/buildlog_ubuntu-trusty-amd64.unity_7.1.2%2B14.04.20140219.2-0ubuntu1_FAILEDTOBUILD.txt.gz [18:01] and restarted compiz [18:01] and no [18:01] so, I guess there is a bug that we don't read the same key from the decoration than when rendering the panel [18:01] mlankhorst, I guess some .pc got a Requires xcb-dri3 but the -dev shipping the .pc didn't get its depends updated? [18:01] but then, which key? :p [18:01] didrocks: mh, I'm always using the same default gtk theme... [18:01] what do I have on my user account which is so special (it's not a system issue, it doesn't happen on a guest account) [18:02] looking in standard paths: ~/.local/share/themes -> ~/.themes ... /usr/share/themes... [18:02] Trevinho: gtk-theme='Ambiance' [18:02] Trevinho: yeah but I see missing Nux/Nux.h there.. [18:02] mlankhorst: that's because pkg-cofnig fails [18:02] Trevinho: nothing in ~/.local/share/themes -> ~/.themes [18:02] didrocks, try to strace -f compiz 2>&1 | grep theme? [18:02] let's try [18:02] Trevinho, do you know how are the images for the decoration called? [18:02] mlankhorst: basically the pkg-confg gl fails, then all fails [18:03] look for xcb-dri3 on that log [18:03] oh that [18:03] mlankhorst, something Requires dri3 but doesn't have a depends on the corresponding dev I guess [18:03] http://paste.ubuntu.com/6961165/ [18:03] seb128: yeah so it would seem [18:03] so, nothing specific to my directory [18:04] mlankhorst: thanks, please consider this high prio... we've features to land :) [18:04] mlankhorst: libgl1-mesa-dev [18:04] mlankhorst, you are on it? ff is tomorrow and that's blocking unity landings [18:04] * didrocks retries [18:04] didrocks, try to grep for didrocks? [18:04] seb128: I really have to go in a few minutes :/ [18:04] didrocks, to see what it reads from you user dir [18:05] mlankhorst: If it's just that then I can do that upload now [18:05] mlankhorst, you can't let trusty broken, if you prefer I can delete your update from today ... or Laney can fix it if you are confident that's going to work [18:05] Laney: egl needs that update too, I'll try to push a fix [18:05] get to love the pre-ff issues [18:05] oh [18:06] [pid 19718] access("/home/didrocks/.local/share/themes/Ambiance/unity/close.png", F_OK [18:06] didrocks, that would do it ;-) [18:06] ah no [18:06] doesn't exist [18:06] :-( [18:06] mlankhorst: yeah I got it [18:06] laney@iota> rgrep GL_PC_REQ_PRIV | grep \.pc ~/temp/mesa-10.1.0~rc1 [18:06] src/mesa/gl.pc.in:Requires.private: @GL_PC_REQ_PRIV@ [18:06] src/egl/main/egl.pc.in:Requires.private: @GL_PC_REQ_PRIV@ [18:06] didrocks, you can -e open in the strace [18:06] Laney: sec I'll try to finish building it locally.. [18:07] okey dokey [18:07] I actually don't know of an automated tool that checks for this [18:07] that'd be interesting [18:07] didrocks: that's weird... [18:08] not sure why libgl1-mesa-dev depends on x11proto-dri2-dev [18:08] didrocks: mhmh... it's like that it doesn't fail for that file, then it uses the cairo-generated buttons (maybe)? [18:09] ok, it killed all my terminal after restarting compiz too many times :p [18:10] Trevinho: once you will be a little bit less busy, I would appreciate looking at that with you, old unity got the default icons, metacity is getting the default icons as well, only your new decorations are taking something else :) [18:10] Trevinho: but I'll let you land features first, that's something that just needs to be fixed before release :) [18:10] Laney: pushed, should hopefully fix all issues [18:10] ok [18:10] mlankhorst, thanks [18:10] if not I'm away until tomorrow morning :-) [18:11] mlankhorst, have fun, and let's hope that fixes the unity build [18:11] didrocks1: if it can help... [18:11] http://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/unity-shared/DecorationStyle.cpp#L318 [18:11] and the weird thing is that both the panel and the decorations share the same codepath... [18:11] different textures, but same "look for this file" codepath [18:12] Trevinho: yeah, there is clearly a bug somewhere :) [18:12] nice part is undoing the same thing for mesa lts-trusty backport ;-) [18:12] Trevinho: it's not different textures, it's really different icons :p [18:12] didrocks1: is GTK_DATA_PREFIX set for you there? [18:12] if I disable dri3 support entirely the backport to precise is a lot easier.. [18:12] didrocks1, did you strace -f -e open and try to grep on e.g png? [18:12] Trevinho: I saw that, I don't think so, but I'll look in a clean session [18:13] seb128: well, that's what killed my session, I'm under metacity launched from a tty [18:13] so, I'll look tomorrow morning [18:13] didrocks1, k [18:13] to ensure I have the right env variables [18:13] reminds you of the good old time right? ;-) [18:13] not sure we have the same definition of "good" :) [18:13] but yeah, it's like a flashback [18:13] I should thanks Trevinho ;) [18:13] * Trevinho done that for you [18:14] :D [18:14] hey, welcome to the cool style session [18:14] * didrocks1 hugs Trevinho :) [18:14] I thought you loved the flashbacks... so... there's a special easter egg for didrocks user [18:14] heh [18:15] * didrocks1 will grep unity new code to look for if username == didrocks [18:15] ah didrocks1 I see the fosdem page has not been updated (not that I care much)... [18:15] Trevinho, bregma: hopefully https://launchpad.net/ubuntu/trusty/+source/mesa/10.1.0~rc1-1ubuntu2 does it for the unity build [18:15] Trevinho: yeah, it will take weeks apparently :p [18:15] but I sent all infos [18:15] that would be too clear... It's a new kind of coding: "whitespace encoding"... [18:15] heh [18:15] tabs, space... like 0 and 1... [18:16] and i was really aiming not to break unity this time :P [18:16] anyway, see you tomorrow guys! [18:16] cheers didrocks1 [18:16] didrocks1, night! [18:17] seb128: should we wait to rebuild or is it already there? [18:18] I'm watching the rebuilds [18:18] mlankhorst: thanks btw [18:19] Trevinho, bregma: you need to wait that rmadison -s trusty-proposed shows the new binary [18:20] which should be less than half an hour [18:20] take a cup of coffee, or 2, watch your favorite movie ;-) [18:20] the more coffee I drink the longer everything seems to take [18:20] unfortunately these are not the days we can, but i would love to :) [18:25] Laney: as per your suggestion, i submitted a bugreport. not that i expect that you process this, but for the future: is this okay for merging/uploading or would you expect more/other infos? https://bugs.launchpad.net/ubuntu/+source/xubuntu-default-settings/+bug/1282154 [18:25] Launchpad bug 1282154 in xubuntu-default-settings (Ubuntu) "update default settings for trusty" [Undecided,New] [18:42] Trevinho: https://launchpad.net/ubuntu/trusty/+source/mesa/10.1.0~rc1-1ubuntu3 is what you want to look for btw, there were more dependencies missing [18:42] Sarvatt, thanks [18:43] Sarvatt: ah [18:43] bregma: ^ [18:43] * bregma sighs [18:45] bregma, Trevinho: well, the previous one might be enough for unity [18:45] let's try... :) [18:45] we need to make unity to fail in a better way in these cases though [18:45] or well instruct cmake to do so [18:46] yeah, well handling build nicety is for another day I guess [18:46] let's focus on landing features for this week [18:47] I haven't seen 10.1.0~rc1-1ubuntu2 land yet, it's till waiting to build [18:47] bregma, on what archs? [18:48] nm, I see it now [18:49] rmadison was not telling me, but I see it on launchpad [18:50] what did you rmadison for? [19:19] * desrt caved in [19:20] We appreciate your business at dell.ca. We will send an e-mail confirmation of your order to desrt@desrt.ca within 2 business days. This e-mail will include: [22:02] robert_ancell, guh, dumb me. That last diff was so large with all the test changes, I overlooked the l_debug line [22:03] robert_ancell, fixed [22:11] mterry, did you see the following comment? [22:11] robert_ancell, nope! [22:11] * mterry reads [22:12] robert_ancell, eh, the start session call includes XDG_GREETER_DATA_DIR in it [22:12] robert_ancell, we match pretty loosely, but it at least confirms that the dir was created/passed to the session [22:12] mterry, yeah, it's no a biggie. Just wondered if you'd noticed [22:12] robert_ancell, I mean that the call includes XDG_GREETER_DATA_DIR in the test notify line [22:13] robert_ancell, I had. But I didn't want to bother doing the .*/username thing again. I just wanted to confirm it existed [22:13] so left it as TRUE/FALSE [22:13] mterry, I just ran make check here. some tests need updating e.g. test-additional-config [22:14] hmm.. Maybe I didn't merge from trunk? Everything passed for me and jenkins... [22:14] hang on, let me try some more [22:14] this is a fresh checkout [22:15] mterry, ah, the chown is failing under test [22:15] mterry, we don't fake it [22:16] robert_ancell, right, I did sudo make check. I thought it was understood that tests were run as root? [22:16] mterry, no, they can run as a normal user [22:16] I see... [22:17] robert_ancell, the chown calls are in test-runner, which doesn't run under libsystem, right? [22:17] robert_ancell, my new ones weren't the only chown calls in there, so I figured it was assumed that root was running it [22:17] mterry, it does. If I make chown return 0 they work, except for the LIST-SHARED-DATA-DIRS call which now can't check the permissions easily [22:17] yeah, not sure what that was added for previously [22:18] robert_ancell, to make the home dirs of the other users correct permission wise [22:18] ah, we only do the existing chown if getuid() == 0 [22:19] mterry, tell you what, let's just land it and I'll make a new MP to get the tests working as non-root again [22:19] robert_ancell, but this instance isn't optional. Wait, test-runner is run under libsystem? I didn't think so... it's called directly from the test-* scripts [22:20] mterry, yes, you're right [22:20] robert_ancell, and the existing chown call in there isn't guarded by getuid()... which call are you looking at? [22:21] mterry, the one in src/session.c [22:21] robert_ancell, oh sure, in the source itself [22:21] robert_ancell, but there is another call in test-runner.c for chowning fake user dirs. I assumed that implied test-runner needed root [22:21] robert_ancell, I wasn't sure how to get the functionality I needed without either running under libsystem or being root [22:22] mterry, yeah, let's solve it later. It's not a huge problem [22:26] mterry, all tests are passing for me except for test-shared-data-dirs-autologin [22:26] it only lists one shared data dir created [22:27] * mterry looks again [22:28] mterry, the test expects a "lightdm" directory to exist, but I can't see why that should be [22:28] robert_ancell, the greeter's session creates it (which is desired, as the greeter may want it to stuff temporary files like camera photos before shunting them to the proper user dir) [22:29] robert_ancell, I didn't add the variable printout to greeter session starts in scripts though [22:29] robert_ancell, figured that it was largely uninteresting usually. It was more important to test the various session cases [22:29] mterry, why not use a normal cache dir? It's not really a shared dir if it's only shared with itself [22:30] robert_ancell, this way it's on the same partition and is guaranteed to stay across reboots and such [22:30] robert_ancell, could use greeter's HOME too I suppose [22:30] robert_ancell, but everything in the same place seemed nice [22:30] mterry, yeah, there are existing locations for things like this [22:30] robert_ancell, because on phone right now HOME is cleared after every boot [22:31] robert_ancell, (greeter's HOME that is) [22:31] mterry, the test failure http://paste.ubuntu.com/6962479/ [22:31] robert_ancell, we could change that, but it's easier if we just poke a hole for lightdm-data directory [22:32] robert_ancell, yeah that test shouldn't have the lightdm directory, since the greeter isn't involved. Not sure how my tests or jenkins let that slide [22:33] * mterry runs tests again [22:33] mterry, jenkins doesn't run the tests [22:33] robert_ancell, ah that explains that [22:33] robert_ancell, but anyway. All the existing locations I didn't like (they all are cleared after boot) [22:33] on phone anyway [22:34] it's debatable whether we should keep greeter's HOME but if we can avoid doing it, all the better [22:34] mterry, ok [22:35] mterry, you can share between greeters with that directory anyway, so I guess it makes sense [22:35] robert_ancell, oh god. A user takes photos with one greeter, then reboots and uses another greeter? And then logs into a user? I hate lightdm use cases [22:36] Not that that's much different than using HOME [22:36] no [22:36] (it's not) [22:37] robert_ancell, for me, test-unity-compositor-fallback keeps failing in 'make check' but not when I manually run the test. Does that make any sense? (don't think it's related to my changes) [22:37] mterry, yeah it's being unreliable for me but I don't think it's related either [22:38] this is why jenkins isn't running the tests [22:38] robert_ancell, well, test that failed for you is updated [22:38] (it failed for me too) [22:38] Must have been focusing on the other new test and forgot to rerun the autopilot one [22:38] mterry, approved, thanks for the patience! [22:39] robert_ancell, np, thanks for the stickler reviews :)