[FFmpeg-devel] [PATCH] Implement version check in configure

Måns Rullgård mans
Sun Sep 21 19:44:23 CEST 2008


Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:

> On date Saturday 2008-09-20 13:10:46 +0200, Stefano Sabatini encoded:
>> On date Saturday 2008-09-20 12:25:21 +0200, Diego Biurrun encoded:
>> > On Fri, Sep 19, 2008 at 12:23:52AM +0200, Stefano Sabatini wrote:
>> > > 
>> > > --- configure	(revision 15366)
>> > > +++ configure	(working copy)
>> > > @@ -608,6 +627,17 @@
>> > >  
>> > > +require3(){
>> > 
>> > This is not a good name.  Not that require2 was one, but let us not make
>> > things worse.
>> 
>> require_headers_func_version?
>> require_with_version?
>> 
>> Suggestions are welcome.
>
> Choosen the first one, seems more consistent with the new name for
> check_func_headers used by Diego for another check function.

The name is way too long.  Furthermore, the test could be made more
generic.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list