[MPlayer-cvslog] r26467 - trunk/Makefile

Diego Biurrun diego at biurrun.de
Sat Apr 19 20:55:00 CEST 2008


On Sat, Apr 19, 2008 at 07:39:41PM +0300, Uoti Urpala wrote:
> On Sat, 2008-04-19 at 11:53 +0200, diego wrote:
> > Add dependency information to recursive rules. While more eager than strictly
> > necessary, this should err on the side of unneeded recursion instead of
> > missing a necessary rebuild.
> 
> > +libmpcodecs/libmpcodecs.a: libmpcodecs/*.[ch] libmpcodecs/*/*.[ch]
> 
> This goes back to MPlayer's old unreliable system which was abandoned in
> favor of unconditional recursion. Files can depend on headers in other
> directories. For example almost all files in libmpcodecs/ depend on
> libmpdemux/stheader.h. If you don't have specific dependencies then you
> must depend on any header file _anywhere_; in practice the recursion
> must be unconditional.

The problem with unconditional recursion was that the libraries were
rebuilt, but the binaries were not relinked.  This problem is now fixed.
I'll make the whole build system non-recursive step by step.  Then these
cross-directory dependency problems will be resolved.

Diego



More information about the MPlayer-cvslog mailing list