Re: [PATCH weston] Make sure config.h is included before any system headers

2014-04-08 Thread Andrew Wedgbury
On Mon, 7 Apr 2014, Kristian Høgsberg wrote: Thanks, patch applied. This patch was oddly messed up though, there was an extra space at the beginning of each line. git send-email is the only reliable way I know to send patches as email and it works with any smpt server you can access with

[PATCH weston] Make sure config.h is included before any system headers

2014-04-07 Thread Andrew Wedgbury
There was an issue recently in screen-share.c where config.h was not being included, resulting in the wrong definition for off_t being used on 32 bit systems. I checked and I don't think this problem is happening elsewhere, but to help avoid this sort of problem in the future, I went through

Re: [PATCH weston] Make sure config.h is included before any system headers

2014-04-07 Thread Kristian Høgsberg
On Mon, Apr 07, 2014 at 12:40:35PM +0100, Andrew Wedgbury wrote: There was an issue recently in screen-share.c where config.h was not being included, resulting in the wrong definition for off_t being used on 32 bit systems. I checked and I don't think this problem is happening elsewhere, but