[03:10] Hello. What does unity-panel-service consist of? It's using 7.9 GB GiB on my system, is this normal? [03:10] I'm running a BTRFS balance, btw. [03:38] AndyChow_888: It consists of code. Sounds like a leak. Please log a bug here: https://bugs.launchpad.net/ubuntu/+source/unity/+filebug [04:20] Ok, reported as: https://bugs.launchpad.net/ubuntu/+source/unity/+bug/1641820 [04:20] Ubuntu bug 1641820 in unity (Ubuntu) "Memory leak" [Undecided,New] === oao is now known as jjb === JanC is now known as Guest44518 === JanC_ is now known as JanC [14:19] mterry, hey, just noticed your comment about gdk pixbuf after actually finishing it [14:20] pstolowski: sorry :( [14:20] mterry, if you have a better solution feel free to push it === dandrader is now known as dandrader|afk [14:21] pstolowski: well there is this cross-project shared script that sets up a lot of this stuff for us, and I figure we should use that to be on same page [14:21] https://github.com/ubuntu/snapcraft-desktop-helpers/blob/master/qt/launcher-specific and others [14:21] It happens to solve the mime and pixbuf stuff for us [14:23] mterry, ah, right. i didn't think of looking there :/ === dandrader|afk is now known as dandrader [14:45] ltinkl: heads up, I just committed a change that uses the snapcraft-desktop-helpers bit and pulls in locales-all. So your wizard branch should be unblocked I'd guess [14:45] mterry, kk thanks, I'll merge in [14:46] mterry, but it's blocked on the fact we don't quite agree what to show in the wizard (and uss), whether all the available locales or just those with translations [14:46] tedg: with current fresh u8 snap, snaps aren't launching (but legacy apps are)... [14:47] ltinkl: you mean u8 and uss disagree or there's a discussion somewhere among product people? [14:47] mterry, u8 and uss will show the same stuff (as we reuse their plugin) but we had a discussion earlier with Saviq [14:50] mterry: Man, you're screwing with me, I saw the notification while in another channel and couldn't figure out what was happening :-) [14:51] mterry: Are they showing up in the scope? [14:51] tedg: ye [14:51] s [14:51] Okay, I've just got the failed signal left to connect and then I can go back to getting these branches stable again. Stop adding things for a bit. [14:51] tedg: error like "Executing XMir on PID: 478XMir has closed unexpectedly" [14:52] Hmm, that's not good. Do you get a denial in U8 logs? [14:53] tedg: looks like u8 just gets a "ApplicationManager::onProcessFailed" event -- no other related log messages about the failure [15:04] ltinkl: you say "~/snap/unity8-session/current/.config/user-dirs.dirs has still values pointing to “x1”" -- I'm not seeing that... [15:05] ltinkl: did you start fresh or is this some upgrade-from-old-version difficulty? [15:05] mterry, not sure know, will have to try again, you did too many changes :p [15:05] ltinkl: fair :) [15:05] progress! [15:10] mterry, are you building on top of the ubuntu-app-platform? [15:11] ltinkl: no... not yet [15:12] mterry, the diff now looks more sensible :) https://code.launchpad.net/~lukas-kde/unity8-session-snap/locale-fixes/+merge/310864 [15:13] mterry, what about "shared-mime-info", guess we need it too? [15:13] ltinkl: shared-mime-info is already in [15:14] mterry, umm where does it get pulled in? [15:14] ltinkl: as is dmz-cursor-theme from your brnach [15:14] mterry, via the desktop helpers? [15:15] ltinkl: that list of packages in snapcraft.yaml pulls in dependencies [15:15] ah [15:15] mterry, as long as we're building from debs right :) === dandrader is now known as dandrader|afk [15:16] ltinkl: so I'd say take our your package changes that aren't about language-packs [15:16] and indicator-keyboard [15:17] mterry, yup... since Saviq wants to bundle all langpacks as a start, I'm gonna include them all [15:17] mterry, then we have to figure out how to query for those in uss/u8 [15:17] ltinkl: also don't need qttranslations5-l10n [15:17] mterry, and show them instead of _all_ locales [15:18] mterry, qttranslations5-l10n also fetched as a dep? [15:19] yup [15:19] ltinkl: ^ [15:19] kk [15:20] smoke testing now, hopefully can land soon, looks fine besides [15:36] Trevinho: hey! [15:37] Trevinho: I see an unity SRU silo in Bileto ready for publish, but the bugs it tries to fix are not prepared from the SRU formality side [15:37] Trevinho: (i.e. I don't see all the bugs using the SRU template) [15:37] Trevinho: could you take care of that? [15:38] sil2100: sure, it's something we want to do... But i didn't publish it yet because of that and because andyrock has some more tests to do on it [15:38] Ah, ok, then good I didn't publish it [15:38] As it appears as 'publishable' right now [15:52] mterry, fine with this? https://code.launchpad.net/~lukas-kde/unity8-session-snap/locale-fixes/+merge/310864 [15:53] ltinkl: yeah looks good, just going to give it a sanity test [15:53] ltinkl: though wait, where did the wizard re-enablement go? [15:53] mterry, we can't re-enable it (just yet) [15:53] ltinkl: oh bummer [15:53] what's the issue? [15:54] mterry, wrote earlier ^^ basically Saviq and tsdgeos don't agree with the fact that we'd list all the available locales [15:55] mterry, so I gotta find a way to query for those langpacks, instead of locales [15:55] s/available/existing/ [15:55] yeah [15:55] locale != language [15:55] ltinkl: I see -- I didn't realize that was a blocker. k [15:55] Saviq, sure but it still sets LC_ALL=foo as a result [15:56] but you choose a language, not locale - setting locale is a side effect, because that's likely the right thing to do [15:56] it'd actually make more sense to base locale off of the time zone setting :P [15:57] Saviq, kind of, yes; but that's another discussion [15:57] yup, which is why I don't want to expose *locales* in the wizard, because it doesn't say "Locale" at the top :) [15:57] Saviq, I don't disagree with you, best would be imho to have the lang/locale separation somewhere, the list of translations is too limiting imo [15:58] ltinkl, yes, which is what we discussed before, language + separate settings for time, date, monetary preferences, not tied to language/locale at all [15:58] ltinkl: i don't agree we should list all locales in the language selection screen [15:58] this is for selecting languages not locales [15:58] or at least it's called language-plugin [15:59] you can set defaults for those based on language, but even *I* really wouldn't know what to choose from a list of 200+ locales to have my preferred setup [16:00] the worst thing there is that probably 95% of those locales choose from what? 3 different configs? === dandrader|afk is now known as dandrader [16:01] so I'm fine with those being used as a source of defaults, but not with exposing the user to that [16:11] mterry, you may want to rename desktop/qt5 to desktop-qt5 (if i remember correctly) [16:11] to avoid the deprecation warning from snapcradt [16:17] pstolowski: will it find it online? [16:17] pstolowski: that's a cloud part [16:20] mterry, yep, it should work, that's what a number of projects in snappy-playpen uses (I've just checked) [16:20] pstolowski: ah great thanks [16:22] mterry, see also https://wiki.ubuntu.com/snapcraft/parts [16:25] pstolowski: ok updated [16:25] thx [16:26] yw === JanC_ is now known as JanC [17:01] mterry, the paths in user-dirs.dirs are still not good :/ moreover everything (like .cache, .local, etc) got duplicated between ~/snap/unity8-session/x13 and ~/snap/unity8-session/common [17:01] mterry, interestingly, the common dir has the paths right [17:07] ltinkl: right it would [17:07] ltinkl: the ones in x13 are not used anymore [17:07] ltinkl: snapd automatically copies those files around to the new version [17:07] you can delete them and it'd be fine [17:42] ltinkl: language packs for br and cy aren't in zesty? [17:43] oh I may not have overlay installed [17:45] mterry, those are coming from phone, so yeah overlay [17:45] mterry, and, indicator-datetime didn't start in u7 again, even after reboot [17:46] ltinkl: well do you know why? [17:46] like is there an erro? [17:46] error [17:49] mterry, where should I look? [17:50] ltinkl: ... i'd start with indicator-datetime's upstart logs [17:50] in ~/snap/unity8-session/common/.cache/upstart [17:52] mterry, in u7 it doesn't start, which in fact might be related to silo 2195 [17:52] mterry, that's the log: http://paste.ubuntu.com/23481550/ [17:53] ltinkl: oh sorry, u7. sure. well the only change in the silo is prefixing the command with $SNAP -- do you think SNAP is defined in your u7 environment? [17:53] mterry, nope [17:54] ltinkl: that log doesn't look great no. You could try purging the ppa and trying again. But I really doubt this is caused by the indicator-datetime branch itself. Possibly something else in the silo? [17:55] mterry, yeah... dunno at this point; it starts tho if I launch the -service manually === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [22:40] Hi [22:40] anybody fromthe design team here? [22:40] how can you suggest things? [22:41] design wise?