On 13-08-02 11:27 PM, Alan Coopersmith wrote:
>>   AC_ARG_ENABLE(dri, AS_HELP_STRING([--disable-dri],
>> -                                  [Disable DRI support
>> [[default=auto]]]),
>> +                                  [Disable DRI support
>> [[default=enabled]]]),
>
> I might leave that as default=auto, but otherwise looks good to me.

I agree, as the stated default is not quite true. The DRI feature will
not be enabled if it is not present.
Moreover this same option on the remaining 6 drivers have "auto".

    ./xf86-video-tdfx/configure.ac:        [Disable DRI support
    [[default=auto]]]),
    ./xf86-video-mach64/configure.ac:      [Disable DRI support
    [[default=auto]]]),
    ./xf86-video-mga/configure.ac:         [Disable DRI support
    [[default=enabled]]]),
    ./xf86-video-savage/configure.ac:      [Disable DRI support
    [[default=auto]]]),
    ./xf86-video-r128/configure.ac:        [Disable DRI support
    [[default=auto]]]),
    ./xf86-video-intel/configure.ac:       [Disable DRI support
    [[default=auto]]]))
    ./xf86-video-sis/configure.ac:         [Disable DRI support
    [[default=auto]]]),


_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to