Re: Recursive makefiles

2013-08-29 Thread Andreas Enge
On Thu, Aug 29, 2013 at 10:04:10PM +0200, Ludovic Courtès wrote: > I just did that in commit a9424c0 (already pushed). I think it does > the right thing, but if it doesn’t, let’s discuss it. :-) It works well for me, and the patch fixing the size of the graph is also a nice addition. Thanks! An

Re: Recursive makefiles

2013-08-29 Thread Ludovic Courtès
Andreas Enge skribis: > On Thu, Aug 29, 2013 at 12:21:47AM +0200, Ludovic Courtès wrote: >> Hmm, there was no Makefile in doc/, so how could “cd doc; make pdf” work? > > Sorry, indeed, it did not work of course. "make doc/guix.pdf". > >> With that, “make pdf” at the top level creates doc/guix.pdf

Re: Recursive makefiles

2013-08-28 Thread Ludovic Courtès
Andreas Enge skribis: > On Wed, Aug 28, 2013 at 11:09:40PM +0200, Ludovic Courtès wrote: >> Please don’t do that: the build system purposefully avoids recursive >> makefiles, to improve parallelism etc. (see >> <http://miller.emu.id.au/pmiller/books/rmch/>.) > >

Re: Recursive makefiles

2013-08-28 Thread Andreas Enge
On Thu, Aug 29, 2013 at 12:21:47AM +0200, Ludovic Courtès wrote: > Hmm, there was no Makefile in doc/, so how could “cd doc; make pdf” work? Sorry, indeed, it did not work of course. "make doc/guix.pdf". > With that, “make pdf” at the top level creates doc/guix.pdf, and leaves > no intermediate f

Re: Recursive makefiles

2013-08-28 Thread Andreas Enge
On Wed, Aug 28, 2013 at 11:09:40PM +0200, Ludovic Courtès wrote: > Please don’t do that: the build system purposefully avoids recursive > makefiles, to improve parallelism etc. (see > <http://miller.emu.id.au/pmiller/books/rmch/>.) I did it for two reasons: - It allows to type &quo

Recursive makefiles

2013-08-28 Thread Ludovic Courtès
ctory doc. Please don’t do that: the build system purposefully avoids recursive makefiles, to improve parallelism etc. (see <http://miller.emu.id.au/pmiller/books/rmch/>.) However, the snippet about the ‘doc’ directory could be moved in, say, doc.am, and have doc.am included from Makefile.am. WDYT? Thanks, Ludo’.