[FFmpeg-devel] [PATCH] Fix big/little endian check in configure (broken on Mac OS X)

[Måns Rullgård]

Patrice Bensoussan <patrice.bensoussan at free.fr> writes:

> On 13 Mar 2008, at 22:35, M?ns Rullg?rd wrote:
>
>> Patrice Bensoussan <patrice.bensoussan at free.fr> writes:
>>
>>> Hello,
>>>
>>> It seems the endian check in configure has been broken on Mac OS X  
>>> (tr
>>> doesn't seem to like binary files). Here is a fix which should
>>> hopefully work everywhere...
>>>
>>> Patrice
>>>
>>> Index: configure
>>> ===================================================================
>>> --- configure   (revision 12435)
>>> +++ configure   (working copy)
>>> @@ -1563,9 +1563,9 @@
>>>  # ---
>>>  # big/little-endian test
>>>  check_cc <<EOF || die "endian test failed"
>>> -unsigned int endian = 'B' << 24 | 'I' << 16 | 'G' << 8 | 'E';
>>> +unsigned int endian = 'B' << 24 | 'I' << 16 | 'G' << 8;
>>>  EOF
>>> -tr -c -d BIGE < $TMPO | grep -q BIGE && enable bigendian
>>> +strings -n 3 $TMPO | grep -q BIG && enable bigendian
>>
>> It used to be like that, and broke horribly on some systems.
>
> Do you remember what was the issue? (btw sorry, I missed the related  
> thread on solaris, altough grep seems to be the issue and not tr as on  
> Mac OS X).

It was something about strings only printing things that looked like
actual null-terminated strings.

Anyhow, I've committed an alternate fix.  Please report any malfunction.

-- 
M?ns Rullg?rd
mans at mansr.com