[06:33] <oSoMoN> good morning desktoppers
[06:34] <didrocks> good morning!
[06:35] <oSoMoN> salut didrocks
[06:40] <didrocks> ça va oSoMoN ?
[06:40] <oSoMoN> didrocks, ça va, et toi?
[06:41] <ricotz> hey desktopers
[06:42] <oSoMoN> hey ricotz
[06:44] <didrocks> oSoMoN: ça va bien :) il commence à faire un peu plus chaud à nouveau :)
[06:44] <didrocks> hey ricotz
[06:44] <ricotz> didrocks, hi
[06:44] <ricotz> oSoMoN, hey
[07:41] <duflu> Morning oSoMoN, didrocks, ricotz, world
[07:41] <duflu> (western world that is. It's no longer morning here in the east)
[07:42] <duflu> Although that all depends west of /what/
[07:54] <didrocks> hey duflu
[08:13] <Nafallo> o/
[08:13] <Nafallo> I'm not fully awake yet, so morning feels a little abstract still :-P
[08:15] <duflu> Morning Nafallo
[08:16] <didrocks> Mornin Nafallo, complete with the finale g as you see fit ;)
[08:16] <Nafallo> :-D
[08:40] <oSoMoN> chrisccoulson, good morning! chromium-browser 66 fails to build for xenial in https://launchpad.net/~canonical-chromium-builds/+archive/ubuntu/stage/+packages because I bumped the clang build-dep to 5.0 which is in xenial-updates, but the PPA builds against -security only, what would you recommend doing to address that?
[08:40] <oSoMoN> should I copy llvm-toolchain-5.0 from xenial-updates to the PPA?
[08:41] <didrocks> oSoMoN: I think today is a bank holidays in the UK
[08:43] <didrocks> (just for the record, shouldn't the deps be "default", so that you grab both against -security and -updates?)
[08:43] <didrocks> (well, if you don't copy from the ppa to -security afterwards, ofc)
[08:45] <oSoMoN> didrocks, that's the thing, that PPA is a staging area from where chromium updates are copied to -security
[08:45] <oSoMoN> right, I forgot that bank holiday thing, it's gonna be a quiet day today
[08:45] <didrocks> understood, makes sense
[08:45] <didrocks> yeah ;)
[08:45] <oSoMoN> perfect for getting things done :)
[08:46] <didrocks> exactly! Crazy it's not even 11am and what I was able to do since the start of the day
[09:32] <c-lobrano> morning all :)
[09:36] <didrocks> hey, or rather, re c-lobrano! ;)
[09:37] <c-lobrano> Trevinho: hi! It seems that dropbox double menu issue is still reproducible on communitheme session https://bugs.launchpad.net/ubuntu/+source/gnome-shell-extension-appindicator/+bug/1716056 and a dropbox version newer than the one that should have the fix. Do you think there is something missing in our session to make it work?
[09:37] <c-lobrano> Trevinho: here the ticket on communitheme https://github.com/ubuntu/gnome-shell-communitheme/issues/172
[09:37] <ubot5-ng> ubuntu bug 172 in gnome-shell-communitheme "Dropbox double menu" (comments: 6) [Open]
[09:37] <didrocks> I bet dropbox/electron hardcoded the session name
[09:38] <didrocks> or something like "XDG_CURRENT_DESKTOP" starts with ubuntu:
[09:38] <c-lobrano> didrocks: yep I worries this is the thing
[09:38] <c-lobrano> *worry
[09:39] <didrocks> c-lobrano: can you try starting dropbox just changing the env var?
[09:39] <didrocks> (well, kill current process and restart it)
[09:39] <c-lobrano> let me try
[09:39] <didrocks> try with XDG_CURRENT_DESKTOP=ubuntu:GNOME
[09:39] <didrocks> (that's what the main session has)
[09:40] <c-lobrano> didrocks: yep, confirmed, this change works
[09:41] <c-lobrano> also the icon is nicer :)
[09:42] <didrocks> special icon! :p
[09:43] <didrocks> IIRC, the main patch was on electron
[09:44] <didrocks> and here we go: https://github.com/electron/electron/blob/master/lib/browser/init.js#L166
[09:45] <c-lobrano> olè! harcoded things work best :D
[09:45] <c-lobrano> was there a bug already opened on this?
[09:45] <didrocks> yeah… until it breaks :)
[09:45] <c-lobrano> lol
[09:45] <didrocks> I don't thinks so, mind opening it?
[09:45] <didrocks> let me flush out a quick fix PR
[09:45] <didrocks> and reference your bug
[09:46] <didrocks> plotting a riot :p
[09:46] <c-lobrano> sure, on the same github repo above, right?
[09:46] <didrocks> yes!
[09:46] <c-lobrano> alright
[09:52] <didrocks> c-lobrano: fix ready, just paste the bug link here
[09:58] <c-lobrano> https://www.irccloud.com/pastebin/ObCO8IPx/
[09:58] <didrocks> pastebin deleted?
[09:59] <c-lobrano> bad link, this is the right one https://github.com/electron/electron/issues/12843
[09:59] <ubot5-ng> electron bug 12843 in electron "Double trayicon menu when session name does not match hardcoded ones" (comments: 1) [Open]
[09:59] <didrocks> perfect!
[09:59] <c-lobrano> oh my, I should really improve my "open new bug" skills
[09:59] <c-lobrano> :)
[10:00] <didrocks> I prefer you don't improve that skill while keeping a "don't create bug in communitheme"  ;)
[10:00] <didrocks> one*
[10:01] <didrocks> I'm amazed as the themes can be quite fragile that there has been no big breakage yet! Good work
[10:01] <didrocks> c-lobrano: for reference: https://github.com/electron/electron/pull/12844
[10:01] <ubot5-ng> electron bug (Pull request) 12844 in electron "Support multiple inherited ubuntu session for indicators" (comments: 1) [Open]
[10:01] <didrocks> the issue is that we then need to wait for it to be merged, then electron release, then dropbox rebase on newest electron ofc
[10:02] <c-lobrano> ahaha, I'm a developer, I made bugs for a living, but don't tell anyone ;D
[10:02] <didrocks> shhhhhhhh ;)
[10:02] <didrocks> (we all do :p)
[10:03]  * duflu turns to camera - that's not how it works, kids
[10:03] <c-lobrano> of course, my code is bug free, I was only joking (wink)
[10:04] <didrocks> sureeeeeeeeeee, I've never fix either any bugs on any of my projects. It's only in the other ones… :p
[10:04] <duflu> didrocks, you (or someone) can probably land Andy's ubiquity fix
[10:04] <didrocks> duflu: yeah, I can't merge though the ubuquity repo, so I prefer someone have merge rights to do so
[10:04] <didrocks> ubiquity*
[10:05] <didrocks> there is no rush unfortunately, we are not going to have an ISO respin this week…
[10:05] <duflu> didrocks, of course, but being early reduces the risk of being late
[10:06] <duflu> or more late regressions
[10:06] <didrocks> indeed
[10:06] <duflu> AIUI that one happened on release day?
[10:06] <duflu> Not sure. Anyway, I am exhausted. Good night
[10:07] <didrocks> I guess it was a day after release
[10:07] <didrocks> have a good night duflu!
[10:07] <didrocks> tomorrow is off in France, see you on Wednesday
[10:08] <duflu> Heh, I've already worn the writing off my bionic installer USB
[10:08] <duflu> OK, catch you Wednesday didrocks
[10:20] <andyrock> morning all
[10:21] <didrocks> hey andyrock
[10:27] <c-lobrano> morning andyrock :)
[10:49] <Trevinho> morning
[11:18] <ricotz> any chance for https://bugs.launchpad.net/ubuntu/+source/vte2.91/+bug/1765389
[12:20] <jbicha> ricotz: could you ask again once cosmic is actually open? you can tell when it's open because https://launchpad.net/ubuntu/cosmic won't say Pre-release Freeze (and there should be an announcement email)
[12:32] <ricotz> jbicha, hi, ok, still having this in bionic eventually would be great
[12:50] <jbicha> ricotz: could you update the bug description to use the SRU template? https://wiki.ubuntu.com/StableReleaseUpdates#Procedure
[17:29] <oSoMoN> time to wrap up, have a good evening/rest of the day everyone
[21:01] <andyrock> jbicha: can you take a look https://code.launchpad.net/~azzar1/gnome-shell/fix-1768786/+merge/345203 ?