[01:38] <callmepk> good morning
=== cpaelzer_ is now known as cpaelzer
[05:46] <jibel> Good morning everyone!
[05:48] <oSoMoN> good morning desktoppers
[05:55] <jibel> Salut oSoMoN 
[05:56] <oSoMoN> salut jibel 
[06:09] <callmepk> hi jibel oSoMoN 
[06:15] <oSoMoN> hey callmepk 
[06:23] <jibel> Hi callmepk 
[06:30] <ricotz> good morning :)
[06:36] <jibel> callmepk, regarding wsl-integration.sh issue I'd go with a simpler syntax than the proposed fix. ig UBUNTU_WSL_GUI_THEME=${UBUNTU_WSL_GUI_THEME-} 
[06:36] <jibel> instead of adding a function to check the sanity of the variables
[06:36] <jibel> what do you think?
[06:37] <oSoMoN> good morning ricotz 
[06:38] <jibel> good morning ricotz 
[06:39] <callmepk> hi ricotz 
[06:39] <callmepk> jibel let me check it now
[06:40] <jibel> callmepk, I can propose a patch if you prefer
[06:43] <callmepk> jibel I am a bit confused, can you help me propose a patch? Thanks!
[06:43] <jibel> callmepk, that should do it https://paste.ubuntu.com/p/GyMYjSVtk8/
[06:44] <jibel> i'll test it and propose something
[06:45] <callmepk> I will test it now
[06:49] <jibel> callmepk, is there a reason to declare the variables readonly ?
[06:51] <jibel> https://paste.ubuntu.com/p/zmWJqy9Cc8/ without RO variables
[06:52] <callmepk> jibel it is mostly used to prevent accidental change of the vaiable
[06:53] <jibel> right, but since we maintain the script. Also the way it's done, if you make a typo in the conf file, then you'll declare a variable that doesn't exist in the script. 
[06:54] <jibel> I mean since we maintain the script there is little risk to accidentally change the variable
[06:55] <callmepk> Okay
[06:58] <jibel> but it's just a proposal, do as you prefer
[07:06] <didrocks> good morning
[07:07] <callmepk> hi didrocks 
[07:07] <didrocks> hey callmepk 
[07:08] <jibel> Salut didrocks 
[07:09] <didrocks> salut jibel 
[07:09] <seb128> goood morning desktopers
[07:09] <jibel> salut seb128 
[07:09] <seb128> hey didrocks, didrocks, callmepk, how are you?
[07:09] <callmepk> hi seb128 
[07:12] <callmepk> I am okay seb128 , how about you?
[07:12] <seb128> I'm alright, had a busy but nice weekend and it's nice and sunny today
[07:13] <didrocks> salut seb128, ça va :)
=== cpaelzer_ is now known as cpaelzer
[08:04] <laney> \o
[08:10] <marcustomlinson> morning all
[08:14] <jibel> morning marcustomlinson laney 
[08:15] <marcustomlinson> hey jibel
[08:27] <oSoMoN> good morning didrocks, seb128, laney, marcustomlinson 
[08:28] <didrocks> salut oSoMoN, hey marcustomlinson, laney 
[08:29] <laney> hey marcustomlinson jibel oSoMoN didrocks 
[08:34] <marcustomlinson> hey oSoMoN didrocks laney
[08:39] <callmepk> morning laney marcustomlinson 
[08:44] <laney> ahoy callmepk!
[08:45] <marcustomlinson> hi callmepk
[08:45] <jibel> callmepk, I merged the fix from microhobby then when you have a minute to review https://github.com/canonical/ubuntu-wsl-integration/pull/4 
[08:45] <ubottu> Pull 4 in canonical/ubuntu-wsl-integration "Simplify variable declaration check" [Open]
[08:45] <gitlab-bot> canonical issue (Pull request) 4 in ubuntu-wsl-integration "Simplify variable declaration check" [Open]
[08:45] <jibel> we can then do a release
[08:49] <callmepk> jibel I have tried with some other ways like trap but with no luck, so I merged it
[08:52] <jibel> yeah, same. Since it's sourced when the session is initialized it's a bit tricky. However failing to start if this scripts fails seems okay, since we wouldn't be able to properly initialize the wsl session
[09:01] <laney> jibel: callmepk: not sure if intentional, but if you use the "${FOO-}" form rather than "${FOO:-}" then it doesn't fire for set-but-empty (like if someone uses "FOO=" in a script to clear it instead of using "unset"), usually you want the latter
[09:06] <jibel> laney, it is intentional 
[09:07] <laney> alright, I saw bool-like values and thought that it would be odd if they ended up empty
[09:07] <laney> but as long as you did it on purpose
[10:30] <ricotz> seb128, hi :)
[12:30] <cpaelzer> hi, I'm unsure about https://launchpad.net/ubuntu/+source/xrdp/0.9.15-1 while on +1 duty
[12:31] <cpaelzer> is this something the desktop team would look after anyway or is this +1-worthy as it would otherwise be languishing in -proposed?
[12:31] <callmepk> jibel Sorry for being late, but I got the new version ready on github: https://github.com/canonical/ubuntu-wsl-integration/releases/tag/v0.31.3. Could you help me on uploading?
[12:42] <jibel> callmepk, sure
[12:42] <callmepk> thanks jibel 
[13:24] <didrocks999> callmepk: sponsored!
[13:24] <callmepk> Thanks didrocks999 !
[13:26] <didrocks999> yw :)
[14:37] <seb128> ricotz, hey, sorry forgot to reply earlier, how are you?
[14:39] <ricotz> seb128, hey, I am good, finally some sunny days, how are you?
[14:39] <ricotz> seb128, I noticed you retried libreoffice, this might be a real issue
[14:40] <ricotz> I have triggered autopkgtests for the hirsute ppa builds of the identical source
[14:41] <ricotz> the newer openjdk-lts package might be the cause
[14:41] <ricotz> hey hellsworth 
[14:46] <seb128> ricotz, ok, thanks for the headsup, I usully retry once in case it's flaky
[14:46] <ricotz> seb128, I already retried it earlier ;) https://autopkgtest.ubuntu.com/packages/libr/libreoffice/impish/armhf
[14:47] <seb128> right, it was not there at the time I retried, the website doesn't make it easy to see if there is already a pending retry
[14:48] <ricotz> so java is crashing there, if the hirsute/armhf autopkgtest, I would suggest to force libreoffice to transition to avoid blocking things
[14:48] <hellsworth> good day desktopers, nice to be back after a week off
[14:48] <seb128> hey heather, wb!
[14:48] <hellsworth> thanks!!
[14:49] <seb128> ricotz, we should perhaps retry 1:7.1.3-0ubuntu1 see if that regressed
[14:50] <ricotz> seb128, that sounds good
[14:51] <ricotz> please do so
[15:10] <oSoMoN> welcome back hellsworth 
[15:11] <hellsworth> thanks oSoMoN :)
[15:31] <jibel> Good morning hellsworth 
[15:31] <hellsworth> hi there jibel :)
[16:57] <KGB-1> tracker pristine-tar 3904927 Iain Lane tracker_3.1.1.orig.tar.xz.delta tracker_3.1.1.orig.tar.xz.id * pristine-tar data for tracker_3.1.1.orig.tar.xz * https://deb.li/3ESth
[16:57] <KGB-1> tracker upstream/latest d13ffe4 Iain Lane * pushed 1405 commits (first 5 follow) * https://deb.li/5762
[16:57] <KGB-1> tracker upstream/latest c78a0f7 Carlos Garnacho src/libtracker-data/tracker-data-manager.c * libtracker-data: Plug leak * https://deb.li/3XYfC
[16:57] <KGB-1> tracker upstream/latest 4b20a14 Carlos Garnacho src/libtracker-sparql/tracker-endpoint-dbus.c * libtracker-sparql: Plug leak * https://deb.li/JYoU
[16:57] <KGB-1> tracker upstream/latest 19ce818 Carlos Garnacho src/libtracker-data/tracker-data-manager.c * libtracker-data: Plug leak * https://deb.li/3GirF
[16:57] <KGB-1> tracker upstream/latest 45b44b9 Carlos Garnacho src/libtracker-sparql/tracker-namespace-manager.c * libtracker-sparql: Plug leak on API misuse * https://deb.li/3Usw7
[16:57] <KGB-1> tracker upstream/latest 143f1bb Carlos Garnacho src/libtracker-data/tracker-db-interface-sqlite.c * libtracker-data: Plug leak * https://deb.li/it7Yd
[16:59] <laney> TBC
[18:46] <KGB-1> gnome-control-center signed tags 0d7e35e Marco Trevisan ubuntu/1%3.38.6-0ubuntu2 * gnome-control-center Debian release 1:3.38.6-0ubuntu2 * https://deb.li/ZJ9G
[18:46] <KGB-1> gnome-control-center ubuntu/master af5ed95 Marco Trevisan (Treviño) debian/patches/ series 0033-display-Ensure-we-list-available-scaling-values-keeping-t.patch * debian/patches: Ensure we list all the usable scaling values * https://deb.li/3LMWM
[18:47] <KGB-1> gnome-control-center ubuntu/master 84c9686 Marco Trevisan (Treviño) debian/patches/0008-Allow-tweaking-some-settings-for-Ubuntu-Dock.patch * debian/patches: Do not leak a variant builder on single monitor setup * https://deb.li/08Xy
[18:47] <KGB-1> gnome-control-center ubuntu/master 6a8d9b9 Marco Trevisan (Treviño) debian/changelog * Update changelog * https://deb.li/ipxd4
[18:47] <KGB-1> gnome-control-center ubuntu/master 37a8d30 Marco Trevisan (Treviño) debian/changelog * Upload to impish * https://deb.li/jVxe