Reinhard Tartler <[email protected]> writes:

> +fate-rsync:
> +ifdef SAMPLES

There is already a section conditional on SAMPLES.  Including this there
might look nicer.  The duplicated target line doesn't matter IMO.

> +     rsync -vaLW rsync://fate-suite.libav.org:/fate-suite/ $(SAMPLES)

Why the extra : ?

> +else
> +     @echo "use 'make fate-rsync SAMPLES=/path/to/samples' to sync the fate 
> suite"
> +endif
> +
>  .PHONY: documentation *test regtest-* alltools check config
> -- 
> 1.7.4.1
>

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to