On Sun, 19 May 2013, Luca Barbato wrote:

On 05/19/2013 10:38 PM, Martin Storsjö wrote:
If upgrading an older build config, where newly added e.g.
decoders aren't present, to a newer git version where fate tests
exist that have dependencies declared on this codec, all make
invocations error out with "No such config: CONFIG_NEW_DECODER".
This is an intended feature to make sure that fate test
dependencies actually refer to existing config items.

In cases where upgrading an old build (and reconfiguring with
"make config"), this is an inconvenience. With this patch in
place, the caller can ignore these missing dependencies
by setting ALLOW_MISSING_CONFIG=1 during that call.
---
 tests/Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

I like the idea.

Perhaps the error message also could be extended to mention this, so that people that run into this issue won't have to search for what parameter to set.

I also toyed with the idea to make this the default, but that'd easily lead to people forgetting to set the variable to make sure fate tests they add are properly set up.

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to