Review Request 113139: Try to export include targets for Plasma as well
Stephen Kelly
steveire at gmail.com
Mon Oct 7 10:35:33 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113139/#review41331
-----------------------------------------------------------
Ship it!
> Unfortunately it isn't entirely successful as it seems camelcase headers are installed by KF5::plasma into include/KDE/Plasma/
Odd. I tried to add this dir, but seem to have hit a bug I'll look into. Patch looks good for now I think, thanks!
- Stephen Kelly
On Oct. 7, 2013, 8:13 a.m., Ben Cooksley wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113139/
> -----------------------------------------------------------
>
> (Updated Oct. 7, 2013, 8:13 a.m.)
>
>
> Review request for kdelibs, Plasma and Stephen Kelly.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> Add include targets for KF5::plasma, which will hopefully contribute towards fixing the kde-workspace[master] build on build.kde.org.
> Unfortunately it isn't entirely successful as it seems camelcase headers are installed by KF5::plasma into include/KDE/Plasma/ but it should be a start...
>
>
> Diffs
> -----
>
> src/plasma/CMakeLists.txt b21fd7b
>
> Diff: http://git.reviewboard.kde.org/r/113139/diff/
>
>
> Testing
> -------
>
> In place on CI build system. Proper include path now given to compiler.
>
>
> Thanks,
>
> Ben Cooksley
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131007/682d5651/attachment.html>
More information about the Plasma-devel
mailing list