On 05/20/2013 09:50 AM, Martin Storsjö wrote: > 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.
The best would be making so make config is the only thing triggering it... Maybe is feasible. > > // Martin > > > _______________________________________________ > libav-devel mailing list > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel > _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
