[MPlayer-dev-eng] [PATCH] configure: check for gsm.h and openjpeg-1.5/openjpeg.h
Alexander Strasser
eclipse7 at gmx.net
Sat Jan 26 03:13:39 CET 2013
Hi all!
Josh wrote:
> On 1/25/2013 19:37, Diego Biurrun wrote:
> >
> >I must say this whole business of duplicating ./ffmpeg/configure in
> >./configure is getting crazier by the day. ./configure is approaching
> >10k lines of brittle, hard-to-maintain shell scripting with dizzying
> >amounts of code duplication.
> >
> >Almost all of the changes done to configure are part of this cat and
> >mouse game. Somebody needs to step up and fix this properly instead.
I have to say I agree with Diego on this one!
Diego, I even dug out your old patch and the discussion. It is
definitely on my list to look into it. Before I have to finish some
other things though.
I also wanted to look into how the MPlayer forks do it.
> Just let it be until compiling breaks and actually needs fixing. I
> have GSM in my mplayer build and despite the wundef playing gsm
> encoded files through libgsm or ffgsm seems to work perfectly fine.
IMHO the patches can be committed if they are correct and done
anyway. It doesn't do much harm anymore and will all go away once
we have a better solution to deal with FFmpeg configuration.
Alexander
More information about the MPlayer-dev-eng
mailing list