/srv/irclogs.ubuntu.com/2018/03/23/#ubuntu-desktop.txt

Trevinhojbicha: would you mind to backport this commit https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/1758224 ?02:57
ubot5Ubuntu bug 1758224 in gtk+3.0 (Ubuntu) "Scaled gtk theme assets are ignored in HiDPI" [High,Triaged]02:57
Trevinhoerr https://gitlab.gnome.org/GNOME/gtk/commit/e36b629c367fa11e6e544ff8b8203bcf29ec73ee02:57
dufluTrevinho, thanks for the bionic theme update... I'm seeing a 1px gap between bubble menus and their arrow though. Is that new?03:16
Trevinhoduflu: just more visible now03:16
Trevinhoduflu: but it's already fixed03:16
dufluAlso the colour of the arrow is different to the bubble03:16
dufluslightly03:16
Trevinhoduflu: there are two reviews if you want to check/approve them, I can land them straigth away03:16
dufluTrevinho, if there are screenshots then I'll probably review them straight away :D03:29
dufluAlthough I would also be happy if you got some sleep03:31
Trevinhoduflu: yeah, now that i'm done with another fix, i'm going...03:50
Trevinhoduflu: and fix is this one03:51
Trevinhohttps://usercontent.irccloud-cdn.com/file/QZ2WV3hG/image.png03:51
didrocksgood morning06:48
tsimonq2o/ didrocks06:49
dufluHi didrocks, tsimonq206:53
didrockshey tsimonq2, duflu06:59
tsimonq2o/07:00
=== ecloud_wfh is now known as ecloud
=== pstolowski|afk is now known as pstolowski
=== ochosi_ is now known as ochosi
willcookehoy08:39
* duflu doesn't understand but waves to willcooke08:43
willcookeo/08:43
willcookeright, now I'm going again.  Dr's bbiab08:43
didrockshey willcooke08:43
jibelhola everyone08:43
dufluHi jibel08:45
didrockshey jibel08:52
jibelLaney, jbicha re design change in ubiquity, I agree with acheronuk and tsimonq2 and won't merge until the KDE frontend is updated too.08:58
Laneyhi09:01
ShriHarihi laney09:02
didrocksoh, only the GTK frontend was updated?09:03
didrocksnot what I mentioned during the team meeting with jbicha09:03
dufluHi hi Laney09:04
Laneyjibel: ok, but I agree with jbicha that it's not introducing a regression09:07
Laneyand if he's not willing to do the work then I guess we don't get the change09:07
tsimonq2Laney: It's a UX regression of anything.09:08
tsimonq2s/of/if/09:08
Laneywhat?09:08
Laneythat change is not09:08
tsimonq2One frontend gets new sections and radio boxes and the other gets a terser section of text.09:09
LaneyIt already has the previous UI09:09
LaneyThe UI is not changing09:09
LaneyTherefore the UI is not regressing09:09
seb128Laney, the description is09:09
seb128there is a string change09:09
LaneyYou might think it's worse than the GTK one, but it is not regressing versus what it currently has09:09
Laneyno?09:09
Laneyhey seb12809:09
seb128hey Laney09:09
seb128sorry, I'm half back, need to catch a train in a bit09:09
tsimonq2It is a regression09:09
seb128Laney, see https://launchpadlibrarian.net/361682891/kde-front-end-before.png vs https://launchpadlibrarian.net/361682873/kde-front-end-after.png09:10
seb128the substring/description09:10
seb128I didn't notice it at first either09:10
tsimonq2Also, why only put one frontend closer to spec?09:11
seb128otherwise I agree there is no reason we should force people to update the kde frontend when they change the gtk one, but in this case it makes a string used less nice09:11
seb128tsimonq2, because that spec is our GTK desktop  one09:11
seb128kubuntu team could have different intend09:11
tsimonq2Hm. OK.09:11
seb128like not being interested by minimal desktop09:11
tsimonq2No, we have the option.09:11
seb128also you can't force our team to take the cost of maintaining flavors09:11
didrocksgood morning seb12809:12
tsimonq2That's fine, but why weren't we involved before release team acked?09:12
dufluHi seb12809:12
tsimonq2Kubuntu was an afterthought at best.09:12
LaneyI think the release team said to check with Kubuntu, which is what happened, no?09:13
jibelIndeed but they should both sync so a change in a flavor doesn't have a negative impact on another flavour09:13
tsimonq2Laney: I interpreted your comment as an ACK, was that incorrect?09:13
LaneyIt was ACK but check with Kubuntu, which is what happened.09:13
acheronuknot volunteering here, but a brief hack at it this morning: https://i.imgur.com/8ku1e9E.png09:13
LaneySo you weren't afterthoughted.09:14
LaneyYou were perithoughted09:14
willcookeback09:14
tsimonq2I guess I'm just personally wondering why we weren't involved before the MP was made so we could have had the opportunity to introduce the code.09:15
tsimonq2I'll accept perithought. :P09:15
tsimonq2s/personally //09:15
tsimonq2acheronuk might work on something, but until then, I NACK on the grounds of not having a Qt equivalent there.09:17
LaneyQt equivalent isn't a fair thing to ask for09:19
LaneyNot making the UX regress is - sorry for not noticing that straight away09:19
tsimonq2Not from the Desktop Team; I understand that.09:20
tsimonq2But we should still have the opportunity.09:20
tsimonq2And I'd argue that it should be done in one upload.09:20
LaneyIt can be, but there's no reason it needs to be09:20
tsimonq2Kubuntu's installer regresses09:21
LaneyFeeling a bit circular now, sorry09:21
tsimonq2I'd be surprised if the Release Team ACKs despite regressions.09:21
tsimonq2No problem. I think I've made my point. :)09:22
acheronuktsimonq2: I may carry on with it later, but not 100% certain on correctness of what I have done or doing09:22
* didrocks feels that fixing it will take less time than this conversation TBH09:23
tsimonq2Probably.09:23
tsimonq2acheronuk: ACK, feel free to share your WIP and I'll be happy to peer review.09:24
acheronukok. jibel jbicha etc, is there a particular reason this needs to be landed before the weekend? I would assume not, and Kubuntu or whoever else can have the weekend to get something sorted?09:26
acheronukthat is of course unless someone else with more skill feels like kindly knocking it out quickly today09:27
Laneyj_bicha was trying to make UIF09:31
Laneyacheronuk: thx for working on the fix09:32
acheronukand UIF came into effect yesterday without this getting in I assume?09:33
LaneyI think notification would have been nice, even if I don't agree we should in principle block on changes on both side09:33
Laneyyes09:33
Laneyit's missed it now09:33
acheronukLaney: it would have been nice and collaborative to get a "we are working on these changes, would you like to contribute equivalents for the KDE front end?"09:34
acheronukbut things do get done in a hurry at times like this09:34
acheronukI'll busy most of rest of workday, but can look more tonight09:35
acheronuk*be busy09:36
jibelacheronuk, nothing other that giving enough time to translators to update the strings09:36
acheronukjibel: ok. so a few extra days will no cause significant issues there? as long as it is just days09:37
jibelit depends on your definition of "few" :) the translation deadline for Ubiquity is in 3 weeks. It'd be nice to leave as much time as possible to translate and verify the UI09:40
acheronukjibel: well as I said, hopefully by Monday. if it takes longer than that I would be (a) very surprised and (b) give it up as a bad job09:42
jibelacheronuk, Monday is okay, thanks for working on this.09:43
acheronukok :)09:44
seb128hey didrocks duflu09:46
Laneya happy ending ... ?09:46
seb128tsimonq2, not implementing the same new options Ubuntu decide to give (or remove) isn't a regression but a choice09:48
didrocksif I knew that would have triggered that amount of polemic, I wouldn't have implemented it in the KDE variant at first :(09:48
seb128acheronuk, tsimonq2, what's the point to delay Ubuntu improvements landing just because other flavors need to catch up? Things should land when ready, not be blocked on "if we can't have our cake you are not geting yours either" reasons09:49
seb128didrocks, yeah, teaching you for trying to help :)09:50
Laneythink we can probably draw a line now ...09:51
Laneyhugs to one and all09:51
didrocksyeah! ;)09:52
acheronukdidrocks: it's very much appreciated that you did09:53
Laneygoing to london for the weekend09:54
Laneyjust found out it's the oxford / cambridge boat race09:54
Laneymaybe i should go spectate09:54
didrocksLaney: is it a thing where you sit for 3 hours to wait for all of them passing by you in like… 10s? ;)09:55
Laney#sport09:55
Laneyprobably a huge crowd too09:56
didrocksyeah09:56
LaneyI saw there is the oxford / cambridge goat race on the same day09:56
Laneythat one might be better09:56
seb128I'm in Paris atm, next to gare de l'est09:56
seb128they locked down one side of the station because there is a leftover box outside09:56
Laneyseb128: get to nord, take the eurostar, we'll hang out09:56
seb128they are having the anti-bomb robot getting ready :)09:56
didrocksseb128: close your hears when they will make it explode09:57
seb128Laney, haha, in fact I'm going to Nord but taking the Thalys :p09:57
didrockslast time at gare de Lyon (in Paris thus :p), it made a lot of noise09:57
seb128didrocks, thanks, I'm in my hotel some streets away now with some luck they clear it out before I go09:57
didrocksyeah, you should be fine :)09:58
ackktkamppeter, hi, is there anything more I could debug about https://bugs.launchpad.net/ubuntu/+source/hplip/+bug/1731417 ?10:23
ubot5Ubuntu bug 1731417 in hplip (Ubuntu) "Installed network printer removed automatically when turned off" [Undecided,Confirmed]10:23
jbichaacheronuk: I suggest changing the sidebar title from "Prepare" to "Software" to better match the new page title at10:53
jbichahttps://launchpadlibrarian.net/361682873/kde-front-end-after.png10:53
acheronukjbicha: I was using the old vesrion in a live session to quickly test modify. if you changed that title string in your revision/template, I think it will be carried over when I apply the changes on top of yours10:55
acheronukif it doesn't, I will change it10:56
jbichayeah, GTK+ doesn't use a one-word page title like that so that's a KDE-specific change10:57
willcookeLaney, jibel - just attached a "no grain" version of the wallpaper.  Now 1.6MB!!! And looks sharper IMO.11:04
willcookehttps://bugs.launchpad.net/ubuntu/+source/ubuntu-wallpapers/+bug/175808911:04
ubot5Ubuntu bug 1758089 in ubuntu-wallpapers (Ubuntu) "18.04 default wallpapers" [Undecided,New]11:04
Laneywillcooke: ta duck11:05
amanodidrocks, c-lobrano told me on the Ubuntu community HUB that there was a Mutter toggle to open new windows in the middle of the screen instead of the top left corner. I think that opening in the middle would be much preferable visually. Having the window in the top left by default creates a very dark area there for the Community theme with the dark headerbar and the dark topbar crammed together. And in the very top left corner11:08
didrocksamano: hum, interesting, mind opening a bug against mutter as a whishlist so that I keep that in mind (and linking here)?11:11
didrocksthey don't open on the top left corner for me, but currently in a multiscreen env11:11
amanoI will open a bug for you :)11:13
didrocksthx! :)11:14
c-lobranoamano, didrocks it's center-new-windows under /org/gnome/mutter11:14
c-lobranoit works with new windows only, I noticed that some apps stay opened in background so position does not change (like nautilus, gnome software, ...)11:15
didrocksgood hint :)11:20
jbichainteresting, does that work ok for dual-monitors?11:20
jbicharicotz: it looks like you added the center-new-windows setting. Do you know why it's not enabled by default?11:21
jbichadidrocks: I assume you saw https://code.launchpad.net/~jbicha/ubuntu-seeds/minimal-remove-totem/+merge/341921 and have just been busy?11:30
seb128Thalys first class space at the train station is crap11:32
seb128like internet, but port filtered, no imap/irc/vpn, welcome to soviet union11:32
didrocksjbicha: no, I didn't see it, just approved it, are you handling the merge + upload?11:41
jbichaI can push, yes, it doesn't need an actual ubuntu-meta deb upload, right?11:42
didrocksjbicha: shouldn't, yeah11:44
jbichadone11:44
amano_didrocks, https://bugs.launchpad.net/ubuntu/+source/mutter/+bug/175831411:54
ubot5Ubuntu bug 1758314 in mutter (Ubuntu) "[Whishlist] Open new windows in the middle of the screen" [Undecided,New]11:54
jibelwillcooke, just 10x smaller ;)11:57
willcooke:)11:58
jibelnow if you convert to 4k and optimize a bit that should be reasonable11:58
willcookeYou think 4k is enough?  Should be, right?11:58
tkamppeterackk, sorry, I have overlooked that you have posted your files in the bug report. I am looking into it now.12:00
ackktkamppeter, thanks12:00
willcookejibel, uploaded a 4k version to the bug.  Only 652k now \o/12:01
jibelYou think 8k is common?12:01
jibelit's good, artful was 607K12:02
Nafallosupported for five years. the question is surely if 8k is enough by then end of 2023? ;-)12:03
didrocksackk: thx!12:03
jibelsupported doesn't mean it's common12:03
Nafalloin 2023... :-)12:03
jibelwe'll SRU a new wallpaper then12:03
Nafallonow, that makes sense to me indeed :-)12:04
jibeland since telemetry will give us that info ....12:04
Nafallo4k is starting to become common fwiw.12:04
jibelwillcooke, 552K12:04
didrocksamano_: thx! (still thanks, ackk though ;))12:05
willcookejibel, did you crush it some more?12:05
jibelwillcooke, yes with optipng12:07
LaneyI'd usually do all this you know12:08
Laneybut thanks for helping12:08
amano_jbicha, I saw that the gnome-terminal package carries a debian distro patch to use the scrollbar background colour from the theme. in practice the terminal scrollbar background is always dark for me. That looks ok with the default terminal theme but it looks slightly wrong with bright themes. Can you consider dropping the patch to have a standard scrollbar with the next terminal upload?12:09
jbichajibel: have you ever tried zopfli? we use it to compress the pngs for fonts-noto-color-emoji (but it makes the build take a long time)12:09
jbichaamano_: have you tried building gnome-terminal without the patch?12:09
jibeljbicha, no I didn't; I'm used to this tool but I'll try12:10
amano_nope, not yet12:10
ackktkamppeter, fwiw it seems that disabling bonjour on the printer might be a workaround for the issue12:10
ackkalthough it'd be nice to be able to have it on12:11
mgedminoh is that why gnome-terminal's scroollbar has this weird 1px white border that sticks out on Adwaita?12:11
jbichaamano_: ok, I believe the patch is needed for Ubuntu's default themes12:11
willcookejibel, if Laney's happy, want to add it to the bug?12:19
jibelwillcooke, I did12:19
willcookeoh yeah12:21
willcookethanks12:21
jibelWe'd need a marmot for the slideshow of the installer too. Should we reuse this one or design will provide another one?12:30
jibelwillcooke, ^12:30
jibelusually it's done on release week ...12:31
jibelthe earlier, the less report we'll receive telling it is not the right animal12:32
willcookejibel,  I have one12:36
willcookeI'm waiting on a couple of changes and will upload12:36
jibelwillcooke, okay, thanks12:40
Trevinhoseb128: hey, we need to get this https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/1758224 (see commit linked there), want me to do a debdiff or anyone else can handle it?12:53
ubot5Ubuntu bug 1758224 in gtk+3.0 (Ubuntu) "Scaled gtk theme assets are ignored in HiDPI" [High,Triaged]12:53
seb128Trevinho, hey, I saw that, I can handle it12:55
Trevinhoseb128: thanks, it's quite bothering... bad thing was that I noticed it, I found the issue, wrote the fix... and then... It was already fixed in the past two weeks, but I didn't git pull in the mean time -_-13:02
seb128Trevinho, that happens indeed :/13:08
ricotzjbicha, it was meant to not change the current behaviour and providing this option which is used by e.g. elementaryOS13:11
jbicharicotz: have you asked GNOME Design about changing the behavior?13:12
ricotzjbicha, no13:12
jbichaok13:13
ricotzthe goal of that commit was to provide this feature for 3rd party users of libmutter ;)13:13
ackktkamppeter, attached avahi outut as requested13:15
jbichaI'm asking in #gnome-design now…13:17
=== amano_ is now known as amano
=== marco is now known as Guest47662
flocculantwillcooke: any more thoughts on the long time to desktop thing ?13:53
willcookeflocculant, jibel has been looking at it.  There is a new release of snapd which we think might help.  jibel - did you test it yet?13:58
flocculantsnapd? I guess you're looking at something else then lol13:59
flocculanttimeout's I see on ubuntu and xubuntu are bluetooth issues14:00
flocculantanyway - what do I know lol14:00
jibelwillcooke, the new version just allows to delay the refresh of snap packages. There is still the issue with decompressing the snaps during boot and several timeouts like bug 1750846 for gsd or bluez like flocculant reported14:03
ubot5bug 1750846 in gnome-settings-daemon (Ubuntu Bionic) "several g-s-d services are timing out when booting a live session" [High,New] https://launchpad.net/bugs/175084614:03
flocculantaah cool - different thing then - I was confused for a bit there :)14:07
flocculantcheers jibel14:07
kenvandinetranslated xdg-user-dirs https://imgur.com/a/8EeUK14:37
kenvandinewoot14:37
kenvandineusing the gedit snap14:38
Laneynice14:42
Laneywhat's the fix?14:42
seb128back online14:46
kenvandinehttps://github.com/ubuntu/snapcraft-desktop-helpers/pull/10314:53
kenvandineLaney, that's the fix14:53
kenvandinehey seb128!14:53
kenvandinetranslated xdg-user-dirs https://imgur.com/a/8EeUK14:53
jbichaseb128: just wondering, what IRC client do you use?14:54
Laneykenvandine: you have permission to read those?14:54
kenvandineLaney, yes14:55
kenvandinewe didn't in the past14:55
kenvandinebut we do now14:55
seb128hey kenvandine14:56
seb128jbicha, xchat-gnome14:56
seb128jbicha, why?14:56
seb128kenvandine, nice14:56
jbichaseb128: LP: #175816314:57
ubot5Launchpad bug 1758163 in xchat-gnome (Ubuntu) "Please remove xchat-gnome from Ubuntu (again)" [Undecided,Fix released] https://launchpad.net/bugs/175816314:57
jbichaseb128: this is what happens when you're not around ;)14:58
jbichahave you tried polari?14:58
seb128jbicha, not this cycle14:58
seb128I should give it another try, maybe it improved14:59
seb128jbicha, that bug is a joke, "it is too much of a maintenance overhead"15:00
jbichait probably didn't change much recently15:00
seb128it's not like we had lot of fixing to do on it in recent cycles15:00
jbichaI thought the hexchat bug was a joke too! :|15:00
seb128it feels like that has been a reaction to the xchat-gnome removal bug15:01
seb128but it's a bit ridiculous to kick out a gtk3 client as "unmaintained" and putting forward a gtk2 one as an alternative solution15:02
jbichaSteve said he'd reaccept xchat-gnome if someone wants to upload it and maintain it https://irclogs.ubuntu.com/2018/03/23/%23ubuntu-devel.html#t03:5015:02
seb128jbicha, was it a trolling bug or do you have a real motivation to try to clear than one out, out of the stack of old unmaintained craps in universe15:03
mdeslaurI filed the hexchat bug in an attempt to get someone to finally take on moving it to gtk315:03
jbichawell you heard about the xchat removal bug controversy, right? xchat-gnome is pretty similar (presumably in worse shape actually)15:04
seb128I did read about this one15:04
seb128anyway, I don't really care either way15:04
seb128if somebody reupload xchat-gnome good15:04
seb128otherwise I'm probably going to snap it and use that15:04
jbichaI like the idea of Snaps for old stuff we don't want in the distro any more15:05
seb128those removals are ridiculous, there are tons of packages more outdated and less useful than xchat-gnome in  universe15:05
seb128why not going to try to get those out first...15:05
mdeslaurwell, I want xchat-gnome in the distro, which is why I've been maintaining it in the first place15:05
seb128wfm15:05
seb128and good that some people find hexchat good for them15:05
seb128it doesn't work for me, I find the UI not easy to "read" and none of the available colors/settings make it look clean/easy to me15:06
seb128while xchat-gnome is15:06
jbichaI am working on removing packages from Debian (libgnome depends these days). It just takes a long time with Debian processes and "maintainers"15:06
mdeslaurremoving packages that still build and work is a disservice to users15:07
seb128especially when they work, have users and are not depending on outdated techs we want to get out15:08
seb128anyway15:08
seb128don't hate on softwares just because you don't use them :)15:08
seb128on that note, enough said on the topic for me15:08
jbichamdeslaur: packages that have been removed from Debian should have an Ubuntu maintainer to stay in Ubuntu15:13
* mgedmin tried very hard to migrate from the nice xchat-gnome to the promising polari, but in the end switched to irssi because of all the polari problems15:15
juliankpolari looks nice15:16
juliankmgedmin: I could only setup connection to bouncer with mc-tool because both polari and empathy lacked options...15:17
juliankI'm thinking about writing a GTK+3 HIG WeeChat relay client15:17
Laneyjbicha: any reason not to merge xdg-user-dirs?15:23
jbichaLaney: I don't understand what we're doing with translations and maybe that should be pushed upstream15:24
jbichabut if you want to merge, go for it15:24
LaneyTrevinh_o's got his fix up for sponsoring and that would be one way to get it15:26
Laneythx15:26
Laneyseb128: do you just download https://translations.launchpad.net/ubuntu/bionic/+source/xdg-user-dirs/+pots/xdg-user-dirs/+export and use `quilt shell' or something to copy it over?15:27
seb128Laney, yeah, I download, rename the files from somethiong_LO.po to LO.po, copy over and diff -Nru old new basically15:29
seb128and update LINGUAS to list the new/added locales15:29
seb128jbicha, we just import the translations from launchpad15:30
=== jbicha_ is now known as jbicha
* Laney makes tea while waiting for the tarball to arrive15:32
jbichaif we upstream the translations, maybe we could just turn off translations in LP for the package and point translators to https://translationproject.org/domain/xdg-user-dirs.html15:33
seb128you can argue that for any translations made on launchpad for any source15:36
seb128we could yes15:36
seb128but then we loose an ease-to-contribute/easier-integration point from Ubuntu15:37
jbichabut most packages don't require this manual handling which slows down updates15:37
jbichamaybe we could do it for Chaotic Camel15:37
seb128I've no strong opinion15:39
seb128upstreaming the changes is not easy since we don't speak all the languages to argue of the value/difference15:39
seb128not easy to decide who between upstream and launchpad translators has the best variant and why15:40
seb128but we could just decide next cycle to close those and tell translators to report bugs upstream for their improvement15:40
seb128maybe send upstream new locales translations15:40
seb128since we have a few of those15:40
Laneyseb128: does https://paste.ubuntu.com/p/jdrrkV8ysk/ look like I did it right?16:10
jibelTrevinho, this fix for grey item on dark grey background introduced a regression, now I've comboboxes with light grey on light grey16:13
jibelTrevinho, https://imgur.com/a/uwlL116:18
jibelbiab16:18
Trevinhojibel: weird but thanks... I'll add to the list16:19
kenvandineTrevinho, would you mind taking a look at https://github.com/ubuntu/snapcraft-desktop-helpers/pull/10316:24
Trevinhook16:24
kenvandineTrevinho, thx16:24
Trevinhooh, that is now available? :-)16:24
kenvandineTrevinho, yes!16:24
Trevinhokenvandine: so couple of things...16:26
TrevinhoWe should probably add support for moving the data we added in $SNAP_HOME to these locations now, no? Otherwise users might lose track of stuff from within the snap16:27
Trevinhoalso.... [ -f ]  is it working now? Since in the past it wasn't enough to check if we had access16:27
Trevinhoas it was gettting false postives16:27
Trevinhothere's a bug somewhere about test not working properly16:27
TrevinhoI had to use a different check (which was... I need to look :D)16:28
kenvandinei'd be interested in better ways to check :)16:28
Trevinhokenvandine: in snap shell if you do `test -f /home/$USER/.config/monitors.xml && cat /home/$USER/.config/monitors.xml` it will fail for example16:30
Trevinhoon cat...16:30
Trevinhoso let me look I think I had a branch16:30
kenvandineok16:30
Trevinhobug is https://bugs.launchpad.net/snappy/+bug/174851016:31
ubot5Ubuntu bug 1748510 in Snappy "shell's test gives false positive on readability of files" [Undecided,New]16:31
Trevinhobut I didn't post the workaround there mhmhmh16:31
Trevinhokenvandine: I used this16:32
Trevinhoif (! head -c0 "$XCOMPOSEFILE" &> /dev/null); then ......16:32
Trevinhothat wont' actually read anything but will do an `open()`16:33
Trevinhokenvandine: also... with such change what will happen is that the XDG_DATA_DIRS will be set relative to $SNAP_HOME (referring to that to $HOME) when you're inside a snap, so it's not enough... Also an user might have set those vars to somewhere that is not readable by snap16:34
TrevinhoSo...16:34
TrevinhoYou should go through each of them and check if we've acecess16:35
Trevinhoif we have you've to create the dirs that are referred inside the $SNAP_HOME.... Ie. for each XDG_DIR*  that in that file we refer to as $HOME/foobar you've to create the relative $SNAP_HOME/foobar as symlink to the actual location.16:36
Trevinhobecause for example qt inside a snap what will try to do  is reading that file, replace $HOME with what is defined as $HOME in that context...16:37
Trevinhothen for dirs that instead are defined as full paths, you need to check access16:38
Trevinhoif those tests fail, you just create the $SNAP_HOME/$XDG_DIR_* and use that16:38
Trevinhoanother option16:38
Trevinhois to read that file...16:39
Trevinhoand then regenerate a new one that points to the same dirs (if possible) but considering that the $HOME is not aymore the user $HOME16:39
Trevinhothis has to be redone everytime we start a snap, though... or check if the file has changed since the last time we regenerated it (better).16:40
Trevinhoas if user changes something this won't be reflected...16:40
TrevinhoSo... yeah, finally I think this is the best option :)16:40
Trevinhosorry thinking at loud voice, but at leasts you can follow (I hope) my mental process16:41
kenvandinei think so16:41
Trevinhokenvandine: I think I've added better comments in the review :)=16:51
kenvandineTrevinho, thanks!16:51
TrevinhoI hope I didn't miss something16:52
=== pstolowski is now known as pstolowski|afk
Laneyffs17:22
Laneygnome-keyring is going crazy on my laptop17:22
Laneyok I don't have the new patches17:23
jbichaLaney: I had to reboot when gnome-keyring started hogging CPU, I haven't had it again in a few days :(17:24
Laneyjbicha: that was with the current version?17:28
jbichayes, did you apt upgrade gnome-keyring today?17:29
Laneydoing it now17:29
Laneybut if you had it, I'm guessing this particular bug is not fixed17:29
Laneybut maybe rebooting will be good to me ...17:29
LaneyI don't have the dbgsyms to get a backtrace and now it's upgraded17:29
Laneystill bad17:33
Trevinhodamn... I've a talk to do tomorrow and still slides to write xD17:37
Laneyworks after deleting from seahorse and re-adding17:56
Laneyseb128: I'm just going to upload that xdg-user-dirs thing, sorry if it's buggy - we can fix next week17:56
* Laney makes Trevinho happy17:57
LaneyTrevinho: who's the talk to?17:57
willcookehappy weekend all.  Good luck Trevinho .17:57
willcookeLondon on Monday, won't be around in the morning17:57
Laneystupid yak shaving meant I didn't get bluetooth debugging info17:58
LaneyI originally just restarted to get pulseaudio being more verbose :(17:59
Laneyanyway I'm off too to watch/not watch the boat race18:00
Laneylaters18:00
=== alan_g is now known as alan_g|EOW
didrocksenjoy/don't enjoy Laney :)18:11
TrevinhoLaney: https://merge-it.net/18:45
seb128Laney, sorry you missed me by a few minutes, looks fine to me18:51
seb128have a nice w.e desktopers18:52

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!