> On April 1, 2017, 7:59 p.m., David Faure wrote: > > Makes sense to me, +1. > > Andreas Sturmlechner wrote: > Thanks, anyone else who wants to +1? > > I've tried to test migration today but it didn't work. May as well have > nothing to do with te patch and be caused by the permanently troubled > migration agent though... KMail happily gets its IMAP password from kwallet5 > though after manually export/import via XML files. > > Andreas Sturmlechner wrote: > As suspected, on my test system migration is broken regardless of with > these patches or not. > > David Faure wrote: > Are you planning on looking into that? ;-) > > These patches are related to migration, it feels a bit wrong to commit > changes around migration and still leave it broken. > > Andreas Sturmlechner wrote: > I don't feel like I'm up to that task. Also, the reason for why it works > for some, but not all the systems, has afaik never been established. > > Andreas Sturmlechner wrote: > Could we push it to give it some testing by others until the upcoming > release? > > René J.V. Bertin wrote: > Was this tested, in the end? > > I only became aware of the change after I installed 5.35.0 on my > FrankenStux box with a Plasma4 desktop and KF5 installed into /opt/local, and > I had to re-grant access to my wallets to all applications accessing them. > > I cannot yet vouch that my wallets were migrated completely. I think they > were, but how do I verify that now? Even the KDE4 kwalletmanager goes through > kwalletd5. It too had to be granted access. > > Confusingly, it still shows the old access list, which clearly it > shouldn't if the information is stale.
If you see your old wallets now in kwalletd5, then migration has worked. I've never heard about issues with partial migration; if it does not work, it fails to migrate completely. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130058/#review102982 ----------------------------------------------------------- On May 27, 2017, 2:32 p.m., Andreas Sturmlechner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/130058/ > ----------------------------------------------------------- > > (Updated May 27, 2017, 2:32 p.m.) > > > Review request for KDE Frameworks and Stefan Brüns. > > > Repository: kwallet > > > Description > ------- > > These are not my own patches, I'm creating this review request after having > been made aware of *kwalletd4_dbus_compat* branch in kwallet.git, which I > simply rebased on top of current master (author of course preserved) to be > able to test it. I think it would be a great improvement over the current > situation that is rather confusing to the users. > > The changes are organised in 5 commits: > > - Check for unique applicaton instance as early as possible > Exit before KWalletD and the MigrationAgent has been initialized. > The return value is changed, but concurrent instatiation of kwalletd is > not a fault. > > - Only start timer for migration agent if necessary > - Whitespace fixup > - Signal completion of migration agent > - Replace kwalletd4 after migration has finished > kwalletd5 can service both org.kde.kwalletd5 and org.kde.kwalletd > > > Diffs > ----- > > src/runtime/kwalletd/kwalletd.h 3571535cd8bd78415002795f9b61adf9f6cfb8c1 > src/runtime/kwalletd/kwalletd.cpp 18ef9fa7e6ddaeba6e0b32deae3de1dae39df5bb > src/runtime/kwalletd/main.cpp ff9620886fa1959e14b594be6bbb4644b637c000 > src/runtime/kwalletd/migrationagent.h > 0f6467c1753ef34b7f7f7e282503ec5607927db9 > src/runtime/kwalletd/migrationagent.cpp > f3da94743ecd83fe406e058f560d4238caec1be8 > > Diff: https://git.reviewboard.kde.org/r/130058/diff/ > > > Testing > ------- > > Migration itself was not tested so far, but a legacy application like ksirk > was able to create a new wallet just fine and can access it as well. I do not > have kwalletd4 installed anymore. > > > Thanks, > > Andreas Sturmlechner > >
