Review Request 117954: Port keditfiletype away from kde4support

[Aurélien Gâteau]

> On May 20, 2014, 5:08 p.m., Alexander Potashev wrote:
> > You probably need to check in CMakeLists.txt that version of shared-mime-info is at least 0.40.

shared-mime-info is only required at run time so I am not sure it makes sense to check at build time. Furthermore, 0.40 was released in June 2008, so I think it is safe to assume the installed version is at least 0.40.


- Aurélien


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117954/#review58199
-----------------------------------------------------------


On May 20, 2014, 4:56 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117954/
> -----------------------------------------------------------
> 
> (Updated May 20, 2014, 4:56 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kde-cli-tools
> 
> 
> Description
> -------
> 
> This patch contains multiple commits, which I can make available if needed. It ports keditfiletype away from kde4support, which helps identifying its dependencies so that it can hopefully be moved closer to the classes which need it in frameworks (right now kwidgetsaddons and kio forks it)
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt d62ee8b 
>   keditfiletype/CMakeLists.txt 566e76a 
>   keditfiletype/filegroupdetails.cpp c733ba4 
>   keditfiletype/filetypedetails.h 7c0abb4 
>   keditfiletype/filetypedetails.cpp bfcbb07 
>   keditfiletype/filetypesview.h 82fd0ab 
>   keditfiletype/filetypesview.cpp 886892f 
>   keditfiletype/keditfiletype.h 0b235ce 
>   keditfiletype/keditfiletype.cpp aab0830 
>   keditfiletype/kservicelistwidget.h c817d24 
>   keditfiletype/kservicelistwidget.cpp 88dface 
>   keditfiletype/kserviceselectdlg.h 8e6d0d0 
>   keditfiletype/kserviceselectdlg.cpp 09c7c5d 
>   keditfiletype/mimetypedata.h 042b043 
>   keditfiletype/mimetypedata.cpp 057113f 
>   keditfiletype/mimetypewriter.h 5a618a6 
>   keditfiletype/mimetypewriter.cpp be5e5d4 
>   keditfiletype/newtypedlg.h fba8385 
>   keditfiletype/newtypedlg.cpp c933a8f 
>   keditfiletype/sharedmimeinfoversion.h 9824d7b 
>   keditfiletype/sharedmimeinfoversion.cpp 9dabf0e 
>   keditfiletype/tests/CMakeLists.txt 489bb4d 
>   keditfiletype/tests/filetypestest.cpp 68eef26 
>   keditfiletype/typeslistitem.h 05e425a 
>   keditfiletype/typeslistitem.cpp 97cd3c1 
> 
> Diff: https://git.reviewboard.kde.org/r/117954/diff/
> 
> 
> Testing
> -------
> 
> Program starts and looks like it behaves correctly. Unit-test is failing, but was already failing before :/
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140522/88302df9/attachment-0001.html>