* Martin Klapetek <[email protected]> [2015-08-12 12:25:27 -0400]:
> On Tue, Aug 11, 2015 at 7:30 AM, Jonathan Riddell <[email protected]> wrote: > > > On Tue, Aug 11, 2015 at 01:18:20PM +0200, Valentin Rusu wrote: > > > * Jonathan Riddell <[email protected]> [2015-08-05 18:11:56 +0200]: > > > > > > > some changes to pkg-kde git today: > > > > > > > > I added kwallet-pam to plasma > > > > > > > > > > Why not include the kwallet-pam module in the kwallet framework? The > > > src/runtime would be the perfect place for that module and users would > > > have all the required pieces together. If nobody is against this merging > > > - and to be sure I copy kde-frameworks-devel - then it's a matter of > > > importing the kwallet-map repo into a directory under > > > kwallet/src/runtime. I could help with that. > > > > This is a change in Plasma release tars (rather than Kubuntu > > packaging) and it was added there because Martin K said it was ported > > and should be released, and I think his motivation was just that > > enough people bugged him that he made Alex's work compile. But nobody > > is around who especially wants to maintain it so if you want to move > > it into wallet framework that would be very welcome. > > > > What Jonathan said. > > +1 from me to move it. Though keep in mind that this is for kwallet4 as well > and so it needs to be compilable for kwallet4 too, somehow. Which might > make the KWallet(5) CMake foo a bit too complicated. So, actually, it might > be > better off on its own as long as kwallet4 is supported. Oh, I'm against mixing code for kwallet4 in KF5::Wallet. So let it live in its own repo until we'll stop the 4 version. Cheers, -- Valentin Rusu IRC: valir -- kubuntu-devel mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
