[PATCH v3 1/2] dix: Add facilities for client ID tracking.

Tiago Vignatti tiago.vignatti at nokia.com
Tue Oct 19 09:36:54 PDT 2010


On Tue, Oct 05, 2010 at 09:55:59AM +0200, ext Rami Ylim�ki wrote:
> 
>   On 10/02/2010 02:00 AM, Jesse Adkins wrote:
> >> +AC_ARG_ENABLE(clientids,      AS_HELP_STRING([--disable-clientids], [Build Xorg with client ID tracking (default: enabled)]), [CLIENTIDS=$enableval], [CLIENTIDS=yes])
> > Shouldn't that be --enable-clientids instead of --disable-clientids?
> 
> Alan already told the difference of these options, but did you 
> originally mean that the option should be disabled by default and 
> therefore --enable-clientids should be used instead?
> 
> I'm about to start work on the client side and propose a new request for 
> XResource extension for querying the local client PID. Therefore I think 
> it makes sense to keep the feature enabled by default in X server. 
> Having that enabled shouldn't affect X server performance in a 
> significant way.

the "general consensus" was to enable code that stress compilation paths and
built without care too much whether the default is the option that mostly
distros will want. For instance, in MeeGo we will want to disable this option
when we're going to ship X for the products.

Therefore, just leave the patch like this which is fine. Thanks!

             Tiago


More information about the xorg-devel mailing list