[FFmpeg-cvslog] r11475 - trunk/Makefile

[Diego Biurrun]

On Mon, Jan 14, 2008 at 11:27:36PM +0000, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Thu, Jan 10, 2008 at 11:41:04AM +0100, Diego Biurrun wrote:
> >> On Wed, Jan 09, 2008 at 01:24:41AM +0000, M?ns Rullg?rd wrote:
> >> > Michael Niedermayer <michaelni at gmx.at> writes:
> >> > 
> >> > > On Wed, Jan 09, 2008 at 12:02:40AM +0100, diego wrote:
> >> > >> 
> >> > >> Log:
> >> > >> Add compilation rule for tools/trasher.
> >> > >> 
> >> > >> --- trunk/Makefile	(original)
> >> > >> +++ trunk/Makefile	Wed Jan  9 00:02:39 2008
> >> > >> @@ -113,6 +113,9 @@ tools/qt-faststart$(EXESUF): tools/qt-fa
> >> > >>  
> >> > >> +tools/trasher$(EXESUF): tools/trasher.c
> >> > >> +	$(CC) $(CFLAGS) $< -o $@
> >> > >
> >> > > may i protest against this ...
> >> > > i did try
> >> > > make tools/trasher before i svn added it and it did work without that
> >> > 
> >> > Yes, that is a builtin rule in make.
> >> 
> >> Mine is smaller ;-p
> >
> > Also, I don't think the builtin rule will take $(EXESUF) into account,
> 
> So write a new generic rule.  Something like this:
> 
> %$(EXESUF): %.c
> 	$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^

I've added a generic rule.  The binaries are slightly bigger, but I
don't think anybody really cares.

Diego