| Summary: | Crash on every download attempt | ||
|---|---|---|---|
| Product: | [Applications] kasts | Reporter: | lencho |
| Component: | general | Assignee: | bart |
| Status: | RESOLVED WAITINGFORINFO | ||
| Severity: | crash | ||
| Priority: | NOR | ||
| Version First Reported In: | 24.12.1 | ||
| Target Milestone: | --- | ||
| Platform: | Manjaro | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/multimedia/kasts/-/commit/42987516e9efdbed2f2a2dbb3800a6e6c0d312eb | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
lencho
2025-02-09 16:03:13 UTC
We will need more information to debug this. Could you share the podcast and episode that is failing? If you don't like to share that info publicly, then you can also email it directly. Well, it is every episode of every podcast I try! But I think I got it, it fails to parse the path of the file because it contains an accentuated letter. If I save the podcasts to /home/myuser it's fine! Note that is broke since last update, because it used to work fine in the same path. I noticed already Kasts is a bit cautious/lazy (sorry!) and just removes accentuated letters from the file names, but now it fails with the path because it cannot do anything about it. I still don't understand where the special character is. Is it in the path that you've pointed it to to save episodes? Is it in the name of the podcast, or in the name of the episode? Or somewhere else? Nothing has changed to that part of the code since many versions, so if it didn't happen before, then maybe it's in some dependency lib. Yes, in the path where to save the episode, sorry if I was not clear. If it's not in Kasts, do you know where it could come from? Thanks, Could you perhaps mention which character is causing the issue? In the meantime, I have an idea about a potential cause for the crash. Especially given that you mentioned that it only popped up since the last version. Confirmed with an accented character. I see what's happening here: the taglib library was not getting the correct path if there were special characters involved. The fix is fairly easy. It should be fixed in the next release. A possibly relevant merge request was started @ https://invent.kde.org/multimedia/kasts/-/merge_requests/242 Git commit 43f0e8c90188c5e97104eb5dea7d7f7dbf49a504 by Bart De Vries. Committed on 10/02/2025 at 14:02. Pushed by bdevries into branch 'master'. Fix path being passed to taglib and avoid opening non-existent files M +9 -7 src/enclosure.cpp M +7 -5 src/models/chaptermodel.cpp M +1 -1 src/models/chaptermodel.h https://invent.kde.org/multimedia/kasts/-/commit/43f0e8c90188c5e97104eb5dea7d7f7dbf49a504 Git commit 42987516e9efdbed2f2a2dbb3800a6e6c0d312eb by Bart De Vries. Committed on 10/02/2025 at 14:12. Pushed by bdevries into branch 'release/24.12'. Fix path being passed to taglib and avoid opening non-existent files M +9 -7 src/enclosure.cpp M +7 -5 src/models/chaptermodel.cpp M +1 -1 src/models/chaptermodel.h https://invent.kde.org/multimedia/kasts/-/commit/42987516e9efdbed2f2a2dbb3800a6e6c0d312eb Great, thank you for your promptness, Bart! |