[MPlayer-dev-eng] [PATCH 1/2] configure: remove 'T' flag from ar flags on OS/2

[KO Myung-Hun]

Hi/2.

Alexander Strasser wrote:
> Hi!
> 
> On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
>> OS/2 linker does not support a thin archive.
>> ---
>>  configure | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/configure b/configure
>> index 2e94bf1f0..9d21320ec 100755
>> --- a/configure
>> +++ b/configure
>> @@ -1854,6 +1854,8 @@ if os2 ; then
>>    # And ensure that $ld_static should be at first in a library list because it
>>    # has effects only libraries after it.
>>    test -n "$ld_static" && ld_static='-Wl,-static'
>> +  # OS/2 linker does not support a thin archive. Remove 'T' flag.
>> +  _arflags=$(echo $_arflags | tr -d T)
> 
> If the build on OS/2 is broken and works again
> after this patch, commit please. (You can commit
> to MPlayer SVN, right?)
> 

Yes. But currently, my svn commit does not work due to symbolic link.

I'll appreciate if you commit instead. ^______^

> 
> Can you confirm that in your installation
> 
>   sh -x -c 'ar 2>&1 | grep "\[T\] " ; echo "RC $?"'
> 
> works as expected?
> 
> If so does it mean OS/2 "ar" can create thin arhives
> but the remaining tool chain does not support linking
> them?
> 

Exactly.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.os2.kr/