[PATCH] configure: set X render animated cursors optional

[Tiago Vignatti]

On Wed, Sep 02, 2009 at 06:15:41PM +0200, Julien Cristau wrote:
> On Wed, Sep  2, 2009 at 14:16:10 +0300, Tiago Vignatti wrote:
> 
> > I share the same thoughts as you here, Keith [0]. For instance, look some
> > recently commits that I've done to set autoconf options: 
> > 
> >     root at xeron:~/xserver# git shortlog --author=vignatti --since="3 months ago" |
> >     grep configure
> >           configure: Provide the --enable/disable-xaa option.
> >           configure: introduce --{enable,disable}-vgahw
> >           configure: introduce --{enable,disable}-vbe
> >           configure: introduce --{enable,disable}-int10-module
> >     root at xeron:~/xserver# 
> > 
> I'm not sure that's a good thing.  More options means a *lot* more
> combinations of options, when just a few of them will actually get
> build-tested (let alone runtime-tested).
> 

I agree with you in respect of the number of combinations, that will increase,
thus implying in more untested cases.

But this does not justify the fact that some systems that don't need some
modules necessarily include them - think about tiny systems.


            Tiago