| === kveremitz is now known as skeerymitz | ||
| === skeerymitz is now known as mitznite | ||
| === liushuyu1 is now known as liushuyu | ||
| ghosteau | Hello. I found a bug in gdebi. This bug causes a Python warning in the latest version (Python 3.12), and in newer versions it'll cause an error. Where should I report this bug and a possible patch? | 14:20 |
|---|---|---|
| ghosteau | Bug: https://git.launchpad.net/gdebi/tree/gdebi#n113 the regex with \S between regular quotes "..." is not correct in Python and triggers a warning; in future Python versions it'll trigger an error. Should be r"..." or \\S | 14:25 |
| ghosteau | (issue also found on another file I can't remember right now | 14:25 |
| ghosteau | ah, GdebiCli.py | 14:29 |
| ghosteau | Anyway. Can't find where to report bugs on Gdebi. https://launchpad.net/gdebi says it doesn't know. | 14:39 |
| cjwatson | ghosteau: Already reported as https://bugs.launchpad.net/ubuntu/+source/gdebi/+bug/2060431 | 14:46 |
| -ubottu:#ubuntu-devel- Launchpad bug 2060431 in gdebi (Ubuntu) "Python 3.12 regex syntax warning message during gdebi-core installation" [Undecided, Confirmed] | 14:46 | |
| cjwatson | (I don't think there's currently a timeline for turning that into SyntaxError, FWIW) | 14:49 |
| ghosteau | cjwatson: ah, thanks :) | 14:59 |
| ghosteau | https://docs.python.org/3/whatsnew/3.12.html#other-language-changes A backslash-character pair that is not a valid escape sequence now generates a SyntaxWarning [...] In a future Python version, SyntaxError will eventually be raised | 15:01 |
| ghosteau | So no timeline but definitely a stated intent to do it | 15:01 |
| ghosteau | These sort of fixes tend to take long to propagate so the earlier they're fixed the better, IMO. | 15:02 |
| ghosteau | By the way, gdebi didn't work at all when I first tried it in this fresh Xubuntu install, at least graphically. Had to run it from command line. But last time I tried it it seemed to work though. | 15:03 |
| ghosteau | I don't know if whatever was going on was fixed or if it is related to my fix. | 15:04 |
| ghosteau | Is there a way to fast-track the fix? I can submit a patch or pull request or whatever is common practice in launchpad | 15:14 |
| ghosteau | Is it considered impolite to subscribe someone else (developer) to a bug report? I just added a couple of gdebi devs to it | 15:43 |
| ghosteau | (one of them last seen on libera 86 weeks ago; the other one apparently active on another IRC network called... Freenode?) | 15:46 |
| ghosteau | make that two developers. (are many devs actually active in Freenode, or did they all just forget to update their IRC contact info after "the event"? Or is it safe to assume that these devs are not active any longer?) | 15:52 |
| ghosteau | In any case, last commit in gdebi is from 2015. I'm not hopeful it'll be maintained. | 15:55 |
| cjwatson | It's probably not very maintained given that it mostly isn't particularly needed any more, now that you can just do "apt install ./foo.deb" | 16:01 |
| cjwatson | Though https://tracker.debian.org/pkg/gdebi does show somebody looking after it in Debian; might be more productive to file a Debian bug | 16:03 |
| ghosteau | ...you can?? | 16:04 |
| cjwatson | Oh, actually, that looks like it's been fixed: https://bugs.debian.org/1076721 | 16:04 |
| -ubottu:#ubuntu-devel- Debian bug 1076721 in gdebi-core "gdebi gives a SyntaxWarning invalid escape sequence when run" [Minor, Fixed] | 16:04 | |
| ghosteau | damn apt, simplifying all the things. Kids today have it easy. Back in MY day... | 16:04 |
| cjwatson | So should be fixed as of 24.10 | 16:05 |
| ghosteau | why not 24.04? Backport! | 16:05 |
| cjwatson | Not my problem :) | 16:05 |
| ghosteau | Also, it is not fixed as per the git repo | 16:05 |
| cjwatson | Sure, the bzr repo is unmaintained it seems - Debian is maintaining it by NMUs | 16:05 |
| cjwatson | Also, there's little point fixing it in 24.04 given that a newer system Python version that turns that into a syntax error is certainly not going to ever make it into 24.04 | 16:06 |
| cjwatson | Fixing it in newer Ubuntu releases is fine for this kind of thing | 16:06 |
| ghosteau | It's a git repo, not bazaar | 16:06 |
| ghosteau | (the one I saw) | 16:06 |
| ghosteau | OK, maybe it makes sense that it's not urgent to fix, but it's still annoying to see the error | 16:07 |
| cjwatson | Yeah, but "annoying" isn't enough for a stable update | 16:07 |
| ghosteau | There's another issue with gdebi (apparently unrelated) that doesn't allow me to use it graphically -- it says "Refusing to render service to dead parents" | 16:08 |
| cjwatson | Oh yeah, I was going off the outdated metadata in the package - the git repo is much the same | 16:08 |
| cjwatson | ghosteau: https://bugs.launchpad.net/ubuntu/+source/gdebi/+bug/1854588 | 16:09 |
| -ubottu:#ubuntu-devel- Launchpad bug 1854588 in gdebi (Ubuntu Noble) "[SRU] gdebi-gtk calls pkexec inappropriately" [Undecided, Confirmed] | 16:09 | |
| cjwatson | Dunno if anyone's chasing that for noble; it does sound more SRU-worthy | 16:10 |
| ghosteau | OK so I've apparently followed your same steps. Went to Debian, found the link to the source, it was a bazaar repo, last change one year ago, linking it to the git repo, last updated 9 years ago. I'm confused, but yeah I now get what you meant. | 16:12 |
| ghosteau | Btw, gdebi works fine for upgrading packages. It's just installing from scratch which fails. Or at least it works fine for upgrading discord. | 16:13 |
| cjwatson | I think this is just another one of those things that was built in Ubuntu many years ago when it was useful, pretty much superseded by developments elsewhere, but kind of shambles on as a zombie | 16:13 |
| ghosteau | yeah, that's why I volunteered to submit the patch; I assumed no one else would need it :) | 16:13 |
| cjwatson | And the original developers have long since moved on, but somebody's dealing with it in Debian | 16:14 |
| ghosteau | but I mean, if double-clicking a file uses gdebi by default (at least on xubuntu)... | 16:14 |
| cjwatson | Yeah, I'd kind of argue that the flavours that use gdebi should maybe look into not doing that any more | 16:14 |
| ghosteau | OK this is super weird because that file has indeed changed in 24.10 source but I can't find it hosted anywhere. Git, bazaar... | 16:17 |
| ghosteau | also the regex changed. They went from [[(] to a plain \[. Only brackets, no parentheses. But I guess that's OK? | 16:18 |
| ghosteau | Ooooh, I guess that's what they meant by "dead parens"? :D | 16:18 |
| cjwatson | I presume so | 16:19 |
| cjwatson | You can get the code from the source package in the archive | 16:20 |
| cjwatson | Or from https://code.launchpad.net/ubuntu/+source/gdebi which is automatically imported | 16:20 |
| ghosteau | But it's no longer hosted on a publicly available git/bazaar repo? | 16:21 |
| cjwatson | I just gave you one | 16:21 |
| ghosteau | Oh, that's a(nother) git repo | 16:22 |
| cjwatson | I wouldn't say it's exactly maintained there - I think people dealing with it in Debian are just doing so by way of source package uploads - but you can consume it from there | 16:22 |
| ghosteau | I made the mistake to look into the git tree. It's beautiful, I'll give you that. | 16:24 |
| ghosteau | Seems to be a completely different repo though. (Different git history) | 16:26 |
| ghosteau | Not a "development git" but rather a "we use git as a way to dump files and apply patches, but this is pretty much an automated repo". | 16:27 |
| ghosteau | But well, it's something. Thanks :) | 16:27 |
| ghosteau | cjwatson: so... will bug 1854588 be fixed and backported to LTS? | 23:26 |
| -ubottu:#ubuntu-devel- Bug 1854588 in gdebi (Ubuntu Noble) "[SRU] gdebi-gtk calls pkexec inappropriately" [Undecided, Confirmed] https://launchpad.net/bugs/1854588 | 23:26 | |
| ghosteau | says "Fix released" for 24.10, which I suppose means it works in that version if it's up to date | 23:27 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!