[FFmpeg-devel] [PATCH] build: fix install rule with --progs-suffix.

Clément Bœsch ubitux at gmail.com
Sun Sep 11 01:49:55 CEST 2011


On Sat, Sep 10, 2011 at 12:09:50AM +0200, Stefano Sabatini wrote:
> On date Friday 2011-09-09 14:53:42 +0200, Clément Bœsch encoded:
> > On Fri, Sep 09, 2011 at 01:15:11AM +0200, Stefano Sabatini wrote:
> > > On date Thursday 2011-09-08 23:16:56 +0200, Clément Bœsch encoded:
> > > > From: Clément Bœsch <clement.boesch at smartjog.com>
> > > > 
> > > > ---
> > > >  Makefile |    3 ++-
> > > >  1 files changed, 2 insertions(+), 1 deletions(-)
> > > > 
> > > > diff --git a/Makefile b/Makefile
> > > > index ef1f3d4..09bdfb1 100644
> > > > --- a/Makefile
> > > > +++ b/Makefile
> > > > @@ -16,6 +16,7 @@ PROGS-$(CONFIG_FFPROBE)  += ffprobe
> > > >  PROGS-$(CONFIG_FFSERVER) += ffserver
> > > >  
> > > >  PROGS      := $(PROGS-yes:%=%$(EXESUF))
> > > > +INSTPROGS   = $(PROGS-yes:%=%$(PROGSSUF)$(EXESUF))
> > > 
> > > Isn't setting:
> > > PROGS      := $(PROGS-yes:%=%$(PROGSSUF)$(EXESUF))
> > > 
> > > simpler?
> > > 
> > 
> > It is, but unfortunately, it doesn't work: the target name has to match
> > the source filename; ffmpeg ← ffmpeg.o. If you add suffixes to the
> > target, it won't match the source file.
> 
> OK, so should be fine if tested.

Pushed

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110911/2e57d65e/attachment.asc>


More information about the ffmpeg-devel mailing list