[FFmpeg-devel] [PATCH] Define FFMPEG_PREFIX in config.h

Stefano Sabatini stefano.sabatini-lala
Tue Sep 30 22:25:12 CEST 2008


On date Monday 2008-09-29 23:48:44 +0100, M?ns Rullg?rd encoded:
> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
> 
> > Hi, as in subject,
> >
> > conceived for supporting $prefix/share/ffmpeg preset paths but useful
> > by its own.
> >
> > Regards.
> > -- 
> > FFmpeg = Frightening Foolish Multipurpose Proud Erudite God
> >
> > Index: configure
> > ===================================================================
> > --- configure	(revision 15451)
> > +++ configure	(working copy)
> > @@ -2055,6 +2055,7 @@
> >  echo "#define FFMPEG_CONFIGURATION \"$FFMPEG_CONFIGURATION\"" >> $TMPH
> >  
> >  echo "FFMPEG_CONFIGURATION=$FFMPEG_CONFIGURATION" >> config.mak
> > +echo "#define FFMPEG_PREFIX \"$prefix\"" >> $TMPH
> 
> What if $prefix contains '"' or '\' characters?

A quoting hell.

Check the patch below.

BTW configure seems broken if we use a prefix containing spaces like:
configure --prefix="hello world"

Regards.
-- 
FFmpeg = Frightening & Fantastic Magical Pitiless Evil Geek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: prefix-in-config-h-01.patch
Type: text/x-diff
Size: 798 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080930/50269919/attachment.patch>



More information about the ffmpeg-devel mailing list