Re: [STATUS] 1.3b1 Thu Aug 28 18:36:36 EDT 1997

[Randy Terbush <ra...@zyzzyva.com>]

This patch looks good. +1


> >From paul@ukweb.com Sat Aug 30 14:42:20 1997
> Date: Sun, 24 Aug 1997 13:21:46 +0100 (BST)
> From: Paul Sutton <pa...@ukweb.com>
> Reply-To: new-httpd@apache.org
> To: new-httpd@hyperreal.org
> Subject: Make libraries; fix dependencies
> 
> Here is an updated version of my patch to fix the dependencies between the
> top level (src) Makefile and the object files in the sub dirs (core, os/*,
> modules/*). Currently src/Makefile knows the names of all the object files
> in core, os/* and modules/standard and makes httpd dependent on them.
> Doing a "make httpd" builds the object file *in the src directory*, which
> is wrong. 
> 
> This patch fixes this by building core, os/* and modules/standard object
> files into a library. src/Makefile only knows about the library files, and
> doesn't care about the individual object files. To rebuild the libraries
> when the object files change src/Makefile always does a make in os/*, core
> and modules/standard. (For modules/standard, read "any modules/* directory
> where Configure auto-creates a makefile). 
> 
> The advantages of this patch are
> 
>  (a) src/Makefile does not need to repeat information that only
>      sub-directory makefiles should know, such as object file names
>  (b) doing "make httpd" now works
>  (c) the os specific directory (os/unix) can contain multiple object
>      files since src/Makefile does not need to know about them
>  (d) it is now easier to add additional subdirectories if necessary
>      by adding them to the SUBDIRS macro
>  (e) we can add additional recursive targets such as depend, distclean
>      etc with rules such as "depend distclean clean :: ...."
> 
> The disadvantages are
> 
>  (a) ordering of the .a files on the link line may be important
>  (b) httpd always gets relinked on doing a make at the top level
> 
> I think this is a useful clean up, fixes some problems with the current
> makefile, and reduces redundant information between src/Makefile and the
> lower level makefiles, so I give it +1. 
> 
> //pcs