[Kde-pim] Review Request 124925: Fix build on FreeBSD <11
Adriaan de Groot
groot at kde.org
Tue Aug 25 20:45:10 BST 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124925/#review84359
-----------------------------------------------------------
Every other (header) file in syndication that uses time_t already #includes <ctime>, but tools.cpp doesn't. That header is probably pulled in transitively by other headers when building in Linux, not on FreeBSD 10.
- Adriaan de Groot
On Aug. 25, 2015, 7:31 p.m., Tobias Berner wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124925/
> -----------------------------------------------------------
>
> (Updated Aug. 25, 2015, 7:31 p.m.)
>
>
> Review request for KDEPIM and Sergio Luis Martins.
>
>
> Repository: syndication
>
>
> Description
> -------
>
> ctime needs to be included
>
>
> Diffs
> -----
>
> src/syndication/tools.cpp 17fa834
>
> Diff: https://git.reviewboard.kde.org/r/124925/diff/
>
>
> Testing
> -------
>
> built locally
>
>
> Thanks,
>
> Tobias Berner
>
>
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list