[Ffmpeg-cvslog] CVS: ffmpeg configure,1.211,1.212

[Rich Felker]

On Mon, Oct 31, 2005 at 02:40:46PM +0100, Diego Biurrun wrote:
> On Mon, Oct 31, 2005 at 02:33:07PM +0200, Ivan Kalvachev wrote:
> > 2005/10/19, Diego Biurrun CVS <diego at mplayerhq.hu>:
> > >
> > > Modified Files:
> > >         configure
> > > Log Message:
> > > DESTDIR support , patch taken from the Debian package
> > > by Samuel Hocevar < sam == at == zoy == dot == org >
> > >
> > > --- configure   12 Oct 2005 22:24:03 -0000      1.211
> > > +++ configure   19 Oct 2005 14:37:15 -0000      1.212
> > > @@ -1276,10 +1276,10 @@
> > >  echo "/* Automatically generated by configure - do not modify! */" > $TMPH
> > >  echo "#define FFMPEG_CONFIGURATION "'"'"$FFMPEG_CONFIGURATION"'"' >> $TMPH
> > >
> > > -echo "prefix=$prefix" >> config.mak
> > > -echo "libdir=$libdir" >> config.mak
> > > -echo "bindir=$bindir" >> config.mak
> > > -echo "mandir=$mandir" >> config.mak
> > > +echo "prefix=\$(DESTDIR)$prefix" >> config.mak
> > > +echo "libdir=\$(DESTDIR)$libdir" >> config.mak
> > > +echo "bindir=\$(DESTDIR)$bindir" >> config.mak
> > > +echo "mandir=\$(DESTDIR)$mandir" >> config.mak
> > 
> > This is no-sense
> > [unverified claims] 
> > Please revert.
> 
> Nonsense.  Read the patch and the discussion about it.  You will see
> that the commit is correct.

Diego seems to be correct. $(DESTDIR) is not expanded by the configure
script (notice the \$ instead of $) but inserted literally into the
makefile.

Rich