[FFmpeg-devel] [PATCH] Other configure version check patches

Stefano Sabatini stefano.sabatini-lala
Sun Oct 19 00:10:52 CEST 2008


On date Saturday 2008-10-18 22:50:37 +0200, Diego Biurrun encoded:
> On Sat, Oct 18, 2008 at 08:51:01PM +0100, M?ns Rullg?rd wrote:
> > Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
> > 
> > > MinGW patche by Ramiro, this patch also suggest a better way to convey
> > > the meaning of the check to the user, so I think that
> > > require_cpp_condition() may be safely removed.
> > >
> > > --- configure	(revision 15634)
> > > +++ configure	(working copy)
> > > @@ -1736,7 +1736,7 @@
> > >  enabled libx264    && require  x264 x264.h x264_encoder_open -lx264 -lm \
> > > -                   && require_cpp_condition x264 x264.h "X264_BUILD >= 65"
> > > +                   && { check_cpp_condition x264.h "X264_BUILD >= 65" || die "libx264 version must be >= 0.65."; }
> > 
> > Are those { } really needed?  It seems to me it should work fine
> > without (and that's how I would probably have done it in the first
> > place).
> 
> I wondered the same thing.

You're right. New patch attached.

[...]
 
Regards.
-- 
FFmpeg = Fostering and Fiendish Meaningless Proud Exploitable Gorilla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: improve-libx264-check-00.patch
Type: text/x-diff
Size: 788 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081019/d4097939/attachment.patch>



More information about the ffmpeg-devel mailing list