[PATCH] xf86xv.h cannot be included without first including xorg-server.h.
Stefan Dirsch
sndirsch at suse.de
Mon Oct 13 20:25:56 PDT 2014
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 <michel at daenzer.net> wrote:
> >>
> >> On 07.10.2014 18:46, Daniel Martin wrote:
> >>
> >> On 7 October 2014 11:29, Stefan Dirsch <sndirsch at suse.de> 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 <daniels at collabora.com>
> >
> >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
Thanks,
Stefan
Public Key available
------------------------------------------------------
Stefan Dirsch (Res. & Dev.) SUSE LINUX Products GmbH
Tel: 0911-740 53 0 Maxfeldstraße 5
FAX: 0911-740 53 479 D-90409 Nürnberg
http://www.suse.de Germany
--------------------------------------------------------------
SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix
Imendörffer, HRB 16746 (AG Nürnberg)
--------------------------------------------------------------
More information about the xorg-devel
mailing list