[MPlayer-users] Configure severely broken on IRIX

[Matthew Saunier]

IRIX is a BSD UNIX, so it's sh is ksh88. 6.0 and up are compliant with 
SVR4, UNIX95, and POSIX as of 1999. Autoconf generated configure scripts 
have no problem working under these conditions, so if you're going to 
have a hand-coded configure it must be more resilient.

Also, MIPSpro is THE compiler on IRIX. Apps that do compile under GCC on 
IRIX tend to run much slower, and are harder to debug since ProDev 
expects MIPSpro to be present.

I've attached the output of your configure, do what you want, but if one 
mainstream but slightly old commercial UNIX is having issues, there will 
be other platforms that have issues as well.

    -Matt

The Wanderer wrote:
> On 05/06/2009 04:13 PM, Matthew Saunier wrote:
>
>> Hi everyone, mplayer doesn't even try to compile on IRIX. I haven't
>> dug into the code yet, but I have fixed your configure. It is not, as
>> the slashbang indicates, a legal sh script. Under bash it mostly
>> works, but it assumes GCC for everything and is incapable of
>> detecting the architecture of SGI machines. I've added MIPSpro
>> compiler support to it and cleaned up some bad assumptions and
>> linux-isms, patch attached.
>
> Patches should be sent to mplayer-dev-eng, following the rules in
> DOCS/tech/patches.txt.
>
> Also, MPlayer's configure script does not (or should not) require bash;
> it does, however, require POSIX. Any POSIX-compliant /bin/sh should be
> able to run it just fine.
>
>> @@ -1327,7 +1327,7 @@
>>  # Determine our OS name and CPU architecture
>>  if test -z "$_target" ; then
>>    # OS name
>> -  system_name=$(uname -s 2>&1)
>> +  system_name=`uname -s 2>&1`
>
> The specification at
>
> http://www.opengroup.org/onlinepubs/9699919799/utilities/sh.html
>
> (which I understand to be part of POSIX; correct me if I'm wrong) uses
> the $(subshell) syntax in at least one of its examples, and does not use
> the `subshell` syntax at all. This change, and those like it, are
> unlikely to be accepted; indeed I'm fairly sure I've seen commits which
> changed it in the other direction.
>
> configure is also not supposed to assume GCC - I remember reading that
> people have compiled successfully with e.g. ICC, at the very least. I'm
> somewhat skeptical that the sole compiler-related change which I see in
> your patch will be accepted as is, but there's pretty good odds that
> something with the same goal may go in if you propose it.
>

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: orig.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-users/attachments/20090506/a703a8dc/attachment-0002.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patched.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-users/attachments/20090506/a703a8dc/attachment-0003.txt>