[PATCH] Make sure X_BYTE_ORDER is defined for servermd.h

Keith Packard keithp at keithp.com
Wed Jul 30 23:58:45 PDT 2014


Michel Dänzer <michel at daenzer.net> writes:

> On 31.07.2014 02:16, Keith Packard wrote:
>> Michel Dänzer <michel at daenzer.net> writes:
>> 
>>> -#ifdef HAVE_DIX_CONFIG_H
>>> -#include "dix-config.h"
>>> -#else
>>> -#include "xorg-server.h"
>>> +#if !defined(_DIX_CONFIG_H_) && !defined(_XORG_SERVER_H_)
>>> +#error Drivers must include xorg-server.h before any other xserver headers
>>> +#error xserver code must include dix-config.h before any other headers
>>>  #endif
>> 
>> Might be better to stick this check in os.h or misc.h as those are both
>> included by most of the files in the server.
>
> Sounds good.

How about you 'R-b' my patch and submit a patch to misc.h which I can
R-b and then we can get both merged to master and make it possible to
build drivers again?

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140730/90eb63c9/attachment.sig>


More information about the xorg-devel mailing list