Review Request 125069: [patch] Port dolphin-plugins to KF5 without kdelibs4support

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sat Sep 26 11:59:40 BST 2015


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


Thanks for the patch!

Can you please fix all issues (and mark them as fixed) which were raised by Mark and rebase your patch, because Harald Sitter pushed some changes to port plugins away from kvbox.

- Emmanuel Pescosta


On Sept. 16, 2015, 10:05 p.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125069/
> -----------------------------------------------------------
> 
> (Updated Sept. 16, 2015, 10:05 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Repository: dolphin-plugins
> 
> 
> Description
> -------
> 
> Remove dependecy of kdelibs4support
> 
> 
> Diffs
> -----
> 
>   svn/fileviewsvnplugin.cpp faf6a0f 
>   svn/fileviewsvnplugin.h 2ba36bf 
>   svn/CMakeLists.txt 1627b92 
>   git/tagdialog.h 60563cd 
>   git/tagdialog.cpp 5da01ad 
>   git/pushdialog.cpp 1c82fa3 
>   git/pulldialog.cpp d51789e 
>   git/pushdialog.h 74da471 
>   git/pulldialog.h 861cb80 
>   git/fileviewgitplugin.h 3148b12 
>   git/fileviewgitplugin.cpp 7af37f0 
>   git/commitdialog.cpp 06222cf 
>   git/commitdialog.h dd5bfbf 
>   git/checkoutdialog.h 6d24c7d 
>   git/checkoutdialog.cpp 1eaecef 
>   git/CMakeLists.txt ea58c77 
>   CMakeLists.txt 76314bf 
> 
> Diff: https://git.reviewboard.kde.org/r/125069/diff/
> 
> 
> Testing
> -------
> 
> 1. KDialog to QDialog + QDialogButtonBox using convert-kdialog.pl
> 2. KLineEdit to QLineEdit
> 3. KTextEdit to QTextEdit
> 4. KVBox, KHBox to QWidget + QVBoxLayout, QWidget + QHBoxLayout using convert-kvbox.pl
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20150926/401bb85c/attachment.htm>


More information about the kfm-devel mailing list