/srv/irclogs.ubuntu.com/2018/09/14/#ubuntu-desktop.txt

=== simonquigley is now known as tsimonq2
=== 17SAAK0SL is now known as infinity
didrocksgood morning05:59
dufluMorning didrocks06:01
didrockshey duflu06:02
jibelHi didrocks and duflu07:05
=== pstolowski|afk is now known as pstolowski
dufluHi jibel07:10
didrockssalut jibel07:16
oSoMoNgood morning desktoppers07:24
didrockssalut oSoMoN07:29
oSoMoNsalut didrocks07:29
dufluHi oSoMoN07:30
oSoMoNhey duflu07:36
willcookemorning08:02
didrockshey willcooke08:02
Laneyyo08:02
didrocksmorning Laney08:03
Laneyhey didrocks08:11
Laneyyou good?08:11
didrocksI'm fine, thanks. Yourself?08:12
Laneytired today :<08:12
didrocksit's Friday though!08:13
LaneyYEAH!08:13
oSoMoNlast night I attempted to upgrade to cosmic, and got hit by bug #179253708:28
ubot5bug 1792537 in appstream (Ubuntu) "/usr/bin/appstreamcli:5:g_variant_new_parsed_va:g_variant_builder_add_parsed:as_content_rating_to_variant:as_component_to_variant:as_cache_file_save" [Undecided,Confirmed] https://launchpad.net/bugs/179253708:28
oSoMoNjibel, is that an upgrade path that we test at all? maybe later in the cycle?08:29
jibeloSoMoN, we test that08:30
jibelbut I haven"t looik at the results recently08:30
oSoMoNjibel, ack, would you mind confirming what I'm seeing?08:30
jibeloSoMoN, bionic to cosmic upgrade with do-release-upgrade?08:32
jibeloSoMoN, you didn't have this problem in bionic already when you updated the package cache?08:33
oSoMoNjibel, I ran "update-manager -d"08:33
oSoMoNjibel, I wasn't seeing the problem in bionic, but if I run appstreamcli I can reproduce the crash08:34
oSoMoN(wasn't seeing it when applying regular updates)08:34
oSoMoNit might be https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=90653808:34
ubot5Debian bug 906538 in appstream "appstream: apt-get update fails due to appstream crash" [Important,Fixed]08:34
oSoMoNin which case it's fixed in the version in cosmic08:34
oSoMoNI'll try installing the cosmic package and then run the upgrade again08:35
oSoMoNwill try in a VM first08:35
jibelfrom errors.u.c it is not fixed in cosmic and is not really frequent in bionic08:38
jibelbut I'll have a look08:38
oSoMoNif you confirm and it turns out not to be fixed in cosmic (apparently not), then I can look into it08:39
oSoMoNwell the upgrade seems to be proceeding, so it looks like the version in cosmic fixed that crash08:40
jibelmaybe you have a package installed that triggers this issue08:41
oSoMoNjibel, I was able to reproduce in a clean bionic VM, with supposedly no exotic package08:42
jibeloSoMoN, appstreamcli refresh-cache works fine on a default installation of bionic08:48
LaneyIt's probably when apt updating after switching the sources.list08:49
oSoMoNindeed it's not crashing when run outside of the context of the upgrade, so Laney must be right08:50
oSoMoNsee my comment on bug #1792537, the crash was fixed upstream, we should probably SRU that patch08:51
ubot5bug 1792537 in appstream (Ubuntu) "/usr/bin/appstreamcli:5:g_variant_new_parsed_va:g_variant_builder_add_parsed:as_content_rating_to_variant:as_component_to_variant:as_cache_file_save" [Undecided,Confirmed] https://launchpad.net/bugs/179253708:51
jibellet me try that08:51
jibelLaney is right08:52
jibelchange bionic -> cosmic in sources.list and run apt update08:52
Laneythat makes sense given the bug08:53
Laneyhappy to sponsor it if you want to prepare the debdiff / bug template oSoMoN08:53
Laneyassuming you can't upload that, maybe it is in the set tho08:54
Laney== All uploaders for package 'appstream' ==08:54
LaneyArchive Upload Rights for ubuntu-core-dev: archive 'primary', component 'main' in bionic08:54
oSoMoNLaney, on it08:56
Laney🏋08:56
oSoMoNtest packages building in https://launchpad.net/~osomon/+archive/ubuntu/appstream-sru-1792537/+packages09:03
didrocksjibel: you may want to have a look/answer on https://forum.snapcraft.io/t/add-sweethome-3d-fixtures/736009:14
oSoMoNLaney, I attached the debdiff to the bug report: https://bugs.launchpad.net/ubuntu/+source/appstream/+bug/1792537/+attachment/5188736/+files/debdiff-1792537.diff09:22
ubot5Ubuntu bug 1792537 in appstream (Ubuntu) "/usr/bin/appstreamcli:5:g_variant_new_parsed_va:g_variant_builder_add_parsed:as_content_rating_to_variant:as_component_to_variant:as_cache_file_save" [High,Triaged]09:22
oSoMoNnow going to update the bug description for SRUability09:22
oSoMoNjibel, according to errors.ubuntu.com the crash still happens in cosmic with appstream 0.12.2-2, so it's not fully fixed, but the patch does fix that specific occurence09:26
Laneyit's not always accurate09:28
Laneysometimes crashes which happen around an upgrade are reported against the new version mistakenly09:29
oSoMoNaha, so the patch might be complete indeed09:32
oSoMoNgood09:32
* Laney eyes ximion09:34
Laneycan you cast a string to a GVariantType*?09:34
oSoMoNhttps://developer.gnome.org/glib/stable/glib-GVariantType.html#G-VARIANT-TYPE-ARRAY:CAPS09:36
oSoMoNlooks fine09:36
Laneyusually you would use G_VARIANT_TYPE ("...")09:39
oSoMoNthat would be safer indeed09:55
oSoMoNLaney, I updated the bug report, thanks for the upload10:04
Laneygrazie10:05
zygahello10:11
zygaI need help SRUing this patch: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/179255210:11
ubot5Ubuntu bug 1792552 in qtbase-opensource-src (Ubuntu) "Qt doesn't cope with EPERM returned from statx(2)" [Undecided,New]10:11
zygahmm, I just realized I should have used more parentheses10:12
* zyga goes to refresh the patch10:12
zygadone now10:35
zygapopey: ^ who can we get to SRU this into ubuntu?10:35
popeyGood question. willcooke ^ :)10:36
popeyprobably something you'd want to upstream to debian?10:36
zygapopey: yes,10:37
ahayzenzyga, oh does that help with the docker stuff i was saying the other day ? before i spoke to m_itya57 and t_simonq2 in #ubuntu-qt10:37
zygawe're working on a snapd-side fix but this will not happen immediately10:37
zygaahayzen: hmmm please remind me about the issue (sorry about that)10:38
ahayzenzyga, https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/177473910:38
ubot5Ubuntu bug 1774739 in qtbase-opensource-src (Ubuntu) "Running Qt apps inside a 18.04 container crashes" [Undecided,New]10:38
zygaahayzen: thank you,10:39
zygalooking at the attached backtrace I don't think this is the sme issue10:39
ahayzenyeah maybe not :-/ m_itya57 at the time suspected it was a missing syscall of some kind, so wondered if it was related10:40
willcookeI'm just looking to see who can look at that patch10:42
willcookewe dont maintain qt in the desktop team10:42
zygawillcooke: the patch is litterally a one liner10:43
zygawe're just extending the support for the ESYS error so that EPERM is handled exactly as if the system call was not available in the first place10:44
willcooketsimonq2, is that something you could take a look at? ^10:45
zygas/bfp/bpf/ there but meh :)10:46
willcookezyga, we'll need to wait for t_simonq2 to come online and see if he can take a look.  He's on the qt packagers team, and I think Debian upstream maintainer too - or at least knows the right people to talk to.11:00
tsimonq2willcooke: Sure, I can look into it.11:20
willcookeace!  Thanks a lot tsimonq2  (cc zyga(11:20
willcooke))11:20
willcookecan't leave those brackets hanging11:21
* tsimonq2 sips coffee11:21
tsimonq2willcooke: And if possible, could that be submitted upstream as well?11:28
willcookezyga, can you submit it upstream too? ^11:33
zygawillcooke: sure11:33
zygaI'll get to it11:33
willcookethanks a lot chaps11:33
=== pavlushka is now known as Guest18370
=== pavlushka_ is now known as pavlushka
=== pstolowski is now known as pstolowski|lunch
Mirvwell it sounds related https://bugreports.qt.io/browse/QTBUG-7044712:39
=== pstolowski|lunch is now known as pstolowski
zygaMirv: indeed12:51
zygaman, sending a one liner to qt is pretty complex12:52
zygaMirv: if you have a qt account could you please add a comment that there's a distro patch for that: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/179255212:52
ubot5Ubuntu bug 1792552 in qtbase-opensource-src (Ubuntu) "Qt doesn't cope with EPERM returned from statx(2)" [Undecided,New]12:52
Mirvadded, but please just register too :)13:11
zygaMirv: yeah, in pgoress13:16
kenvandineoSoMoN, didrocks: i've published gtk-common-themes rev 701 to stable13:37
kenvandineit was a big improvement over what we had before13:38
willcookekenvandine, how is the size now?13:39
kenvandine42M13:39
kenvandineup a couple meg13:39
willcookenot to bad13:39
kenvandinewe need to add the mint themes13:39
kenvandineand i think there's a couple we should remove13:39
kenvandinebut let's discuss that next week13:40
willcookeack13:40
willcookegood plan13:40
kenvandinei'd like to get on a release cadence, maybe every couple weeks or so13:40
kenvandineat least until yaru slows down13:40
kenvandinebut i really don't want to push updates too often since it's seeded13:41
oSoMoNkenvandine, good stuff, +1 on discussing that next week13:48
oSoMoNdidrocks, speaking of yaru, I upgraded to cosmic and noticed that the communitheme remained installed, despite being useless on !bionic, how about removing it when installing the yaru deb?13:49
oSoMoNs/communitheme/communitheme snap/13:49
=== gffa is now known as Guest72349
=== jdstrand_ is now known as jdstrand
kenvandineoSoMoN, i'm thinking about naming for the sdk snap repository14:01
kenvandinehow about a repo named gnome-platform sub directories for the runtime and sdk snaps with master and  gnome-3-30-1804 branches?14:03
kenvandineand gnome-3-30-1604 for now14:03
kenvandineoSoMoN, or do you think it would be better to use separate repositories for the runtime and sdk?14:04
oSoMoNkenvandine, I think having runtime and sdk in the same repo is fine14:05
kenvandineand we can build the snaps out of CI14:06
kenvandinealthough, i guess that would trigger builds for the sdk when we only change the runtime14:06
oSoMoNwe can have a script check wether only one of sdk/runtime changed, and trigger the corresponding build only14:07
kenvandineindeed14:07
kenvandineand ultimately i want the runtime to not stage packages but repack files from the sdk snap14:07
tsimonq2zyga: Please add a DEP-3 header and send an MP to Salda.14:20
tsimonq2*Salsa14:20
zygaSalsa?14:20
zygasure, I'll add the header14:20
tsimonq2salsa.debian.org14:20
tsimonq2Debian's Alioth replacement.14:20
tsimonq2You would make us happy if you prepared a Debian upload in Salsa as well. :)14:21
zygaI never did that14:21
zygabut it's Friday14:21
zygaso I can try14:21
tsimonq2¯\_(ツ)_/¯14:21
tsimonq2OK.14:21
tsimonq2Let me know if you need a hand with anything.14:22
zygatsimonq2: I'm still cloning Qt14:29
zygatsimonq2: I reported https://bugreports.qt.io/browse/QTBUG-7055714:29
zygaany hints appreciated, I will try to make the change next14:29
zygas/make the change/patch upstream qt, commit and send/14:29
zygaQt takes *forever* to clone :/14:30
tsimonq2heh14:33
tsimonq2Sure, talk in a bit.14:34
zygabrb, walk14:37
didrocksoSoMoN: I'm unsure how we could do this without hackeries15:06
didrocksoSoMoN: it needs to work offline and so on, last time, those were hacks in the upgrader and it had quite some impacts15:06
oSoMoNdidrocks, couldn't it be a postinst in the yaru package that tries to remove the snap, but doesn't blow up if it can't ?15:08
oSoMoNdunno whether it's a good idea, just thinking out loud15:08
didrocksoSoMoN: we can try snap remove community || true15:08
didrocksoSoMoN: would like to see that working, indeed, removing is supposively less complex than installing :p15:08
oSoMoNdidrocks, note that when I upgraded and rebooted, I was logged into the communitheme session, but I suppose if the snap is removed gdm3 would fall back to the default ubuntu session, right?15:09
didrocksoSoMoN: yes! which would be the positive upside of this15:11
didrocksoSoMoN: is that something you want to work on/test?15:12
oSoMoNdidrocks, sure, I can give it a try15:13
didrockssweet! :)15:13
oSoMoNI'll start by filing a bug to track the proposal15:13
didrocksoSoMoN: I guess use a VM, snapshot and try various scenarios (no network, default session fallback)15:13
didrocksyep15:13
k_alamHi, Is anyone here works on libreoffice on Ubuntu ?15:14
tsimonq2Hey k_alam :)15:15
didrocksa french guy :)15:15
tsimonq2!ask15:15
ubot5Please don't ask to ask a question, simply ask the question (all on ONE line and in the channel, so that others can read and follow it easily). If anyone knows the answer they will most likely reply. :-) See also !patience15:15
tsimonq2That's the canned response. :P15:15
tsimonq2didrocks: Oh he's French? TIL :D15:15
didrockstsimonq2: sure, ZE only one :p15:16
oSoMoNk_alam, I do, what's your question?15:16
k_alamCan you look into this ? If it can be imported as patch until it gets merged on upstream....https://gerrit.libreoffice.org/#/c/60489/215:17
k_alamLO Bug: https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1589215/15:17
ubot5Ubuntu bug 1589215 in libreoffice (Ubuntu) "'Unknown Application Name' menu item in Indicator Applet Appmenu when LibreOffice is running" [High,Confirmed]15:17
tsimonq2didrocks: I thought you were too :)15:17
didrockstsimonq2: *maybe* ;)15:18
oSoMoNk_alam, I filed the upstream bug, the patch is abandoned as it's not complete, but I can have a look next week15:19
k_alamAlright. Thanks.15:20
k_alam:)15:20
zygare15:24
zygatsimonq2: like this? http://paste.ubuntu.com/p/w3K3Y2KHGZ/15:24
zygatsimonq2: I'll check out salda next15:24
oSoMoNdidrocks, bug #179260415:37
ubot5bug 1792604 in yaru-theme (Ubuntu) "Installing yaru should try to remove the communitheme snap" [Undecided,New] https://launchpad.net/bugs/179260415:37
oSoMoNdidrocks, can you see any reason why someone (e.g. a yaru developer) might want to keep the snap around when upgrading?15:38
didrocksoSoMoN: testing other PR, but in that case, they can reinstall15:51
oSoMoNyeah, it would only be a minor inconvenience I guess15:52
didrocksindeed15:52
kenvandinei really wish libgweather was more easily relocatable :/16:04
=== pstolowski is now known as pstolowski|afk
oSoMoNtime to wrap up, have a good week-end everyone16:19
zygakenvandine: hey16:39
zygakenvandine: as in filesystem wise?16:40
kenvandinezyga, yeah16:41
zygakenvandine: what does it need to access?16:41
kenvandineit expects an xml file to be installed at $datadir16:41
zygakenvandine: *layouts* :)16:41
kenvandinei thought we aren't supposed to use that?16:41
zygano?16:41
zygathe plan is to release them out of beta in the next release16:41
kenvandineah16:42
kenvandinecool16:42
zygabut if you want it in 2.35 then yes, a bit more annoying to use16:42
zygayeah, the blocking bug is fixed now, just waiting for reviews16:42
zygaI'll get the team to review it next week if possible16:42
zygawill you be at the sprint?16:42
kenvandinei'm planning to16:42
kenvandinebut...16:42
kenvandinewe have a hurricane bearing down on us16:42
kenvandineso not sure my flight will get out16:43
zygayeah, I'm following the news16:43
zygaI hope everyone will follow the evacuation plans16:43
kenvandinethey canceled flights today and the storm hasn't quite reached our area yet16:43
kenvandinethe worst of it will be tomorrow afternoon, which is when my flight is :(16:44
kenvandineit won't be too bad where I live, but enough wind to cancel flights16:44
zygastay safe!16:44
kenvandinethanks!16:44
kenvandinezyga, can i try to use layouts now?16:44
zyga  absolutely16:45
zygahold on16:45
zygahttps://forum.snapcraft.io/t/snap-layouts/720716:45
zyga:)16:45
zygadocumented!16:45
kenvandinewoot16:45
kenvandinezyga, how long before this is enabled in stable?16:50
zygaI strongly want to enable it next week in master16:54
zygajust as soon as the fix lands16:54
zygaso next release16:54
kenvandineoh, happy times... libgweather now lets me override the path with env :)16:58
zygathat's great :)17:00
kenvandinezyga, i do have a good use case for layouts though17:00
kenvandinei have to build iso-codes into quite a few desktop snaps17:00
kenvandinelayouts should be a nice improvement17:00
tsimonq2zyga: nack, the DEP-3 header should be inside the patch diff, not the patch containing it, if that makes sense.17:03
zygaah17:03
* zyga tweaks17:03
zygahttp://paste.ubuntu.com/p/4JqSb8xRwD/17:03
Laneynight, see some of you sunday!17:04
zygatsimonq2: ah, sorry17:04
tsimonq2zyga: No, in statx-seccomp ;)17:04
zygayou mean that the actual patch should have the17:04
tsimonq2Right.17:04
zygaheader :)17:04
zygano problem :)17:04
tsimonq2Yeah :)17:04
tsimonq2Thanks.17:04
zygahttp://paste.ubuntu.com/p/hJN7FSdYNw/17:06
zygatsimonq2: is salsa just gitlab?17:06
zygaand how do I apply the patch there?17:07
tsimonq2zyga: It's their GitLab instance, yes.17:07
zyganaive search for qtbase-opensource-src yields nothing17:07
tsimonq2https://salsa.debian.org/qt-kde-team/qt/qtbase17:07
tsimonq2Fork it, push to your copy, and make an MP :)17:08
zygak, let me handle the account first17:08
zygaI'm not a DD17:08
zygabut I'm a DM so .. maybe :)17:08
zygatsimonq2: upstream NACKed the patch17:13
zygaso meh :/17:13
dupondjeOh boy! I built firefox with --enable-default-toolkit=cairo-gtk3-wayland, and fonts etc look so much better :)17:43
dupondjeto bad it seems to be lagging a bit now :(17:43
=== Class7_ is now known as Class7
zygadupondje: better how?18:55

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