[Xcb] Should ge.xml and xf86vidmode.xml be included?

[Jamey Sharp]

On Fri, Jan 20, 2012 at 06:59:10PM +0900, Arnaud Fontaine wrote:
> Hello,
> 
> I'm wondering whether  ge.xml and xf86vidmode.xml should  be included as
> part   of   xcb-proto   because   for  now   they   are   missing   from
> xcbinclude_HEADERS?

At a quick glance, your patch looks like a good idea.

Reviewed-by: Jamey Sharp <jamey at minilop.net>

> Also,  I'm wondering what's the  purpose of src/*.py
> files because they seem to not be used at all?

I wrote them to demonstrate how I thought some algorithms should work. I
believe nobody but me ever used them, and I don't particularly care what
happens to them. They're in the git history if I ever want them again.
:-)

Jamey

> From 0883bd7b73472ed43258c14c05b327e4548efe53 Mon Sep 17 00:00:00 2001
> From: Arnaud Fontaine <arnau at debian.org>
> Date: Fri, 20 Jan 2012 18:51:12 +0900
> Subject: [PATCH] Add ge and xf86vidmode protocol descriptions.
> 
> ---
>  src/Makefile.am |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index e1e2743..eef2dbc 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -6,6 +6,7 @@ xcbinclude_HEADERS = \
>  	damage.xml \
>  	dpms.xml \
>  	dri2.xml \
> +	ge.xml \
>  	glx.xml \
>  	randr.xml \
>  	record.xml \
> @@ -18,6 +19,7 @@ xcbinclude_HEADERS = \
>  	xc_misc.xml \
>  	xevie.xml \
>  	xf86dri.xml \
> +	xf86vidmode.xml \
>  	xfixes.xml \
>  	xinerama.xml \
>  	xinput.xml \
> -- 
> 1.7.8.3
> 

> _______________________________________________
> Xcb mailing list
> Xcb at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/xcb

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/xcb/attachments/20120120/8bfe3d58/attachment.pgp>