| Summary: | discover doesn't recognize a fixed broken package problem | ||
|---|---|---|---|
| Product: | [Applications] Discover | Reporter: | Martin Tlustos <martin.tlustos> |
| Component: | discover | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | aleixpol, martin.tlustos, nate |
| Priority: | NOR | ||
| Version First Reported In: | 5.27.3 | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/discover/commit/18a85033c4f79157693b45247c7a19a6a648ec32 | Version Fixed/Implemented In: | 5.27.4 |
| Sentry Crash Report: | |||
|
Description
Martin Tlustos
2023-03-21 06:10:08 UTC
Additional info: I had disabled offline updates. After re-enabling them, applying the next available updates and restarting, the error message did not show up again. So it seems like there was some error log left behind that only gets cleared in offline mode, but when you switch to the old way of doing updates, it will keep on popping up. It works for me now, but I don't mark it as fixed so someone can have a look at whether this behavior should be improved. It would have needed to remove the "/var/lib/PackageKit/offline-update-competed" file. This is something that should be done by PackageKit so at least it's a bug there upstream (on two counts: failing to cancel properly and then not clearing the state as requested). Do you remember if you pressed "Repair System" or you closed the notification? A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/518 (In reply to Aleix Pol from comment #3) > It would have needed to remove the > "/var/lib/PackageKit/offline-update-competed" file. > > This is something that should be done by PackageKit so at least it's a bug > there upstream (on two counts: failing to cancel properly and then not > clearing the state as requested). > > Do you remember if you pressed "Repair System" or you closed the > notification? Yes, I clicked both "repair system" and "open discover" and closed the notification after several startups. Git commit cbecc135d99b535a67f9a0d9a1b8b484e4c47e9a by Aleix Pol. Committed on 31/03/2023 at 14:53. Pushed by apol into branch 'master'. pk: Clear the offline state before starting a new one If there's results left-over from the past, just clear them and hope for better luck next time. M +6 -0 libdiscover/backends/PackageKitBackend/PackageKitUpdater.cpp https://invent.kde.org/plasma/discover/commit/cbecc135d99b535a67f9a0d9a1b8b484e4c47e9a Git commit 18a85033c4f79157693b45247c7a19a6a648ec32 by Aleix Pol Gonzalez, on behalf of Aleix Pol. Committed on 31/03/2023 at 15:46. Pushed by apol into branch 'Plasma/5.27'. pk: Clear the offline state before starting a new one If there's results left-over from the past, just clear them and hope for better luck next time. (cherry picked from commit cbecc135d99b535a67f9a0d9a1b8b484e4c47e9a) M +6 -0 libdiscover/backends/PackageKitBackend/PackageKitUpdater.cpp https://invent.kde.org/plasma/discover/commit/18a85033c4f79157693b45247c7a19a6a648ec32 Can we also clear it after the user clicks the "repair" button in the notification? We have some bug reports about this not working. A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/527 Git commit e87a50f455619dea5c51dea9c290556739112fc7 by Nate Graham. Committed on 11/04/2023 at 20:53. Pushed by ngraham into branch 'master'. Always clear offline updates file after a repair operation No matter what the results of the repair operation were, there's nothing of any value that anyone can do with the file anymore, and if it sticks around, it can annoy the user on every subsequent login for no good reason. So let's just remove it. Related: bug 451753, bug 450973 FIXED-IN: 5.27.5 M +7 -0 libdiscover/backends/PackageKitBackend/PackageKitNotifier.cpp https://invent.kde.org/plasma/discover/commit/e87a50f455619dea5c51dea9c290556739112fc7 Git commit b18ebb137e49ccde3be49239bfaa8d88ace6a0c4 by Nate Graham. Committed on 11/04/2023 at 20:56. Pushed by ngraham into branch 'Plasma/5.27'. Always clear offline updates file after a repair operation No matter what the results of the repair operation were, there's nothing of any value that anyone can do with the file anymore, and if it sticks around, it can annoy the user on every subsequent login for no good reason. So let's just remove it. Related: bug 451753, bug 450973 FIXED-IN: 5.27.5 (cherry picked from commit e87a50f455619dea5c51dea9c290556739112fc7) M +7 -0 libdiscover/backends/PackageKitBackend/PackageKitNotifier.cpp https://invent.kde.org/plasma/discover/commit/b18ebb137e49ccde3be49239bfaa8d88ace6a0c4 |