[PATCH] TOOLS: Makefile instead of c (was: Re: [MPlayer-dev-eng] documentation of TOOLS directory)

[Shachar Raindel]

On Tue, 3 Aug 2004 09:29:52 +0200 (CEST), VMiklos <mamajom at axelero.hu> wrote:
> On Tue, 3 Aug 2004, Diego Biurrun wrote:
> 
> > > I am just a sick, idiotic programmer, but don't you want to have a
> > > different target for each of the files, makeing all build only these
> > > which were changed (the way makefiles should be used), instead of
> > > making all of them rebuilt whenever the developer type make?
> >
> > Would something like the following be more to your taste?
> 
> imho this is what he and you extractly wished
> 
Just few comments: 

A. You want to put dependencies for each of the targets. i.e. (and
internal include files should go into this list as well):

asfinfo: asfinfo.c Makefile
       gcc asfinfo.c   -o asfinfo

avi-fix: avi-fix.c Makefile
       gcc avi-fix.c   -o avi-fix

B. IMHO, you want to split fastmemcpybench: target to a large number
of different targets, with the general rule that there should be
target for each file created on the hard disk. This allows make to do
better decisions about what it should recompile, and allows the
programmer who is only interested in the version to his CPU to build
only this version, without wasting the disk-space and time needed to
compile a version for each CPU mplayer is optimized for.

Just my 2 AGUROT (1/100 of NIS) of common Makefiles insanity (why
don't we write mplayer in FreePascal? It doesn't require Makefiles ;-)
)

    Cheers,
    Shachar