[FFmpeg-devel] [PATCH 2/2] Makefile: simplify test tools handling

Måns Rullgård mans
Fri Jan 21 16:54:39 CET 2011


Diego Biurrun <diego at biurrun.de> writes:

> On Fri, Jan 21, 2011 at 03:35:42PM +0000, M?ns Rullg?rd wrote:
>> Diego Biurrun <diego at biurrun.de> writes:
>> 
>> > On Thu, Jan 20, 2011 at 10:55:02PM +0000, Mans Rullgard wrote:
>> >> Signed-off-by: Mans Rullgard <mans at mansr.com>
>> >> ---
>> >>  Makefile |    5 +++--
>> >>  1 files changed, 3 insertions(+), 2 deletions(-)
>> >> 
>> >> diff --git a/Makefile b/Makefile
>> >> index 98ec5f5..a1fd7b4 100644
>> >> --- a/Makefile
>> >> +++ b/Makefile
>> >> @@ -16,7 +16,8 @@ MANPAGES    = $(PROGS-yes:%=doc/%.1)
>> >>  TOOLS       = $(addprefix tools/, $(addsuffix $(EXESUF), cws2fws pktdumper probetest qt-faststart trasher))
>> >> -HOSTPROGS   = $(addprefix tests/, audiogen videogen rotozoom tiny_psnr base64)
>> >> +TESTPROGS   = audiogen videogen rotozoom tiny_psnr base64
>> >> +HOSTPROGS  := $(TESTPROGS:%=tests/%)
>> >
>> > nit: maybe alphabetical order
>> >
>> >> @@ -162,7 +163,7 @@ testclean:
>> >>  	$(RM) tests/seek_test$(EXESUF) tests/seek_test.o
>> >> -	$(RM) $(addprefix tests/,$(addsuffix $(HOSTEXESUF),audiogen videogen rotozoom tiny_psnr base64))
>> >> +	$(RM) $(TESTPROGS:%=tests/%$(HOSTEXESUF))
>> >
>> > Why not use HOSTPROGS here?  The variable is currently unused...
>> 
>> No, it is modified in some of the lib* makefiles and used in
>> common.mak.  I had the same thought before I grepped it.
>
> OK, but still you can use the HOSTPROGS variable in the rm invocation.

No, because its value will have changed there.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list