Review Request 113196: kdirwatch: include correct header for time_t and ino_t.
Commit Hook
null at kde.org
Sun Oct 13 03:19:29 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113196/#review41623
-----------------------------------------------------------
This review has been submitted with commit 9c3a5107f7939b7b6fbbfdb41090ff5d5fab5ead by Nicolás Alvarez to branch frameworks.
- Commit Hook
On Oct. 11, 2013, 5:25 p.m., Nicolás Alvarez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113196/
> -----------------------------------------------------------
>
> (Updated Oct. 11, 2013, 5:25 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: kdelibs
>
>
> Description
> -------
>
> kdirwatch: include correct header for time_t and ino_t.
>
> We were including sys/time.h and sys/param.h, but Windows doesn't have
> them, and POSIX says those types are defined in sys/types.h.
>
> This doesn't fix the build on Windows, but it's a step.
>
>
> Diffs
> -----
>
> tier1/kcoreaddons/src/lib/io/kdirwatch_p.h 77c7d6d1f83f0c984043ed1c7e3a83a2e824d0c7
>
> Diff: http://git.reviewboard.kde.org/r/113196/diff/
>
>
> Testing
> -------
>
> Still builds and passes tests on Linux/gcc.
>
> Still fails to build on Windows/MSVC2010 but not for this reason :)
>
>
> Thanks,
>
> Nicolás Alvarez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131013/5ee155ae/attachment.html>
More information about the Kde-frameworks-devel
mailing list