<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 7 October 2014 10:54, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 07.10.2014 18:46, Daniel Martin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 7 October 2014 11:29, Stefan Dirsch <<a href="mailto:sndirsch@suse.de" target="_blank">sndirsch@suse.de</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Without this the build fails on systems with the latest glibc,<br>
throwing this error:<br>
[...]<br><br>
This is caused by HAVE_STRNDUP not being set (it is set from xorg-server.h),<br>
causing os.h to redefine it.<br>
</blockquote>
<br>
Have a look at this commit in xf86-video-intel:<br>
     <a href="http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=d209b03915e4d12ce445d72ac09bb4fe664ba664" target="_blank">http://cgit.freedesktop.org/<u></u>xorg/driver/xf86-video-intel/<u></u>commit/?id=<u></u>d209b03915e4d12ce445d72ac09bb4<u></u>fe664ba664</a><br>
<br>
With that HAVE_STRNDUP will be defined in config.h,<br>
which looks like a better solution to me.<br>
</blockquote>
<br></span>
I disagree. xorg-server.h must be included before any other headers from xorg/, so Stefan's patch is the proper solution.</blockquote><div><br></div><div>Yes, absolutely. xorg-server.h does things like define _XSERVER64, which changes the size of CARD32 and thus the entire server ABI. Not including xorg-server.h first is terminally broken; this is just one symptom.</div><div><br></div><div>Stefan:</div><div>Reviewed-by: Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a>></div><div><br></div><div>Cheers,</div><div>Daniel </div></div></div></div>