On 15/03/2011 00:28, Jakob Bornecrantz wrote:
> On Mon, Mar 14, 2011 at 11:08 PM, Jon TURNEY
> <jon.tur...@dronecode.org.uk> wrote:
>> From: nobled <nob...@dreamwidth.org>
>>
>> The Hurd kernel doesn't have DRM yet.
>>
>> Signed-off-by: Jon TURNEY <jon.tur...@dronecode.org.uk>
>> Reviewed-by: Julien Cristau <jcris...@debian.org>
>> ---
>>  configure.ac |   13 +++++++++++--
>>  1 files changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 38e673d..5bcb5a1 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -730,12 +730,21 @@ else
>>  fi
>>
>>  dnl Direct rendering or just indirect rendering
>> +case "$host_os" in
>> +gnu*)
>> +    dnl Disable by default on GNU/Hurd
>> +    driglx_direct_default="no"
>> +    ;;
>> +*)
>> +    driglx_direct_default="yes"
>> +    ;;
>> +esac
> 
> Isn't there somewhere else in the configure file where we setup
> defaults based on OS where this could go? Or is that to late?
> Otherwise looks good.

$ grep 'case "$host_os"' configure.ac | wc -l
11

The style seems to be to have separate checks of $host_os preceding each block
which needs to use some host-specific value, and this change conforms with
that style.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to