Review Request 110426: KWalletHelper class for services using the KWallet

[Vedant Agarwala]

> On Aug. 25, 2013, 8:47 a.m., Mark Kretschmann wrote:
> > Any news for this patch?

I am working on it. Likely to be done in a couple of days


- Vedant


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110426/#review38518
-----------------------------------------------------------


On May 20, 2013, 1:36 p.m., Vedant Agarwala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110426/
> -----------------------------------------------------------
> 
> (Updated May 20, 2013, 1:36 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> I have created a KWalletHelper class so that services like Maganatune, Last.fm and GPodder can use this rather than duplicating code.
> Currently the patch applies only to Magnatune. The KWalletHelper class complies but it doesn't link properly to the MagnatuneConfig class.
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt 4dcb316 
>   src/services/KWalletHelper.h PRE-CREATION 
>   src/services/KWalletHelper.cpp PRE-CREATION 
>   src/services/gpodder/GpodderServiceConfig.h 90447fd 
>   src/services/gpodder/GpodderServiceConfig.cpp 9098d59 
>   src/services/gpodder/GpodderServiceSettings.cpp 34280c7 
>   src/services/lastfm/CMakeLists.txt a895bba 
>   src/services/lastfm/LastFmServiceConfig.h 4b1552e 
>   src/services/lastfm/LastFmServiceConfig.cpp 3c8b550 
>   src/services/magnatune/CMakeLists.txt 91f24c0 
>   src/services/magnatune/MagnatuneConfig.h 552bcf8 
>   src/services/magnatune/MagnatuneConfig.cpp 5842c63 
>   src/services/magnatune/MagnatuneDownloadHandler.h b257440 
>   src/services/magnatune/MagnatuneDownloadHandler.cpp 3bce597 
>   src/services/magnatune/MagnatuneInfoParser.h 7904b67 
>   src/services/magnatune/MagnatuneInfoParser.cpp 1888ced 
>   src/services/magnatune/MagnatuneMeta.cpp 13b41d6 
>   src/services/magnatune/MagnatuneNeedUpdateWidget.cpp 044cf4b 
>   src/services/magnatune/MagnatuneRedownloadHandler.cpp 99c1a54 
>   src/services/magnatune/MagnatuneSettingsModule.h 4728a34 
>   src/services/magnatune/MagnatuneSettingsModule.cpp d45938f 
>   src/services/magnatune/MagnatuneStore.h de95057 
>   src/services/magnatune/MagnatuneStore.cpp 359990e 
> 
> Diff: http://git.reviewboard.kde.org/r/110426/diff/
> 
> 
> Testing
> -------
> 
> The KWalletHelper.cpp complies but fails to link to ManatuneConfig.cpp. Output of "make" command: http://paste.kde.org/743792/
> 
> 
> Thanks,
> 
> Vedant Agarwala
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130826/ee2f6589/attachment.html>