On 14.10.2014 12:25, Stefan Dirsch wrote:
On Tue, Oct 14, 2014 at 10:02:13AM +0900, Michel Dänzer wrote:
On 13.10.2014 21:58, Stefan Dirsch wrote:
On Tue, Oct 07, 2014 at 11:56:55AM +0100, Daniel Stone wrote:
Hi,

On 7 October 2014 10:54, Michel Dänzer <[email protected]> wrote:

     On 07.10.2014 18:46, Daniel Martin wrote:

         On 7 October 2014 11:29, Stefan Dirsch <[email protected]> wrote:

             Without this the build fails on systems with the latest glibc,
             throwing this error:
             [...]

             This is caused by HAVE_STRNDUP not being set (it is set from
             xorg-server.h),
             causing os.h to redefine it.


         Have a look at this commit in xf86-video-intel:
              http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?
         id=d209b03915e4d12ce445d72ac09bb4fe664ba664

         With that HAVE_STRNDUP will be defined in config.h,
         which looks like a better solution to me.


     I disagree. xorg-server.h must be included before any other headers from
     xorg/, so Stefan's patch is the proper solution.


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.

Stefan:
Reviewed-by: Daniel Stone <[email protected]>

Thanks! Anybody, who would like to push it?

Thomas already pushed it. :)

Hmm. Then I don't see it. Latest commit in git I can see is from Sep 22nd.

   vmware/saa: Fix saa_check_poly_fill_rect_noreadback v2

Ah, I thought this was about the -vmmouse patch. Sorry for the noise. :(


--
Earthling Michel Dänzer            |                  http://www.amd.com
Libre software enthusiast          |                Mesa and X developer
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to