[FFmpeg-cvslog] r20406 - trunk/libavcodec/Makefile

Reimar Döffinger Reimar.Doeffinger
Wed Oct 28 21:48:49 CET 2009


On Wed, Oct 28, 2009 at 08:47:01PM +0000, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> 
> > On Wed, Oct 28, 2009 at 08:49:19PM +0100, mru wrote:
> >> Author: mru
> >> Date: Wed Oct 28 20:49:19 2009
> >> New Revision: 20406
> >> 
> >> Log:
> >> Makefile cosmetics
> >> 
> >> Modified:
> >>    trunk/libavcodec/Makefile
> >> 
> >> Modified: trunk/libavcodec/Makefile
> >> ==============================================================================
> >> --- trunk/libavcodec/Makefile	Wed Oct 28 20:49:16 2009	(r20405)
> >> +++ trunk/libavcodec/Makefile	Wed Oct 28 20:49:19 2009	(r20406)
> >> @@ -702,11 +702,12 @@ $(SUBDIR)cos_tables.c: $(SUBDIR)costable
> >>  $(SUBDIR)sin_tables.c: $(SUBDIR)costablegen$(HOSTEXESUF)
> >>  	./$< sin > $@
> >>  
> >> -ifeq ($(CONFIG_MPEGAUDIO_HP),yes)
> >> +ifdef CONFIG_MPEGAUDIO_HP
> >
> > I copy-and-paste-modified that from subdir.mak:
> > ifeq ($(CONFIG_STATIC),yes)
> > should that maybe be simplified, too?
> 
> Why do people *always* copy&paste from a place that does it wrong?
> You picked the one place in all the makefiles that used this ugly
> syntax.  Why?

Simple, because I remembered it was done (obviously in the past) via ifeq
and thus did a grep for ifeq and ended up there.
The fact that it was the only match probably should have been a hint,
but I was too happy to accept the first working solution :-)



More information about the ffmpeg-cvslog mailing list