[00:50] jasoncwarner, no, but xnonx proposed we generate it https://code.launchpad.net/~xnox/unity-greeter/off-the-grid/+merge/205896 === duflu__ is now known as duflu [08:10] Trevinho: hum, it seems unity doesn't respect disabling keyboard shorcut for menu anymore [08:11] Trevinho: like I'm used to use Alt + a on weechat, (I disabled it in gnome-terminal to not use the menu shortcut), but now, alt + a displays "Affichage" (View) menu… [09:02] morning [09:02] hey Laney [09:02] desrt: https://bugzilla.gnome.org/show_bug.cgi?id=722604 [09:02] Gnome bug 722604 in general "Various tests are failing with 2.39.3" [Normal,Unconfirmed] [09:03] howdy larsu [09:03] how do? [09:03] very good, thanks. How about yourself? [09:03] * larsu is trying to fix black-window-syndrome again [09:04] of course, it involves overlay-scrollbar. Again :/ [09:04] urgh [09:04] ya [09:04] I'm good [09:04] ogra_ was complaining yesterday that his evolution wasn't fully fixed btw [09:05] yeah, i have a black frame around the message view [09:05] but i didnt re-login, i only killed evo and restarted it [09:05] ogra_: did you get both updates? (evo and light-themes?) [09:05] do i have to log out ? [09:05] no [09:05] yes [09:06] which theme? [09:06] i will have to check once i'm near my laptop ... Ambiance i think [09:06] * Laney dist-upgrades to check [09:06] it is still broken in adwaita iirc [09:06] good morning desktopers [09:06] hey seb128 [09:06] hey Laney larsu, how are you? [09:06] seb128: did you test the theming fixes? [09:06] seb128: awesome :) [09:07] larsu, yes, it fixes the preview widget color for me [09:07] prima! [09:07] seb128: ogra_ still sees a black background around the mail itself (where the mail headers are) [09:07] oh, that I didn't test [09:07] I don't have that though and was wondering if somebody else did [09:08] wasn't that the part in evo itself than Laney sponsored? [09:08] the revert of the revert [09:08] I didn't notice this issue either [09:08] that was the symptom of the original bug [09:08] that you said wasn't necessary any more [09:08] the revert [09:08] ya, it isn't [09:09] I'm running the package you gave me and my theming fixes [09:09] and it works fine [09:09] lemme upgrade [09:10] no issue here either [09:10] texlive-latex-extra-doc! [09:10] y u so huge [09:11] did anyone got unity8 and co pulled on their dist-upgrade? [09:11] seb128, I saw it wanted to be pulled in [09:12] don't think so [09:13] I saw that there was a problem with recommends from (I think) indicator-datetmie [09:13] what problem? [09:13] or maybe u-c-c [09:13] the version was wrong so it satisfied the alternate which was some touch thing [09:13] that could be the problem [09:13] yeah [09:13] do you remember which package/version it was? [09:13] checking [09:13] that would be useful info [09:13] thanks [09:14] ah [09:14] a recommends that refers to touch would be pretty pointless anyway [09:15] (since touch has recommends disabled [09:15] ) [09:15] erm [09:15] ogra_, we install recommends by default, it makes unity8 being pulled on desktop upgrades [09:15] it's a recommends from indicator-datetime [09:15] so anyway it was unity-control-center (>= 14.04.3) | ubuntu-system-settings, [09:15] seb128, yes, but if the recommends is there for touch stuff it is pointless [09:15] from indicator-datetime [09:15] gnome-control-center-datetime | ubuntu-system-settings [09:15] but there's a new u-c-c now [09:15] so at least that one is now alright [09:15] oh, right [09:15] so maybe it's fixed [09:16] what's that paste? [09:16] old version in my apt-cache show [09:16] I've several versions and picked the wrong one [09:16] if you still see it you can diagnose why [09:16] you are right, should be fixed [09:16] no, I don't see it here [09:17] nod [09:17] I saw it on another box yesterday evening, I guess that was before the new u-c-c landed [09:17] if it didn't have u-s-s then likely [09:17] oh hey, I have evo black borders too! [09:18] right, the box didn't have u-s-s [09:19] it was a new install from this week on the laptop I got for touch testing [09:19] I didn't change anything since the install [09:19] so it was pretty stock trusty [09:19] ok [09:20] but yeah, u-c-c .3 was published 8 hours ago [09:20] so pretty sure that's it [09:20] * Laney tries old evo [09:22] yeh fixed here also [09:22] yeah, that fixes it :/ [09:26] larsu: http://ubuntuone.com/1pJMJrUIRBSSaqECfuvgvX like that [09:26] so it was the patch that got dropped yesterday that fixes it? [09:26] seems it is still needed, for me at least [09:26] why wouldn't everyone see it though? [09:26] moin! [09:32] Laney, well, thats exactly what i see [09:32] you and me do not everyone make [09:32] as sad as that is to admit :P [09:34] Trevinho, bregma, you guys have lot of good work that needs to land, you should file landing ask at least once a day [09:34] Trevinho, bregma: need to keep things rolling ;-) [09:34] hey Sweetshark [09:34] seb128: Can you please sync mdds 0.10.2 from experimental? LO 4.2.1~rc1 needs it, and possibly we want to put ~rc1 into the archive too, as there are some bugs in Calc that its better to push a rc than leaving stuff as is. [09:34] Laney, I wasn't seeing that yesterday, didn't try the version you uploaded yesterday yet [09:35] okay, try upgrading evo then when you get a chance [09:36] https://git.gnome.org/browse/glib/commit/?id=c37cd19feee3a609fec8909f01df8755052c59ab [09:36] HAHA [09:36] LOL [09:36] larsu, ^ :p [09:36] lol [09:38] Sweetshark, https://launchpad.net/ubuntu/+source/mdds/0.10.2-1 [09:43] seb128: thx [09:43] Sweetshark, yw [09:44] Laney: Linux does endless loops in five seconds. Wasnt glib prepared for that? [09:50] Laney, I can confirm the bg color issue for the frame widget [09:52] nod [09:53] hum, need to run for half an hour, bbiab [10:20] Laney: are you sure the package you gave me yesterday had the revert reverted? [10:20] larsu: that's what I installed to take the screenshot [10:20] It would have to, otherwise the theming fix wouldn't make any sense... [10:20] weird. I have the same package installed and it wfm [10:20] install the one from trusty to be super sure [10:24] ya [10:25] Laney: wtf? It works with that one as well... [10:25] O_O [10:25] oh wait, that's already the new one, right? [10:25] ubuntu4 is broken [10:26] I have ubuntu3 [10:26] get 4! [10:26] * larsu updates [10:26] 3 should work [10:26] it was broken for me yesterday, before I fixed the theme [10:26] (ubuntu3, I mean) [10:26] yeah [10:26] three was all black yesterday, but with the theme fix it looks correct now [10:27] four has the black border like in the screenshot [10:27] which is exactly how it looked before the revert I did [10:27] it looked all black for me (frame and mail widget) [10:27] yeah [10:28] I wonder what's different between the one you gave me and the one in trusty, then [10:29] nothing [10:29] if you have ubuntu3 then you don't have the one I gave you though [10:29] I had ubuntu4 [10:29] I installed ubuntu3 after you said "use the one from trusty" [10:29] (didn't update before) [10:29] ah [10:29] (updating now) [10:29] well they're the same, like I said I used those debs to take the screenshot [10:30] okay, I can reproduce [10:30] cool [10:36] seb128, hi [10:45] seb128, if you need to rename the gsettings-ubuntu-touch-schemas, could you then ping me to add the correct dependency in this branch: https://code.launchpad.net/~hikiko/unity-control-center/unity-control-center.per-monitor-fonts-scale-factor-slider/+merge/205227 ? +thanks! :) [10:48] hikiko, hey, sure [10:49] larsu, want another theming issue? ;-) [10:51] no?! [10:52] * larsu is totally confused by how the evolution thing happened [10:52] I definitely had ubuntu4 installed and it worked [10:53] did you have hacks/changes in your theme that didn't get merged and were overwritten by today's theme update? [10:53] no, I diff'ed them [10:53] (before and after updating) [10:56] weird [10:57] larsu, out of themes issues did you want to look at nautilus menus? (did you start on that?) [10:58] yes, I started on that but got distracted by evolution and baobab and glade [10:58] * larsu thought he fixes all the theme things once he's at it [10:58] larsu, sorry about that [10:58] meh, gotta fix it some time [10:58] too bad I'm in o-s code again though :) [10:58] larsu, it would make sense to put those on hold, ff is this week and ideally the menu change should land for it [10:58] ah right [10:59] well, spending some hours is fine I guess, but it would be better to not end up with that eating your week [10:59] we can fix theming issues after ff [10:59] agreed [10:59] btw the one I was about to mention is [10:59] open an archive in file-roller, dnd a row, look at the bg color [10:59] (same in language-selector) [11:04] seb128: noted [11:04] that seems to be fallout from the no-bg patch [11:04] right [11:04] should be easy t ofix [11:11] * didrocks wonders where the icon theme is set on touch… it's not in gsettings [11:15] waow, it's hardcoded in unity8 it seems [11:15] and it seems it's larsu's who last touch it! :) [11:15] didrocks, I would guess something like that [11:15] lol [11:15] didrocks: I did? [11:15] [ Lars Uebernickel ] [11:15] * Fall back to "ubuntu-mobile" icon theme if $UBUNTU_ICON_THEME is [11:15] that's probably quite old [11:15] unset. [11:16] daily release pointed at you! :) [11:16] (ok, you probably just did the fallback :p) [11:16] didrocks: ah right. They were _only_ using $UBUNTU_ICON_THEME [11:16] I added the fallback [11:16] (after yelling a lot at them) [11:16] you touched it though! :p [11:16] ahah [11:16] but we don't set UBUNTU_ICON_THEME on the phone [11:18] so, I guess then all apps inherits from tihs [11:18] this* [11:19] I don't think so... [11:19] probably the sdk calls QIcon::setTheme [11:19] it's weird, I don't see anything in the sdk [11:19] hm [11:19] and they use QIcon::getTheme [11:19] I wouldn't know how apps could inherit it from unity8 though [11:20] yeah, there is no env variable involved [11:20] hum [11:22] indeed [11:25] didrocks: I guess apps don't use themed icons but hardcode everything? [11:25] larsu: well, the sdk is using QIcon [11:25] so something is setting it for the sdk to have access to the theme [11:26] right, but I think it's only unity8 that uses themed icons from the sdk [11:26] not apps [11:26] and unity8 does call QIcon::setThemeName [11:26] don't tell me that :/ [11:26] hm. Sorry. [11:26] it's terrible if it's the case [11:26] let me look [11:26] like dialer-app [11:27] there are dialer icons in the theme [11:27] search for "image://theme" [11:27] I bet they're not using that but "file://" urls [11:27] * larsu hopes he's wrong, though [11:27] but I remember being angry about this at some point last year [11:27] larsu: well, if they did that, they can't use QIcon [11:27] $ grep -r mobile * [11:27] src/dialerapplication.cpp://On desktop use: export DIALER_APP_ICON_THEME=ubuntu-mobile [11:27] hum, nothing else [11:28] if (!iconTheme.isEmpty()) { [11:28] QIcon::setThemeName(iconTheme); [11:28] } [11:28] and where does "iconTheme" come from [11:28] larsu: from the env variable [11:28] there you go :) [11:28] and it's the only setThemeName call [11:29] larsu: it would mean that unity8 is setting that for every app? [11:29] sounds crazy [11:29] yeah... [11:29] let me launch dialer-app [11:29] and look at its env variable [11:30] cat /proc/4010/environ | grep THEME [11:30] root@ubuntu-phablet:/# [11:30] phew [11:30] so no [11:30] not *that* bad :p [11:30] are you sure they're using themed icons? [11:31] I guess at least for the hud [11:31] $ grep -r hud * [11:31] -> nothing [11:31] ok, not that… [11:32] didrocks, larsu: dialer-app has src/dialerapplication.cpp: QByteArray iconTheme = qgetenv("DIALER_APP_ICON_THEME"); [11:32] seb128: scrollback, you are lagging dude! :) [11:32] didrocks, I misparsed what you wrote [11:32] what are you trying to figure out? [11:32] seb128: this part isn't used [11:33] seb128: if you start dialer-app on the phone, the env variable isn't set [11:33] so setThemeName() isn't called [11:33] didrocks: dialer-app doesn't make use of any themed icons [11:33] larsu: it can show the hud though, right? [11:33] (so the hud icon) [11:34] which is part of the theme [11:34] (through the sdk) [11:34] what's that icon called? [11:35] hud.svg in /usr/share/icons/ubuntu-mobile/actions/scalable/hud.svg [11:35] no mention of that anywhere in the sdk or the dialer-app... [11:35] indeed [11:35] hum [11:35] let's take an example which is using the theme icons [11:35] and the icons I do find are included via relative urls [11:35] ../assets/[] [11:36] no mention of image://theme/icon-name [11:36] which is the only way to include themed icons from qml [11:36] ok, let's try system-settings [11:36] (I know because I wrote that after finding out they were not using themed icons at all) [11:37] larsu: I see some QIcon usage in the sdk though, but you think, it's only used in the unity8 code? [11:37] didrocks, we force the theme in u-s-s [11:37] "nice" [11:37] crazy we don't have a default theme [11:37] in the sdk [11:37] src/main.cpp: QIcon::setThemeName("ubuntu-mobile"); [11:37] ok :/ [11:37] didrocks: I only find QIcon in the code that I wrote, which exposes themed icons to apps [11:38] but no actual usage of themed icons in the sdk [11:38] "excellent" [11:38] ya [11:38] but though [11:38] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1098578 also [11:38] Launchpad bug 1098578 in Ubuntu UI Toolkit "On Ubuntu Desktop, the icon theme used by Qt is always 'gnome' (instead of the user set one)" [High,Confirmed] [11:38] if I try notes-app [11:38] https://bugreports.qt-project.org/browse/QTBUG-25718 [11:39] (qt bug, it doesn't respect the xsettings icon theme config, though that's an issue for desktop, not touch) [11:39] I don't understand why we have themed add button [11:39] in the toolbar [11:39] let me grep for that [11:39] seb128: the first bug is the reason I added the fallback to unity8 [11:39] NotesApp.qml: iconSource: Qt.resolvedUrl("Components/graphics/add.png") [11:40] /usr/share/notes-app/Components/graphics/add@18.png [11:40] so they ship it… [11:40] larsu: I don't understand how the hud is working, but I guess it's maybe a separate process after all [11:40] you probably want to talk to Saviq or somebody who know the details of what they are doing [11:40] rather than reverse engineer the behaviour ;-) [11:41] didrocks: that's how it is on the desktop - don't know about the phone [11:41] * didrocks feels even before talking that the whole icon theme switch is going to be painful [11:41] might be the same. Convergence! [11:41] larsu: on the desktop, unity is drawing the hud content [11:41] Saviq, help, where are apps on the phone getting their icon theme from? [11:41] larsu: but maybe it's the hud process showing the hud content on the phone… [11:41] I would guess that it's unity8 [11:42] oh yeah, more possible [11:42] and so, we have the theme icon [11:42] which sets its theme [11:42] ok, that's going to be *fun* [11:42] :) [11:43] why is this an issue right now? Are we changing themes? [11:43] larsu: yeah, from ubuntu-mobile to suru [11:43] suru? [11:43] a new one [11:43] is that just a name change? [11:43] oh, interesting [11:43] no, size changes [11:43] so we'll need unity8 adjustement [11:43] some application icon names changes [11:45] indicator icons marging are changing [11:45] so we'll need other tweaks landing at the same time [11:45] Saviq: so, you will need at least that: https://code.launchpad.net/~didrocks/unity8/suru-switch/+merge/206907 [11:47] didrocks, ok, the indicators should handle it fine [11:47] Saviq: even with the removed margin? [11:47] didrocks, they're grid-unit-aligned in the panel [11:48] ah nice [11:48] didrocks, so it should be fine [11:48] ok, so only my branch with the icon theme [11:48] seb128: want me to add the system-settings one at the same time? [11:48] (to be clear, we won't land that today, just to prepare) [11:48] didrocks, if you want, I can do it as well, as you prefer [11:49] well, it's quick enough [11:49] then, the longer part will be to have the apps icon name which changed to rename so that we can drop the dependency on ubuntu-mobile-icons [11:51] ah I know how seb128 knows about the bug report :p [11:52] seb128: you tried to fool us like if it was the awesome bar and that firefox was better! :) [11:52] but there is a comment in u-s-s: /* HACK: force the theme until lp #1098578 is fixed */ [11:52] Launchpad bug 1098578 in Ubuntu UI Toolkit "On Ubuntu Desktop, the icon theme used by Qt is always 'gnome' (instead of the user set one)" [High,Confirmed] https://launchpad.net/bugs/1098578 [11:54] seb128: FYI: https://code.launchpad.net/~didrocks/ubuntu-system-settings/suru/+merge/206908 (but same, no to switch yet) [12:04] thanks larsu and seb128 for looking as well :) [12:04] yay for giving up [12:04] I was just submitting an MP with "wah wah woe is me, it doesn't work, take this half finished solution" when the problem became visible [12:04] ;-) [12:11] didrocks: should we change the fallback on unity8 as well? [12:11] larsu: this is https://code.launchpad.net/~didrocks/unity8/suru-switch/+merge/206907, right? [12:12] oh, didn't see that one yet [12:12] thanks :) [12:13] didrocks: looks good to me, but I'll let a unity dev approve [12:13] larsu: thanks, anyway, we'll need a silo with that change and the new icon theme [12:14] right [12:21] didrocks, thanks [12:22] yw [12:22] didrocks, (back from lunch), just for the record I found the qt bug with the awesome bar, and I knew about it because I added that code and comment to u-s-s :p [12:22] seb128: yeah yeah yeah :p [12:25] Laney, https://launchpad.net/~ci-train-ppa-service/+archive/landing-001/ btw if you want to give it a round of testing, include my column fix for privacy and your 2 fixes from friday [12:26] Laney, the new one you submitted is going to be for the next landing round [12:26] (the ppa was already set up earlier today before you submitted it) [12:35] seb128: righto [12:37] seb128: I'm wondering if we should turn off the initial update check while that bug is still around btw [12:37] any opinion on that? [12:37] Laney, thanks, I just run autopilot and it's happy, let me know if/when you test so I can press the landing button [12:41] bregma, hey [12:41] seb128, I see you watching me [12:41] bregma, Trevinho was just giving me a list of merge requests in query for a landing and I see you are just putting that in ;-) [12:42] sil2100, didrocks: can we get a silo for l16, ff is this week and unity/compiz has some backlog, we are going to need to iterate over landings ;-) [12:42] *sigh* the impatience of youth [12:43] bregma, you are going to thank me once ff kicks in :p [12:43] bregma, I still don't see how you guys are going to land everything you have in the queue [12:44] seb128: looking [12:44] sil2100, thanks [12:44] Ah ha! [12:44] Assiging [12:44] thanks [12:49] bregma, seb128: assigned! [12:49] sil2100, bregma: thanks, build started === gatox is now known as gatox_brb [12:59] Laney: i managed to track down the cause of the two really offensive ones [13:00] desrt: yo [13:00] I saw the comments about the bus_down() thingy [13:00] we marvelled at your interim fix there earlier [13:03] :) [13:03] note that i said "this must be reverted" :) === alan_g is now known as alan_g|lunch [13:04] but uh ya... basically someone wrote a patch and said "i don't really understand this, but this seems to fix it" === MacSlow is now known as MacSlow|lunch [13:04] and someone else said "great! accepted!" without really reviewing the patch... [13:04] and now we're here [13:05] good morning desrt [13:05] good morning :) [13:05] bregma, Trevinho: https://launchpad.net/~ci-train-ppa-service/+archive/landing-006 has compiz/unity building [13:06] seb128: Umm I get two failures [13:06] they aren't related to these landings though [13:08] mmm. i love it. the freebsd guys are mostly taking care of themselves now and this morning, a 'jhbuild on openbsd' wiki page pops up. [13:09] Laney, which ones? [13:09] test_imei (it's N/A for me) [13:10] aaaaaaaaaaand test_updates_not_in_main [13:10] did you get an apport prompt about the service? [13:10] on the phone [13:10] (did you run on desktop or phone) [13:10] hum, k [13:10] imei being N/A seems buggy [13:10] it's a weird test [13:10] I might run on the phone without a sim card [13:11] oh ok, yeah I'm running without a SIM [13:11]