[06:04] <didrocks> good morning
[06:05]  * rs2009 waves
[06:05] <rs2009> hey didrocks
[06:06] <jibel> Good morning all
[06:07] <duflu> Morning didrocks, rs2009, jibel 
[06:07] <rs2009> hi duflu and jibel
[06:15] <didrocks> hey rs2009, jibel, duflu 
=== ackk is now known as ack
[07:10] <KGB-0> libxklavier Iain Lane 333883 * commented merge request !1 * https://deb.li/37jJM
[08:34] <oSoMoN> good morning desktoppers
[08:42] <seb128> lut oSoMoN, comment ça va ?
[08:49] <duflu> Hi oSoMoN and seb128 
[08:54] <seb128> hey duflu, how are you?
[08:59] <duflu> seb128, going alright. How are you?
[09:03] <seb128> duflu, I got a cold from one of the children, was not feeling great yesterday but today is better
[09:04] <duflu> Hope it improves soon
[09:11] <seb128> thanks
[12:38] <KGB-0> gnome-desktop pristine-tar 9327479 Jeremy Bicha gnome-desktop_43~rc.orig.tar.xz.delta gnome-desktop_43~rc.orig.tar.xz.id * pristine-tar data for gnome-desktop_43~rc.orig.tar.xz * https://deb.li/i3c57
[12:38] <KGB-2> gnome-desktop upstream/latest f3ad20c Jeremy Bicha * pushed 33 commits (first 5 follow) * https://deb.li/32BSi
[12:38] <KGB-2> gnome-desktop upstream/latest 13ac223 Goran Vidović desktop-docs/fdl/hr/hr.po * Update Croatian translation * https://deb.li/qb20
[12:38] <KGB-2> gnome-desktop upstream/latest b67b0e6 Goran Vidović desktop-docs/fdl/hr/hr.po * Update Croatian translation * https://deb.li/38X92
[12:39] <KGB-2> gnome-desktop upstream/latest 69e6ff6 Goran Vidović desktop-docs/lgpl/ LINGUAS hr/hr.po * Add Croatian translation * https://deb.li/Hnis
[12:39] <KGB-2> gnome-desktop upstream/latest 5f9b064 Goran Vidović desktop-docs/lgpl/hr/hr.po * Update Croatian translation * https://deb.li/3RFbU
[12:39] <KGB-2> gnome-desktop upstream/latest 2d0d30d Goran Vidović desktop-docs/gpl/hr/hr.po * Update Croatian translation * https://deb.li/T2hn
[12:46] <KGB-0> tracker signed tags fd3f973 Jeremy Bicha upstream/3.4.0_rc * Upstream version 3.4.0~rc * https://deb.li/3t2Wc
[12:47] <KGB-2> tracker upstream/latest 203125e Jeremy Bicha * pushed 99 commits (first 5 follow) * https://deb.li/tblD
[12:47] <KGB-2> tracker upstream/latest 50132bf Carlos Garnacho src/libtracker-sparql/core/tracker-data-manager.c * core: Cast cursor variable to the right type * https://deb.li/3rXAz
[12:47] <KGB-2> tracker upstream/latest ae15b34 Carlos Garnacho tests/libtracker-sparql/tracker-namespaces-test.c * tests: Remove unused variables * https://deb.li/yBIR
[12:47] <KGB-2> tracker upstream/latest 9d22a62 Carlos Garnacho docs/tools/tracker-docgen-xml.c * docs: Use correct integer type for iterator * https://deb.li/3X2iv
[12:47] <KGB-2> tracker upstream/latest 051359a Carlos Garnacho docs/ reference/libtracker-sparql/meson.build tools/tracker-main.c tools/tracker-ontology-model.c tools/tracker-ontology-model.h * docs: Avoid warnings doubly parsing base ontology * https://deb.li/J2EE
[12:47] <KGB-2> tracker upstream/latest 5904dbb Carlos Garnacho meson.build * build: Use generic .get_variable method to get pkgconfig variables * https://deb.li/3yFLG
=== popey6 is now known as popey
[14:28] <KGB-2> meld signed tags 1c5d580 Jeremy Bicha upstream/3.22.0 * Upstream version 3.22.0 * https://deb.li/3WfuF
[14:28] <KGB-2> meld upstream/latest afa327d Jeremy Bicha * pushed 1014 commits (first 5 follow) * https://deb.li/eHm7
[14:28] <KGB-2> meld upstream/latest 386b9eb Hugo Carvalho po/pt.po * Update Portuguese translation * https://deb.li/3AvQd
[14:28] <KGB-2> meld upstream/latest 3094b9b Yuri Chornoivan po/uk.po * Update Ukrainian translation * https://deb.li/FvJ2
[14:28] <KGB-2> meld upstream/latest 7df17ad Kai Willadsen meld/filediff.py * filediff: Ignore cursor changes for non-focused panes (#646) * https://deb.li/u0ah
[14:28] <KGB-0> meld pipeline Jeremy Bicha 420629 * pending (extract-source: pending; build: created; build i386: created; build source: created; test-build-any: created; test-build-all: created; test-crossbuild-arm64: created; reprotest: created; lintian: created; autopkgtest: created; blhc: created; piuparts: created)
[14:28] <KGB-2> meld upstream/latest 4b3f95d Sergej A po/ru.po * Update Russian translation * https://deb.li/i4HHc
[14:28] <KGB-0> meld pipeline Jeremy Bicha 420629 * running (extract-source: running; build: created; build i386: created; build source: created; test-build-any: created; test-build-all: created; test-crossbuild-arm64: created; reprotest: created; lintian: created; autopkgtest: created; blhc: created; piuparts: created)
[14:28] <KGB-2> meld upstream/latest 98e9a7e Kai Willadsen maint * Update maintenance script for newer Gitlab release API * https://deb.li/3FHdE
[14:34] <KGB-0> xdg-desktop-portal-gnome signed tags 03eb559 Jeremy Bicha upstream/43_rc * Upstream version 43~rc * https://deb.li/1wln
[14:34] <KGB-0> xdg-desktop-portal-gnome pipeline Jeremy Bicha 420633 * pending (extract-source: pending; build: created; build i386: created; build source: created; test-build-any: created; test-build-all: created; test-crossbuild-arm64: created; reprotest: created; lintian: created; autopkgtest: created; blhc: created; piuparts: created)
[14:34] <KGB-0> xdg-desktop-portal-gnome upstream/latest d377a03 Jeremy Bicha * pushed 15 commits (first 5 follow) * https://deb.li/3DwUx
[14:34] <KGB-0> xdg-desktop-portal-gnome upstream/latest dcbbfee Georges Basile Stavracas Neto meson.build * Post-release version bump * https://deb.li/DIx0
[14:34] <KGB-0> xdg-desktop-portal-gnome upstream/latest 3400dda Goran Vidović po/hr.po * Update Croatian translation * https://deb.li/3R0m2
[14:34] <KGB-0> xdg-desktop-portal-gnome upstream/latest 5c1a23a Piotr Drąg po/pl.po * Update Polish translation * https://deb.li/TvPi
[14:34] <KGB-0> xdg-desktop-portal-gnome upstream/latest 202b329 Jürgen Benvenuti po/de.po * Update German translation * https://deb.li/iiUuh
[14:35] <KGB-0> xdg-desktop-portal-gnome upstream/latest b33d02e Peter Hutterer src/ remotedesktop.c screencast.c session.c session.h * session: emit the Closed signal when we're closing on our own account * https://deb.li/QvbN
[14:35] <KGB-2> xdg-desktop-portal-gnome pipeline Jeremy Bicha 420633 * running (extract-source: running; build: created; build i386: created; build source: created; test-build-any: created; test-build-all: created; test-crossbuild-arm64: created; reprotest: created; lintian: created; autopkgtest: created; blhc: created; piuparts: created)
[14:35] <KGB-2> xdg-desktop-portal-gnome pristine-tar 4108ff6 Jeremy Bicha xdg-desktop-portal-gnome_43~rc.orig.tar.xz.delta xdg-desktop-portal-gnome_43~rc.orig.tar.xz.id * pristine-tar data for xdg-desktop-portal-gnome_43~rc.orig.tar.xz * https://deb.li/V0sg
[14:36] <KGB-2> meld pipeline Jeremy Bicha 420629 * [8 minutes and 10 seconds] success (extract-source: success; build: success; build i386: success; build source: success; test-build-any: success; test-build-all: success; test-crossbuild-arm64: success; reprotest: success; lintian: success; autopkgtest: success; blhc: success; piuparts: success)
[14:38] <KGB-2> gnome-disk-utility upstream/latest 23c40f7 Jeremy Bicha * pushed 8 commits (first 5 follow) * https://deb.li/3y0bV
[14:38] <KGB-2> gnome-disk-utility upstream/latest 3a6dca0 Maximiliano Sandoval R flatpak/org.gnome.DiskUtility.json * flatpak: Use devel build for blockdev * https://deb.li/3ifn4
[14:38] <KGB-2> gnome-disk-utility upstream/latest d598b70 Kai Lüke flatpak/org.gnome.DiskUtility.json * flatpak: Use devel branch for UDisks * https://deb.li/i20dD
[14:38] <KGB-2> gnome-disk-utility upstream/latest aad5214 Kai Lüke flatpak/org.gnome.DiskUtility.json * flatpak: Use libfdisk from util-linux * https://deb.li/wHaF
[14:38] <KGB-2> gnome-disk-utility upstream/latest 6647592 Goran Vidović po/hr.po * Update Croatian translation * https://deb.li/3tvH1
[14:38] <KGB-2> gnome-disk-utility upstream/latest 0f3179c Jordi Mas po/ca.po * Update Catalan translation * https://deb.li/GIpd
[14:39] <KGB-2> gnome-disk-utility pristine-tar ccae436 Jeremy Bicha gnome-disk-utility_43~rc.orig.tar.xz.delta gnome-disk-utility_43~rc.orig.tar.xz.id * pristine-tar data for gnome-disk-utility_43~rc.orig.tar.xz * https://deb.li/E0uC
[14:40] <KGB-0> gnome-disk-utility ubuntu/master 6e70319 Jeremy Bicha * pushed 13 commits (first 5 follow) * https://deb.li/lwFE
[14:40] <KGB-0> gnome-disk-utility ubuntu/master 3a6dca0 Maximiliano Sandoval R flatpak/org.gnome.DiskUtility.json * flatpak: Use devel build for blockdev * https://deb.li/3ifn4
[14:40] <KGB-0> gnome-disk-utility ubuntu/master d598b70 Kai Lüke flatpak/org.gnome.DiskUtility.json * flatpak: Use devel branch for UDisks * https://deb.li/i20dD
[14:40] <KGB-0> gnome-disk-utility ubuntu/master aad5214 Kai Lüke flatpak/org.gnome.DiskUtility.json * flatpak: Use libfdisk from util-linux * https://deb.li/wHaF
[14:40] <KGB-0> gnome-disk-utility ubuntu/master 6647592 Goran Vidović po/hr.po * Update Croatian translation * https://deb.li/3tvH1
[14:40] <KGB-0> gnome-disk-utility ubuntu/master 0f3179c Jordi Mas po/ca.po * Update Catalan translation * https://deb.li/GIpd
[15:07] <KGB-2> gnome-initial-setup upstream/latest 8a5a022 Jeremy Bicha * pushed 57 commits (first 5 follow) * https://deb.li/UWq7
[15:07] <KGB-0> gnome-initial-setup pristine-tar 5228927 Jeremy Bicha gnome-initial-setup_43~rc.1.orig.tar.xz.delta gnome-initial-setup_43~rc.1.orig.tar.xz.id * pristine-tar data for gnome-initial-setup_43~rc.1.orig.tar.xz * https://deb.li/Xg6C
[15:07] <KGB-2> gnome-initial-setup upstream/latest d5ef1dc Ask Hjorth Larsen po/da.po * Updated Danish translation * https://deb.li/371wd
[15:07] <KGB-2> gnome-initial-setup upstream/latest 8ef5078 Emin Tufan Çetin po/tr.po * Update Turkish translation * https://deb.li/mPoF
[15:07] <KGB-2> gnome-initial-setup upstream/latest 9441b34 Marek Černocký po/cs.po * Updated Czech translation * https://deb.li/3ayr1
[15:07] <KGB-2> gnome-initial-setup upstream/latest ae3c4b9 Balázs Úr po/hu.po * Update Hungarian translation * https://deb.li/ry8h
[15:07] <KGB-2> gnome-initial-setup upstream/latest d559f81 Jordan Petridis meson.build * Bump version to 43.rc.1 * https://deb.li/3IWtc
[15:35] <KGB-0> eog-plugins pristine-tar ffe906e Jeremy Bicha eog-plugins_42.1.orig.tar.xz.delta eog-plugins_42.1.orig.tar.xz.id * pristine-tar data for eog-plugins_42.1.orig.tar.xz * https://deb.li/ijgGR
[16:04] <KGB-2> gbrainy signed tags 90ea2ee Jeremy Bicha upstream/2.4.5 * Upstream version 2.4.5 * https://deb.li/Sbc
[16:04] <KGB-2> gbrainy pristine-tar d2477aa Jeremy Bicha gbrainy_2.4.5.orig.tar.gz.delta gbrainy_2.4.5.orig.tar.gz.id * pristine-tar data for gbrainy_2.4.5.orig.tar.gz * https://deb.li/36Ydg
[16:10] <KGB-2> xdg-user-dirs tags 1742df3 Jeremy Bicha upstream/0.18 * Upstream version 0.18 * https://deb.li/3giIA
[16:10] <KGB-0> xdg-user-dirs upstream/latest b1cb9b7 Jeremy Bicha * pushed 9 commits (first 5 follow) * https://deb.li/ixSpi
[16:10] <KGB-0> xdg-user-dirs upstream/latest 698c323 Rafael Fontenelle po/pt_BR.po * po: update pt_BR.po (from translationproject.org) * https://deb.li/4sF4
[16:10] <KGB-0> xdg-user-dirs upstream/latest 97ff189 Baris Cicek po/tr.po * po: update tr.po (from translationproject.org) * https://deb.li/3W7ND
[16:10] <KGB-0> xdg-user-dirs upstream/latest 0bbc123 Felix Eckhofer man/ xdg-user-dir.xml xdg-user-dirs-update.xml * Use correct file name in man pages * https://deb.li/ei1j
[16:10] <KGB-0> xdg-user-dirs upstream/latest c2ec04e Rafael Fontenelle git-tp-sync * git-tp-sync: add new langs to LINGUAS * https://deb.li/3ASxR
[16:11] <KGB-0> xdg-user-dirs upstream/latest 5defafb Roddy Shuler .gitignore po/remove-potcdate.sed * build: Remove generated file from repo * https://deb.li/FSdx
[16:42] <oSoMoN> lissyx, new assertions for the hunspell changes are now live in the store, I've verified that the latest snap in the edge channel works OOTB, and I've cherry-picked your changes to the beta branch
[16:42] <lissyx> oSoMoN, with autoconnect?
[16:42] <oSoMoN> yes
[16:45] <lissyx> oSoMoN, it's not  workign for me
[16:45] <lissyx> revision 1818
[16:45] <oSoMoN> what do you mean, it's not being auto-connected?
[16:46] <lissyx> it's autoconnected
[16:46] <lissyx> but there is no hunspell properly populated
[16:46] <oSoMoN> lissyx, are you using parallel installs?
[16:46] <lissyx> I have no idea what this is
[16:47] <lissyx> just did "snap refresh --edge firefox"
[16:47] <oSoMoN> Then probably not :) It allows one to install several revisions of the same snap under different names, e.g. "firefox" and "firefox_nightly"
[16:48] <lissyx> it's 106.0a1
[16:48] <oSoMoN> okay, so there's likely a problem with the connect hook that didn't mount the directory correctly?
[16:49] <oSoMoN> lissyx, what's the output of `mount | grep host-hunspell` inside the snap's shell ?
[16:49] <oSoMoN> and output of `ls -l $SNAP_COMMON/host-hunspell` ?
[16:49] <lissyx> oSoMoN, I've removed completely, reinstalled --edge directly and it works now
[16:50] <oSoMoN> hrm… that sounds racy, I don't like that
[16:50] <lissyx> or it is because I had some hack due to my manual connections?
[16:51] <lissyx> I'm reinstalling stable
[16:51] <lissyx> then will refresh --edge
[16:51] <oSoMoN> I'm seeing the same issue in my "firefox_nightly" parallel install, but I'm not sure how to inspect the logs of running the hooks
[16:51] <oSoMoN> ogra, do you happen to know that? ^
[16:52] <ogra> hooks that exit 0 will never log ... thats hardcoded in snapd ... 
[16:52] <ogra> so you'D add en exit 1 t them 
[16:52] <ogra> *add an
[16:52] <lissyx> exit 1 would fail on legit cases
[16:53] <ogra> yes
[16:53] <lissyx> oSoMoN, ok, so, removal, install of --stable, running firefox, no dic, refresh --edge, it's properly populated
[16:53] <ogra> only failing hooks are allowed to output to logs
[16:53] <lissyx> I really think it was just my manual connections
[16:54] <oSoMoN> I'm not sure, because my firefox_nightly install doesn't have manual connections, and I'm seeing the same problem
[16:55] <oSoMoN> could it be that "post-refresh" was run before "connect-plug-host-hunspell", by any chance?
[16:56] <ogra> which of your hooks populates hunspell ? (i think "install" is only run at install time ... not necessarily for refresh)
[16:56] <oSoMoN> post-refresh does
[16:56] <ogra> ah
[16:57] <ogra> yeah, there could be a race ... you can use "snapctl is-connected <your-fancy-interface>" from inside the hook though ... and wait for the connection to happen
[16:58] <ogra> or simply use a connect hook 
[16:58] <ogra> https://snapcraft.io/docs/interface-hooks
[16:58] <oSoMoN> we do use a connect hook, that one bind mounts the host's /usr/share/hunspell to $SNAP_COMMON/host-hunspell
[16:59] <oSoMoN> what's weird is that inside the snap's shell, I'm seeing  $SNAP_COMMON/host-hunspell, but it's empty
[16:59] <ogra> hmm, weird
[16:59] <lissyx> oSoMoN, any "manual" in your "snap connections firefox" or firefox_nightly ?
[17:00] <oSoMoN> ok, so that means that "snapctl mount --persistent -o ro,bind,noatime,noexec /usr/share/hunspell $SNAP_COMMON/host-hunspell" in the hook failed
[17:00] <oSoMoN> lissyx, no
[17:01] <ogra> then the hook should have failed altogether ... unless you have an "|| true" or some such in there 
[17:01] <oSoMoN> we don't: https://github.com/canonical/firefox-snap/blob/nightly/snap/hooks/connect-plug-host-hunspell
[17:01] <ogra> (iirc hooks are run with -e ... so errors of called commands should kill the hook with nonzero exit code)
[17:02] <oSoMoN> ogra, where would the error be logged, then?
[17:02] <ogra> journal 
[17:03] <ogra> but your snap would also automatically roll back to the installed version 
[17:03] <oSoMoN> it clearly didn't do that
[17:03] <ogra> right
[17:03] <ogra> weird
[17:04] <ogra> then your mount command somehow did not error ... 
[17:04] <ogra> or set -e for hooks is not true anymore 
[17:10] <oSoMoN> that's the relevant bits of the journal: https://paste.ubuntu.com/p/sWdvZ7Fhzy/
[17:13] <oSoMoN> I'm seeing "Mounting Mount unit for firefox_nightly, revision 1818 via mount-control..." after connect-plug-host-hunspell is executed, that doesn't sound right, unless I misinterpret what this "… via mount-control" message means
[17:29] <oSoMoN> I must be misinterpreting indeed, because in a VM where the host's /usr/share/hunspell directory was correctly bind-mounted to $SNAP_COMMON/host-hunspell, the ordering of the messages in the journal is the same
[17:32] <oSoMoN> I can't spend more time on this right now, I'll file a bug to keep track of the isuse
[17:32] <oSoMoN> issue
[17:44] <oSoMoN> there: https://bugzilla.mozilla.org/show_bug.cgi?id=1789695
[17:44] <ubottu> Mozilla bug 1789695 in Firefox "[snap] connect-plug-host-hunspell failed to mount /usr/share/hunspell from the host" [--, New]
[17:51] <lissyx> oSoMoN, are you working on it or should I ?
[17:55] <lissyx> there is definitively some memory leak at some point on gnome-shell
[17:56] <lissyx> 13 days since my session was started, gnome-shell reports 3.1% of memory in htop, while it was 1.1% session fully loaded when I started it
[17:56] <oSoMoN> lissyx, I can't work on it now, which is why I filed the bug to make sure I don't forget about it. If you can take a look yourself, please do!
[17:56] <lissyx> only extensions enabled are Lock Keys and Ubuntu AppIndicator
[17:57] <lissyx> oSoMoN, unfortunately, the audio decoding work is getting more hairy at the moment and it's quite complicated for me to get time to work on both at the same time those days
[17:57] <oSoMoN> understood, no worries
[17:57] <lissyx> next week I'll commute to the office since we have a new member on the team who is doing his onboarding
[17:58] <lissyx> 26-30 we're in hawaii so departing 24, coming back on the 2nd of october?
[17:58] <lissyx> (not that i'm taking time off there, just long travel)