[FFmpeg-devel] [PATCH] configure: use ar and ranlib in deterministic mode

James Almer jamrial at gmail.com
Tue Jan 27 17:06:59 CET 2015


On 27/01/15 4:44 AM, Hendrik Leppkes wrote:
> On Tue, Jan 27, 2015 at 12:31 AM, Andreas Cadhalpun <
> andreas.cadhalpun at googlemail.com> wrote:
> 
>> Hi,
>>
>> On 26.01.2015 23:11, Michael Niedermayer wrote:
>>
>>> On Mon, Jan 26, 2015 at 07:08:42PM -0300, James Almer wrote:
>>>
>>>> On 26/01/15 7:04 PM, Michael Niedermayer wrote:
>>>>
>>>>> reverted, breaks on BSDs
>>>>>
>>>>> ar: illegal option -- D
>>>>>
>>>> [...]
>>
>>> Maybe a configure check for "ranlib -D" could be added.
>>>>
>>>
>>> yes
>>>
>>
>> Attached patch checks if the help output contains the options.
>> That should be sufficient.
>>
>> Best regards,
>> Andreas
>>
> 
> 
> Checking help output with grep? Really?
> Did you ever stop to check how all the other checks work in configure?
> 
> Usually it tries to run the command and see if it fails, instead of doing
> some arbitrary grep which may result in some other matches somewhere.
> 
> - Hendrik

makeinfo, texi2html, podman and rsync all do the same to check for specific functionality.
Not saying it's a good idea, just that there's a precedent.


More information about the ffmpeg-devel mailing list