<div dir="ltr">On Tue, Aug 11, 2015 at 7:30 AM, Jonathan Riddell <span dir="ltr"><<a href="mailto:jr@jriddell.org" target="_blank">jr@jriddell.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Aug 11, 2015 at 01:18:20PM +0200, Valentin Rusu wrote:<br>
> * Jonathan Riddell <<a href="mailto:jr@jriddell.org">jr@jriddell.org</a>> [2015-08-05 18:11:56 +0200]:<br>
><br>
> > some changes to pkg-kde git today:<br>
> ><br>
> > I added kwallet-pam to plasma<br>
> ><br>
><br>
> Why not include the kwallet-pam module in the kwallet framework? The<br>
> src/runtime would be the perfect place for that module and users would<br>
> have all the required pieces together. If nobody is against this merging<br>
> - and to be sure I copy kde-frameworks-devel - then it's a matter of<br>
> importing the kwallet-map repo into a directory under<br>
> kwallet/src/runtime. I could help with that.<br>
<br>
</span>This is a change in Plasma release tars (rather than Kubuntu<br>
packaging) and it was added there because Martin K said it was ported<br>
and should be released, and I think his motivation was just that<br>
enough people bugged him that he made Alex's work compile.  But nobody<br>
is around who especially wants to maintain it so if you want to move<br>
it into wallet framework that would be very welcome.<br></blockquote><div><br></div><div>What Jonathan said.</div><div><br></div><div>+1 from me to move it. Though keep in mind that this is for kwallet4 as well</div><div>and so it needs to be compilable for kwallet4 too, somehow. Which might</div><div>make the KWallet(5) CMake foo a bit too complicated. So, actually, it might be</div><div>better off on its own as long as kwallet4 is supported. Or move a copy together</div><div>with kwallet4 (the code has quite some ifdefs so wouldn't be an exact copy).</div><div><br></div></div><div>Cheers</div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div>