[Ffmpeg-devel] [PATCH] conditionally build parts of lavc

[Diego Biurrun]

On Sat, Jul 15, 2006 at 08:28:39PM +0100, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Sat, Jul 15, 2006 at 05:45:59PM +0100, M?ns Rullg?rd wrote:
> >> Diego Biurrun <diego at biurrun.de> writes:
> >> 
> >> > On Sat, Jul 15, 2006 at 03:42:57PM +0100, M?ns Rullg?rd wrote:
> >> >> 
> >> >> Here's a patch that lets parts of lavc and lavf be built only if some
> >> >> enabled codec needs them.  The dependencies may need to be adjusted a
> >> >> little, and some code could be moved between files to make it work
> >> >> better, but this should be good for an initial review.
> >> >> 
> >> >> Please test and comment.
> >> >> 
> >> >> --- configure	(revision 5752)
> >> >> +++ configure	(working copy)
> >> >> @@ -2094,10 +2095,16 @@
> >> >>    echo "AMR_CFLAGS=-DIF2=1" >> config.mak
> >> >>  fi
> >> >>  
> >> >> +for codec in $DECODER_LIST $ENCODER_LIST $PARSER_LIST $DEMUXER_LIST $MUXER_LIST; do
> >> >> -for codec in $DECODER_LIST $ENCODER_LIST $PARSER_LIST $DEMUXER_LIST $MUXER_LIST; do
> >> >
> >> > Hmm, cosmetic change?
> >> 
> >> Stupid diff.  Can you see *any* difference in those two lines?
> >
> > I thought it might be trailing whitespace, which my mailer does not
> > print..
> 
> Doesn't your commit script prevent that?

Yes, it does anyway.

> > But since we're running a build system cleanup competition anyway.. ;-)
> 
> Oh, so that's what this is...

Let's just keep the competition running forever and never declare a
winner.  Everybody will benefit from the ongoing effort this way ;)

Diego