Review Request: Rename accounts-kcm according to the new policy

Martin Klapetek martin.klapetek at gmail.com
Thu Dec 22 10:44:55 UTC 2011


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


Apparently I have some forgotten stuff in my working tree, so please ignore the code changes, that's to be submitted separately. 

Question - what should be in Messages.sh? 

- Martin Klapetek


On Dec. 22, 2011, 10:36 a.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103502/
> -----------------------------------------------------------
> 
> (Updated Dec. 22, 2011, 10:36 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> This has actually lots of libraries, so I named the main kcm one according to the kcm scheme, its internal library is now 'libktpaccountskcminternal', while the plugins are 'ktpaccountskcm_plugin_X.so', which I'm not sure about. Should I turn it to 'libktpaccountskcm_plugin_X.so' or 'libktpaccountskcmpluginX.so'?
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 2a9aef2 
>   plugins/butterfly/CMakeLists.txt 26859c9 
>   plugins/butterfly/ktpaccountskcm_plugin_butterfly.desktop 3df354a 
>   plugins/gabble/CMakeLists.txt 81ba67d 
>   plugins/gabble/ktpaccountskcm_plugin_gabble.desktop 62c501c 
>   plugins/haze/CMakeLists.txt dfedda5 
>   plugins/haze/ktpaccountskcm_plugin_haze.desktop f044014 
>   plugins/idle/CMakeLists.txt 3b91531 
>   plugins/idle/ktpaccountskcm_plugin_idle.desktop adf8078 
>   plugins/rakia/CMakeLists.txt bfa8da7 
>   plugins/rakia/ktpaccountskcm_plugin_rakia.desktop 906f1d6 
>   plugins/salut/CMakeLists.txt 80da74f 
>   plugins/salut/ktpaccountskcm_plugin_salut.desktop 98ea8e3 
>   plugins/sunshine/CMakeLists.txt 0a08d7a 
>   plugins/sunshine/ktpaccountskcm_plugin_sunshine.desktop aceac75 
>   src/CMakeLists.txt 60542b2 
>   src/KCMTelepathyAccounts/CMakeLists.txt 581edcf 
>   src/account-item.h 8a7bde0 
>   src/accounts-list-delegate.cpp f670e0f 
>   src/accounts-list-model.h 1101172 
>   src/accounts-list-model.cpp b1a6e05 
>   src/kcm-telepathy-accounts.cpp 55d2853 
>   src/kcm_telepathy_accounts.desktop cd20e22 
> 
> Diff: http://git.reviewboard.kde.org/r/103502/diff/diff
> 
> 
> Testing
> -------
> 
> Successfully created an account. Everything works.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111222/936f9c77/attachment.html>


More information about the KDE-Telepathy mailing list