Review Request 125247: Add missing KFileItem includes in dolphin-plugins
Frank Reininghaus
frank78ac at googlemail.com
Sat Sep 26 07:19:58 BST 2015
> On Sept. 25, 2015, 7:31 nachm., Emmanuel Pescosta wrote:
> > Ship It!
Was fixed already in https://quickgit.kde.org/?p=dolphin-plugins.git&a=commit&h=9ceea02f9d1903e260a1d45d1ca0330d6c2d47fe
- Frank
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125247/#review85935
-----------------------------------------------------------
On Sept. 15, 2015, 8:21 nachm., Frank Reininghaus wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125247/
> -----------------------------------------------------------
>
> (Updated Sept. 15, 2015, 8:21 nachm.)
>
>
> Review request for Dolphin.
>
>
> Bugs: 352658
> http://bugs.kde.org/show_bug.cgi?id=352658
>
>
> Repository: dolphin-plugins
>
>
> Description
> -------
>
> This fixes a build failure that happens since the KFileItem include was replaced by a forward declaration in Dolphin's kversioncontrolplugin.h in https://quickgit.kde.org/?p=dolphin.git&a=commitdiff&h=bfa4274f58fcf5c873f20210ce6a88277671ccee
>
>
> Diffs
> -----
>
> git/fileviewgitplugin.h 3e92605
> svn/fileviewsvnplugin.h a1eda54
>
> Diff: https://git.reviewboard.kde.org/r/125247/diff/
>
>
> Testing
> -------
>
> dolphin-plugins now builds fine for me. If I missed anything, please let me know :-)
>
>
> Thanks,
>
> Frank Reininghaus
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20150926/10b8b4fe/attachment.htm>
More information about the kfm-devel
mailing list