[Mplayer-cvslog] CVS: main Makefile, 1.304, 1.305 configure, 1.917, 1.918

[Nicolas Plourde]

Dominik 'Rathann' Mierzejewski wrote:

>On Tuesday, 12 October 2004 at 13:40, Nicolas Plourde wrote:
>  
>
>>On 12-Oct-04, at 6:19 AM, Dominik 'Rathann' Mierzejewski wrote:
>>
>>    
>>
>>>On Tuesday, 12 October 2004 at 11:00, Reimar Döffinger wrote:
>>>      
>>>
>>>>Hi,
>>>>
>>>>        
>>>>
>>>>>>+echocheck "compiler support for -fno-PIC"
>>>>>>+cat > $TMPC <<EOF
>>>>>>+int main(void) { return 0; }
>>>>>>+EOF
>>>>>>+if cc_check -fno-PIC ; then
>>>>>>+  CFLAGS="-fno-PIC $CFLAGS"
>>>>>>+  echores "yes"
>>>>>>+else
>>>>>>+  echores "no"
>>>>>>+fi
>>>>>>+
>>>>>>
>>>>>>            
>>>>>>
>>>>>This patch broke osx build, please fix it.  Looks like osx dont like
>>>>>-fno-pic flag.
>>>>>          
>>>>>
>>>>Can someone tell me the exact reason? How should this be fixed? Or
>>>>should I just remove that check?
>>>>        
>>>>
>>>I don't see what could be wrong with this check (but I don't have 
>>>access to an OSX box). I mean, it simply checks if the compiler accepts
>>>the -fno-PIC flag, right? So does OSX's gcc accept it and doesn't work 
>>>later or what?
>>>
>>>      
>>>
>>Yes osx gcc accept it but condec-cfg and other lib fail to link if its 
>>on. Dont know why.
>>    
>>
>
>Well, we won't know either, until you - for example - show us the error
>message. It could be a bug in OSX's gcc/linker.
>
>R.
>
>  
>
cc -I../libvo -I../../libvo  -fno-PIC -O4 -mcpu=7450 -mtune=7450 
-faltivec -pipe -ffast-math -fomit-frame-pointer -DSYS_DARWIN 
-DCONFIG_DARWIN -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64  -Ilibmpdemux 
-Iloader -Ilibvo -I/usr/local/include/freetype2 
-I/usr/local/include           -g codec-cfg.c mp_msg.c -o codec-cfg 
-DCODECS2HTML 
ld: /var/tmp//ccLcEXCd.o has external relocation entries in non-writable 
section (__TEXT,__text) for symbols:

On osx you should not use -fno-PIC but -mdynamic-no-pic instead. This 
part of the patch obviously break stuff so if its only usefull for Zeta 
configure script should only
check for it in this case.

-- 
====
Nicolas Plourde <nicolas.plourde at sympatico.ca>