[FFmpeg-devel] [PATCH] build: make sure probed tools don't wait for stdin input.

James Almer jamrial at gmail.com
Sun Oct 6 20:51:02 CEST 2013


On 06/10/13 7:45 AM, Michael Niedermayer wrote:
> On Sat, Oct 05, 2013 at 09:06:27PM -0300, James Almer wrote:
>> On 05/10/13 6:44 PM, Michael Niedermayer wrote:
>>> On Fri, Feb 15, 2013 at 12:05:59AM +0100, Clément Bœsch wrote:
>>>> On Wed, Jan 09, 2013 at 12:15:46AM +0100, Clément Bœsch wrote:
>>>>> a758c5e added probing for various tools, such as as. Unfortunately, GNU
>>>>> as is reading stdin with -v, and thus configure is stalled with
>>>>> configure arguments such as --as=as.
>>>>>
>>>>> The --version calls are unchanged because it is supposed to -v + exit.
>>>>>
>>>>> Fixes Ticket #1898.
>>>>
>>>> Ping
>>>
>>> probably ok
>>
>> As i said back in February, the output of GNU as doesn't start with "gcc", 
>> so the line
>>
>>> elif $_cc -v 2>&1 < /dev/null | grep -qi ^gcc; then
>>
>> needs to check for both "GNU" and "gcc".
>> Something like grep -qi '^g\(cc\|nu\)' should do it.
> 
> yes but is this related to fixing the regresion that configure
> waits infinitly on stdin? (which is what this patch is doing)
> 
> thanks

No, it fixes a bug introduced by fixing the hang on stdin.
Since GNU as will not hang up anymore, the grep check will be done but fail 
as it only checks for '^gcc'.


More information about the ffmpeg-devel mailing list