[PATCH 6/6] Clean up Xext/Makefile.am

Jamey Sharp jamey at minilop.net
Tue Aug 3 10:27:47 PDT 2010


I like this series, though I'm not quite comfortable enough with the
code to offer a reviewed-by. It seems like a good step towards merging
xfree86 DDX functionality into DIX and eventually killing the
distinction between DDX and DIX though, so I'm all for it.

One note on this patch:

On Sun, Aug 01, 2010 at 09:28:23PM +0200, Tomas Carnecky wrote:
> -INCLUDES = -I$(top_srcdir)/hw/xfree86/dixmods/extmod @XORG_INCS@
> +INCLUDES = @XORG_INCS@

Doesn't this change belong in the earlier patch that deletes the extmod
directory?

Jamey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100803/becb71c5/attachment.pgp>


More information about the xorg-devel mailing list