On Tue, May 03, 2011 at 17:59:53 (CEST), Måns Rullgård wrote:

> 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.

Changed
>
>> +    rsync -vaLW rsync://fate-suite.libav.org:/fate-suite/ $(SAMPLES)
>
> Why the extra : ?

removed

>> +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
>>

patch pushed

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to