Summary: | kwalletd legacy dbus service file | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-kwallet | Reporter: | Damjan Georgievski <gdamjan> |
Component: | general | Assignee: | Valentin Rusu <valir> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kdelibs-bugs, rdieter |
Priority: | NOR | ||
Version: | 5.51.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
See Also: |
https://bugs.kde.org/show_bug.cgi?id=410999 https://bugs.kde.org/show_bug.cgi?id=414826 |
||
Latest Commit: | Version Fixed In: |
Description
Damjan Georgievski
2018-10-29 22:23:55 UTC
patch available at https://phabricator.kde.org/D16520 Git commit dc5c7229bc4ff75506c34197da5450808c969efe by Stefan Brüns. Committed on 17/07/2019 at 22:33. Pushed by bruns into branch 'master'. Remove kde4 migration agent completely Summary: The migration agent is some quite weird mechanism: - it starts the KDE4 kwalletd - to do this, it asks DBus to start it - as the old kwalletd never shipped a service file, kwalletd5 ships a service file for kwalletd(4), hardcoding its likely path. - it merges the old wallets via DBus requests When the migration has finished, a flag is set in the config and on the next start, the migration agent is skipped. When the migration fails (e.g. because kwalletd(4) is not installed), the migration is attempted on each start. Shipping a a service file pointing to the old daemon also breaks autostart of kwalletd5 for old applications - kwalletd5 provides the kwalletd interface, but only if it has been started already. This leads to a race during startup. See D16520 - proper kwalletd dbus service file for the legacy name Reviewers: #frameworks, cfeck, ngraham, aacid Reviewed By: aacid Subscribers: aacid, lbeltrame, kde-frameworks-devel, damjang Tags: #frameworks Differential Revision: https://phabricator.kde.org/D21002 M +0 -10 src/runtime/kwalletd/CMakeLists.txt M +0 -1 src/runtime/kwalletd/kwalletd.h M +1 -4 src/runtime/kwalletd/main.cpp D +0 -265 src/runtime/kwalletd/migrationagent.cpp D +0 -59 src/runtime/kwalletd/migrationagent.h D +0 -108 src/runtime/kwalletd/migrationwizard.cpp D +0 -44 src/runtime/kwalletd/migrationwizard.h D +0 -122 src/runtime/kwalletd/migrationwizard1.ui D +0 -51 src/runtime/kwalletd/migrationwizard2.ui D +0 -6 src/runtime/kwalletd/org.kde.kwalletd.service https://commits.kde.org/kwallet/dc5c7229bc4ff75506c34197da5450808c969efe I just noticed - the hard way - that KDE Frameworks 5.61 does NOT provide the org.kde.kwalletd interface any more, at if the KDE Neon packages are used. This breaks existing software like Chromium. Could this be related to this issue here? It's the only possibly related change I could find in the changelog. I opened a new issue #410999 for this problem with KDE Frameworks 5.61. |