[FFmpeg-devel] [PATCH 1/3] configure: Use set_all to force the dependency refresh

[Michael Niedermayer]

On Wed, Feb 17, 2016 at 11:29:05PM +0100, Michael Niedermayer wrote:
> On Wed, Feb 17, 2016 at 11:15:17PM +0100, Hendrik Leppkes wrote:
> > On Wed, Feb 17, 2016 at 10:07 PM, Clément Bœsch <u at pkh.me> wrote:
> > > On Wed, Feb 17, 2016 at 10:02:47PM +0100, Michael Niedermayer wrote:
> > >> From: Luca Barbato <lu_zero-aBrp7R+bbdUdnm+yROfE0A at public.gmane.org>
> > >>
> > >
> > > strange mail.
> > >
> > >> Allow to issue twice --enable options.
> > >>
> > >> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > >> ---
> > >>  configure |    2 +-
> > >>  1 file changed, 1 insertion(+), 1 deletion(-)
> > >>
> > >> diff --git a/configure b/configure
> > >> index fb81a19..f492be4 100755
> > >> --- a/configure
> > >> +++ b/configure
> > >> @@ -656,7 +656,7 @@ enable_deep_force(){
> > >>  }
> > >>
> > >>  request(){
> > >> -    disable $* # force the refresh of the dependencies
> > >> +    set_all no $* # force the refresh of the dependencies
> > >>      for var; do
> > >>          enable ${var}_requested
> > >>      done
> > >
> > > i confirm this fixes the issue
> > >
> > 
> > Libav seems to have decided to revert the entire badge that caused
> > this after they were brought up to speed on the remaining issues.
> > So before we worry too much, we should just follow that and then
> > re-evaluate any new fixed patches once available.
> 
> agree, i dont have a full fix for the issues yet either

btw heres a simple script to test configure, dunno if its usefull
but as i already wrote it, should post it too i guess
also maybe someone wants to integrate this or something like this in
fate (maybe not for windows)
the output fro this can be diffed to see changes

#!/bin/bash

set -e

testthis(){
    make distclean >& configuretest-buildlogtmp || true
    echo $*
    dash ./configure --disable-debug --disable-optimizations --cc='ccache clang' $*
    make -j12 >> configuretest-buildlogtmp 2>&1 || cat configuretest-buildlogtmp
}

testthis --disable-everything --disable-network --enable-ffplay
testthis --disable-everything --enable-ffmpeg --enable-gpl --enable-gpl
testthis --disable-protocols --enable-protocol=https
testthis --disable-everything --disable-network

rm configuretest-buildlogtmp


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160218/175ad023/attachment.sig>