[FFmpeg-devel] [PATCH] Add additional FFV1 fate tests

[Peter B.]

On 11/11/2013 04:48 PM, Reimar Döffinger wrote:
> On 11.11.2013, at 16:38, "Peter B." <pb at das-werkstatt.com> wrote:
>> If I interpret this correctly, this is caused by the dependency that a decoding-test (fate-ffv1-dec-%) must have its encoding-test (fate-ffv1-enc-%) run as prerequisite.
>> So, the output is - although it looks "unpretty" - correct.
>> Isn't it?
> I think not in this case (though I am unsure when and why make prints that message).
> Do all these targets at some point somehow get added to .PHONY?

Well, I assumed that the line 207 in "tests/Makefile" would take care of
that?
.PHONY: fate* lcov lcov-reset


But I've now also tried to manually add it in ffv1.mak:
.PHONY: $(FATE_FFV1)


This changes the output messages from "is up to date" to "nothing to be
done for":

[quote]
  TEST    ffv1-enc-v1-bff
  make: `fate-ffv1-enc-v1-bff' is up to date.
  TEST    ffv1-dec-v1-bff
[/quote]

becomes:
[quote]
  TEST    ffv1-enc-v1-bff
  make: Nothing to be done for `fate-ffv1-enc-v1-bff'.
  TEST    ffv1-dec-v1-bff
[/quote]


Sorry. As I said: I'm new to Makefiles...

Thanks,
Pb