On Thu, 18 Jul 2013, Diego Biurrun wrote:

On 2013-07-18 18:17, Martin Storsjö wrote:
On Thu, 18 Jul 2013, Diego Biurrun wrote:

On 2013-07-18 16:37, Hendrik Leppkes wrote:
On Thu, Jul 18, 2013 at 4:32 PM, Diego Biurrun <[email protected]> wrote:
---

As promised some time ago, this should be a rather clean solution.

A clean solution that breaks everyones fate scripts with the first
patch from the set?

Yes.  There's only a handful of people running FATE instances for libav.

The fate-rsync target isn't used only by them, but by anybody who wants
to run the fate tests locally before submitting a patch.

We could deprecate but keep fate-rsync as an alias for samples-rsync and print a message saying it's obsolete. Also, fate-rsync would trigger the error check, while samples-rsync would not.

That's certainly better than dropping it completely. But fate-rsync still feels much better as a name for this target, so I'm still a bit undecided.

The alternative is something like this:
ifneq (,$(filter check fate% $(FATE),$(filter-out fate-rsync,$(MAKECMDGOALS))))

Obviously not as neat as your versions...

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to