[FFmpeg-devel] [PATCH] List enabled code in configure output

Ramiro Polla ramiro
Wed Sep 26 17:17:03 CEST 2007


M?ns Rullg?rd wrote:
> Ramiro Ribeiro Polla <ramiro at lisha.ufsc.br> writes:
>
>   
>> Hello,
>>
>>  wrote:
>>     
>>> Ramiro Ribeiro Polla wrote:
>>>   
>>>       
>>>> Hello,
>>>>
>>>> I uploaded a patch [1] to issue17 [2] on the roundup tracker. Is it ok
>>>> to apply?
>>>>     
>>>>         
>>> No, it's not POSIX shell compatible.
>>>   
>>>       
>> Ok, I'll try again with: mans --verbose 2 ./configure
>>
>> Could you enlighten us where it's not POSIX compatible?
>>     
>
> The ${space:0:$length}

Replaced with a while loop that adds one space at a time.

>  and ${*//$suffix/} substitutions are not POSIX
> compatible.
>   

Replaced with one sed call. This could be applied separately.

Thanks for reviewing. New patch attached.

Ramiro Polla
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: show_list3.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070926/317d6b0b/attachment.txt>



More information about the ffmpeg-devel mailing list