[FFmpeg-devel] [PATCH] move some dependencies from configure to Makefile

[Måns Rullgård]

Aurelien Jacobs <aurel at gnuage.org> writes:

> Hi,
>
> The correct place to describe dependencies between parts of the
> internal libav* code is Makefiles. configure should only be
> used to describe external dependencies (eg: libs) for which
> there will be no way to build related code when the dependency
> is not available.
> Thus, the attached patch is moving some dependencies description
> from configure to Makefile.

At least some of those are wrong.  Some things require the actual
CONFIG_ variable of something else to be defined; merely compiling the
files is not sufficient.  Furthermore, this reverses the logic.  The
current scheme means that if A depends on B, (B disabled) => (A disabled).
With your patch, it becomes (A enabled) => (B enabled).  The idea is
that --disable-foo on the command line should also disable everything
that depends on foo.

-- 
M?ns Rullg?rd
mans at mansr.com