[PATCH xf86-video-vmware] Fix include order.

[Gaetan Nadon]

On Fri, 2011-01-21 at 20:07 +0100, Cyril Brulebois wrote:

> HAVE_LIBDRM comes from config.h, so include it first.
> 
> With CFLAGS="-Wall -Werror", the build fails with:
> |   CC     vmware_drv_la-vmwaremodule.lo
> | cc1: warnings being treated as errors
> | vmwaremodule.c: In function ‘vmware_check_kernel_module’:
> | vmwaremodule.c:100: error: implicit declaration of function ‘drmOpen’
> | vmwaremodule.c:112: error: implicit declaration of function ‘drmClose’
> 
> Signed-off-by: Cyril Brulebois <kibi at debian.org>
> ---
>  src/vmwaremodule.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/vmwaremodule.c b/src/vmwaremodule.c
> index a7ba854..2754879 100644
> --- a/src/vmwaremodule.c
> +++ b/src/vmwaremodule.c
> @@ -26,14 +26,14 @@
>  #include <xorg-server.h>
>  #include <xf86.h>
>  
> -#ifdef HAVE_LIBDRM
> -#include <xf86drm.h>
> -#endif
> -
>  #ifdef HAVE_CONFIG_H
>  #include "config.h"
>  #endif
>  
> +#ifdef HAVE_LIBDRM
> +#include <xf86drm.h>
> +#endif
> +
>  #ifndef HAVE_XORG_SERVER_1_5_0
>  #include <xf86_ansic.h>
>  #include <xf86_libc.h>
> -- 
> 1.7.2.3
> 


This is the correct order. 

Reviewed-by: Gaetan Nadon <memsize at videotron.ca>

I have copied the driver maintainer who will review and potentially push
the patch.
http://www.x.org/wiki/vmware



> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110121/e3ad5651/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110121/e3ad5651/attachment.pgp>