On Tue, Jun 14, 2011 at 10:59:50AM +0100, Måns Rullgård wrote:
> Diego Biurrun <[email protected]> writes:
> 
> > On Tue, Jun 07, 2011 at 10:32:59AM +0100, Mans Rullgard wrote:
> >> Signed-off-by: Mans Rullgard <[email protected]>
> >> 
> >> --- a/Makefile
> >> +++ b/Makefile
> >> @@ -92,7 +92,6 @@ tools/%.o: tools/%.c
> >>    $(CC) $(CPPFLAGS) $(CFLAGS) -c $(CC_O) $<
> >>  
> >>  -include $(wildcard tools/*.d)
> >> --include $(wildcard tests/*.d)
> >>  
> >>  VERSION_SH  = $(SRC_PATH_BARE)/version.sh
> >>  GIT_LOG     = $(SRC_PATH_BARE)/.git/logs/HEAD
> >
> > I believe these two lines
> >
> > TESTTOOLS   = audiogen videogen rotozoom tiny_psnr base64
> > HOSTPROGS  := $(TESTTOOLS:%=tests/%)
> >
> > should also be moved to the subdirectory Makefile.
> 
> They need to stay there for now due to ordering requirements.

Yeah, I just noticed the interaction with common.mak.  Leaving
this behind is ugly but unavoidable I'm afraid, so patch OK.

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

Reply via email to