[Ffmpeg-devel] [PATCH] configure - simplify

Diego Biurrun diego
Tue Feb 6 08:59:44 CET 2007


On Tue, Feb 06, 2007 at 04:14:14AM -0200, ramiro at lisha.ufsc.br wrote:
> 
> libogg.diff simplifies the check for libogg for libvorbis and libtheora.

Thanks, applied.

> else.diff changes occurences of "if;then <1 line> else <1 line> fi" to &&
> and ||

Will look at it later, but next time please send one mail per patch,
this simplifies patch management for us, thanks.

> While echoing to config.mak and $TMPH, some ifs indent 2 and others indent
> 4. Which one is preferred?

I prefer two spaces in shell scripts, but most of FFmpeg configure uses
4 spaces and I think Mans favors 4 spaces.  So I'm fine with 4 spaces
and this is what it should be.

Diego




More information about the ffmpeg-devel mailing list