[patch] merging/importing kwallets

[Friedrich W. H. Kossebau]

Hi Michael,

Am Mittwoch, 30. April 2008, um 01:27 Uhr, schrieb Michael Leupold:
> Hi,
>
> this patch aims at fixing http://bugs.kde.org/show_bug.cgi?id=132808. The
> problem arises basically because kwallet only allows to specifically query
> for maps and passwords. You get stream and unused entries using a generic
> query that returns all data (without being able to figure out the type
> after retrieval). I made kwalleteditor.cpp remember which entries it
> already merged. Maybe the patch could be made even smaller by only checking
> for mergedkeys in the final loop but I wasn't quite sure.

Sorry, cannot comment on your patches, have no clue of KWallet and can not 
reserve enough time to learn more about it.

> IMO the "right thing" to do would be to add functions for streams and
> unused (whatever that's supposed to be used for) entries as well. Wouldn't
> even break the api for existing apps using the interface :) Is kwalletd
> managed by the kdelibs maintainers?

Currently kind of, yes. At least some of them have a bit deeper knowledge 
about it.

If there are too few reactions on this list as noone is yet experienced enough 
with KWallet code you might also try to send your patches to kde-core-devel 
for now, please, so your good looking work does not stay mostly unanswered.

Cheers
Friedrich