Re: [PATCH] Don't build Intel DRM if $CHOST is not i?86-* or x86_64-*

[Jeremy Huddleston <jeremyhu@freedesktop.org>]

yeah, that's probably cleaner (I guess it'll avoid the -*), but it should have the same effect.

I'll make the change.

On Feb 1, 2012, at 12:43 PM, Julien Cristau wrote:

> On Mon, Jan 30, 2012 at 15:25:20 -0800, Jeremy Huddleston wrote:
> 
>> This fixes a failure in 'make check' found by the tinderbox when trying to
>> build this code on Linux/ppc.  This code is only designed to run on
>> Intel platforms, so don't even bother building it if we're not in that set.
>> 
>> Found-by: Tinderbox
>> Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
>> ---
>> 
>> It now causes the intel bits to not build on my Linux/ppc tinderbox, but I'd
>> appreciate someone verifying that it does the right thing on intel boxes as
>> well.
>> 
>> configure.ac |    5 ++++-
>> 1 files changed, 4 insertions(+), 1 deletions(-)
>> 
>> diff --git a/configure.ac b/configure.ac
>> index 773167f..f5ebc1d 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -250,7 +250,10 @@ if test "x$INTEL" != "xno" -o "x$RADEON" != "xno"; then
>> 
>>     else
>>            if test "x$INTEL" != "xno"; then
>> -                   INTEL=yes
>> +                   case $host_os in
>> +                           i?86-*|x86_64-*) INTEL=yes ;;
>> +                           *) INTEL=no ;;
>> +                   esac
>>            fi
> 
> don't you want to check host_cpu rather than host_os?
> 
> Cheers,
> Julien
>