jibelGood morning05:04
dufluMorning jibel 05:18
jibelHi duflu 05:18
oSoMoNgood morning desktoppers05:45
seb128goood morning desktopers05:47
seb128lut oSoMoN, comment ça va aujourd'hui ?05:47
seb128hey ricotz , how are you?05:48
ricotzseb128, hey, a bit sleepy ;P, how are you05:49
oSoMoNsalut seb128, ça va bien, et toi?05:53
oSoMoNhey ricotz 05:53
ricotzoSoMoN, hey05:54
seb128the night has been a bit rough, let's see how the day goes, starting with coffee! 06:01
ricotzseb128, maybe you have a moment after getting coffee for https://bugs.launchpad.net/ubuntu/+source/vala/+bug/1897901 :)06:19
ubot5Ubuntu bug 1897901 in vala (Ubuntu) "FFe: Sync vala 0.48.11-1 (universe) from Debian unstable (main)" [Undecided,New]06:19
dufluMorning oSoMoN and seb128 06:28
marcustomlinsonmorning jibel duflu oSoMoN seb128 ricotz07:18
jibelHi marcustomlinson07:25
GunnarHjGood morning seb128! Thoughts about bug #1897858?07:26
ubot5bug 1897858 in ibus (Ubuntu) "[UIFe] IBus 1.5.23" [Medium,New] https://launchpad.net/bugs/189785807:26
dufluMorning marcustomlinson 07:35
didrocksgood morning07:52
dufluHi didrocks 07:58
didrockshey duflu 07:59
seb128hey marcustomlinson, didrocks, duflu, how are you?08:03
marcustomlinsongood thanks you?08:03
seb128GunnarHj, hey, no opinion about this one, I read over it yesterday but I don't know ibus details much, seems fine to me08:03
dufluseb128, kind of average. See Mattermost :)08:03
seb128marcustomlinson, the night was a bit cahotic but alright otherwise08:03
marcustomlinsonseb128: hope the day goes better08:04
seb128let's see how far coffee carries me08:04
seb128ricotz, you don't need a ffe for a bugfix update, it's only for feature updates. I will sync but the archive is frozen atm for beta so it's probably not going to be accepted today08:05
didrockshey seb128, fine, and you?08:06
didrockshey marcustomlinson 08:06
seb128didrocks, tired but rather ok otherwise :)08:06
ricotzhey marcustomlinson didrocks duflu 08:06
GunnarHjseb128: Ok, good. Then I'll probably ask you to sponsor if the release team (Laney?) approves.08:06
didrockshey ricotz 08:06
ricotzseb128, ah, I see, the kept me thinking https://bugs.launchpad.net/ubuntu/+source/vala/+bug/1891318/comments/4 :(08:07
ubot5Ubuntu bug 1891318 in vala (Ubuntu) "[SRU] Update to vala 0.48.10 in focal" [Undecided,Incomplete]08:07
seb128GunnarHj, try rather chasing they on their channel, L_aney is off until monday08:07
GunnarHjseb128: Ok, will do.08:08
seb128ricotz, right, SRU team got a bit more picky, they also don't want to give an exception to the shell because it can breaks extensions :-/08:08
ricotzseb128, I am still hoping to while keeping 0.48.x in groovy it receives enough testing to be ack'd as focal SRU08:10
GunnarHjseb128: You can blame me about the SRU team's new approach to gnome-shell if you like. :-/ (The bug which started it was filed and fixed by a first time contributor, and I mentored him.)08:11
seb128ricotz, right, well you don't need the standing exception, as the comment points out you could document is a standard SRU and have a ppa showing the result of rdepends rebuilds08:12
seb128GunnarHj, not your fault if gnome-shell upstream includes changes in point updates than can break extensions though...08:12
ricotzseb128, yeah08:13
GunnarHjseb128: No, of course not. But if that bug reporter hadn't got help, the bug might still have been open and the SRU team unaware of that regression.08:15
seb128oh well, it's done now!08:15
cpaelzerseb128: hiho, I have a question I'd need your team-subscribing powers for08:17
cpaelzerhttps://bugs.launchpad.net/ubuntu/+source/pnm2ppa is part of the overall cups stack it seems08:17
cpaelzerbut it was promoted to main in the dark ages (before team splits)08:17
cpaelzertherefore so far still the only subscriber is ubuntu-server and we have no experience how to deal with it08:18
cpaelzermost other "print'y" packages are subscribed by the Desktop team (and in addtion by ubuntu-printing)08:18
cpaelzerseb128: I wanted to ask if we should transition this one as well (you subscribe yours, I unsubscribe my team)08:18
seb128cpaelzer, hey, that sounds reasonable, I subscribed desktop-packages, feel free to unsubscribe server08:19
cpaelzerdone and thank you08:19
didrocksmvo: hey! We have a small PR for you on your very active smb project: https://github.com/mvo5/libsmbclient-go/pull/4 :)10:29
didrocksit’s small, but dense, we first thought the issue was on smbclient side before noticing the swapping of values :p10:29
mvodidrocks: woah, happy to review. out of curisoity, what do you use it for? it's a bit of a toy from my early go days10:31
mvodidrocks: I feel the lack of multithreading in libsmbclient make it not super useful except for special usecases I guess10:32
didrocksmvo: we are exploring GPO support on ubuntu10:33
didrockssome part is AD connection10:33
didrocksthe rest is about downloading the GPOs, through samba…10:33
didrockswe’ll probably add krb5 ticket support10:33
didrocks(to your project)10:33
mvodidrocks: woah, cool. I *never* thought this would be used outside of me playing around with cgo :)10:34
didrocksmvo: heh, there is always a use, and TBH, you rank #2 when googling for smb + golang bindings :p10:35
mvodidrocks: if you really do this I will go over the code again just to make sure that I apply the knowledge I aquired since  I originally wrote it10:36
didrocksmvo: sounds excellent!10:36
didrocksmvo: ofc, smbclient is returning an empty SMBFILE and the dreaded 'file exists' errno on existing file or directory, so we can’t really do anything with it right now (as reported on https://github.com/mvo5/libsmbclient-go/issues/1)11:26
didrocksthis is puzzling why smbclient is reporting this11:27
didrocks(same issue with a local desktop smb share, so not AD related)11:45
mvodidrocks: hm, hm, I see new PRs, looking12:45
mvodidrocks: also will try to play with this, just out of curisity, what are you testing against?13:00
mvodidrocks: samba? a real windows?13:01
didrocksmvo: we tried both: samba (local desktop share integration) and AD13:08
mvodidrocks: thanks, I see what I can do, defintiely strange13:09
didrocksyeah, indeed13:09
* ricotz cries about https://github.com/ubuntu/libreoffice-style-yaru-fullcolor/issues/11513:20
ricotzhellsworth, hey, I was about to mention the running builds for libreofice 7.0.2~rc2, but ^ :(13:20
didrocksmvo: so, in pure C, it returns also EEXISTS as errno13:33
didrocksbut it does return a fp13:33
didrocksmy theory is that cgo return "" for first object if errno isn’t null13:33
didrocks(well, return nil)13:33
mvodidrocks: that sounds plausible13:34
mvodidrocks: I'm doing a quick check with older go versions now to see when this broke13:34
didrocksmvo: ok, if I replace d, err := C.my_smbc_opendir(c.ctx, C.CString(durl)) by d := C.my_smbc_opendir(c.ctx, C.CString(durl)), d now isn’t nil13:36
didrocksso trying to get the errno will nilify the returned object (as Go standard practice)13:36
didrocksannoying Cgo does that for you13:36
didrocksthe issue is that smb never returns a non 0 errno13:37
mvodidrocks: woah, super interessting find13:38
didrocksI never noticed it, maybe that changed over Go version13:38
didrocksbecause I trust that you code used to work :)13:38
didrocksbut annoying in that case, the only way to know something went wrong is to check the returned value13:39
didrocksbut you don’t know which kind of error you got13:39
jibelit's schrondingo, if you cannot watch both the object and its error ;)13:39
didrocksconfirmed on https://utcc.utoronto.ca/~cks/space/blog/programming/GoCgoErrorReturns13:42
mvodidrocks: cool13:43
didrockswell, unsure how cool this is :p but it seems we have to change to not check error and check the returned object, opacifying any error13:43
didrockswe can propose a PR with this13:43
mvodidrocks: sounds good to me, I will see if I can enable GH actions to run the tests by default13:44
didrocksmvo: great! zsys has some example of GH actions with go if you need them13:45
didrocksok. let’s close mvo's bug #1 :p14:06
ubot5bug 1 in Ubuntu Malaysia LoCo Team "Microsoft has a majority market share" [Critical,In progress] https://launchpad.net/bugs/114:06
didrocks(your tests are failing btw, segfault, we don’t touch that, or the Go style for now (underscore in variable names!!! no comment on public method!!!) :))14:07
mvodidrocks: just pushed https://github.com/mvo5/libsmbclient-go/pull/6 that should fix this14:15
mvodidrocks: give it time, this was dormant for 7y ;)14:15
didrocksahah :)14:15
jibelmvo, https://github.com/mvo5/libsmbclient-go/pull/7 14:15
didrocksit never had that much PR in one day :p14:15
* mvo hugs jibel and didrocks 14:15
didrockshappy that we got it working14:15
* didrocks hugs mvo back14:16
mvodidrocks: it never had any PR!14:16
didrocksthanks for bootstrapping that work, it saves us time14:16
jibelYay we're top contributors :)14:16
mvodidrocks, jibel no more eexists error in the testsuite after your PR! 14:17
jibelheh, glad to resurrect a project and fix the testsuite after 7y14:21
oSoMoNseb128, you've probably seen it in #debian-gnome/trello, but just in case: the tepl, gedit and gedit-plugins updates are ready in salsa14:36
seb128oSoMoN, thanks!14:37
hellsworthgood morning desktopers14:47
oSoMoNhey hellsworth 14:47
hellsworthhi there oSoMoN 14:48
hellsworthricotz: oh dear. thanks for the heads up14:50
oSoMoNseb128, the trello card for tracker-miners says 2.3.5, but there's a 3.0 release for both tracker and tracker-miners, is there a reason not to go to 3.0 ?15:34
oSoMoNseb128, pango 1.46.2 is already in Debian unstable, so we can sync after the beta freeze is over15:58
oSoMoNthere's a 1.47.0 upstream release, do we want that?15:58
oSoMoNit appears to mostly add new APIs (https://download.gnome.org/sources/pango/1.47/pango-1.47.0.news)15:59
seb128oSoMoN, not a new serie this cycle no? unless they changed versionning16:15
seb128tracker we said we would avoid the new 3 serie this cycle16:15
seb128unsure if miner needs to match the service16:15
oSoMoNyeah, nevermind that stupid question about tracker 3, I remember we agreed not to go for that new series16:41

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