httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ryan Bloom <...@covalent.net>
Subject Re: [PATCH] allow statically linked support tools (htpasswd, etc)
Date Fri, 28 Sep 2001 03:51:28 GMT
On Thursday 27 September 2001 08:46 pm, Ryan Bloom wrote:
> > This allows the given executable to be statically linked with the
> > apache/apr libs. This was necessary to allow these executables to be
> > relocatable after build-time (for example when they were distributed in
> > binary form).
> >
> > Since these parameters generate variables that are specific to the
> > support/ directory, I've created a new config.m4 script in that
> > directory. (To apply, apply the patch and copy the config.m4 script into
> > support/)
> >
> > I've tested this patch on the following platforms:
> > Linux 2.4.3 (gcc 2.96 from RH7.1) i386
> > Solaris 8 (7/01) (gcc 2.95.2) i86pc
> > FreeBSD 5.0-CURRENT (gcc 2.95.3) i386
> > Index: configure.in
> > ===================================================================
> > RCS file: /home/cvspublic/httpd-2.0/configure.in,v
> > retrieving revision 1.182
> > diff -u -r1.182 configure.in
> > --- configure.in	2001/09/25 15:58:52	1.182
> > +++ configure.in	2001/09/28 02:07:35
> > @@ -257,6 +257,8 @@
> >    APR_ADDTO(CPPFLAGS, -DAP_DEBUG)
> >  ])dnl
> >
> > +sinclude(support/config.m4)
> > +
> >  APACHE_ENABLE_LAYOUT
> >  prefix="$orig_prefix"
> >  APACHE_ENABLE_MODULES
>
> In general, very good.  I'll be applying soon (like 15 minutes), but this
> is unnecessary.  The buildconf script does a find on the entire directory
> tree to find all config.m4 files, so we don't have to sinclude them.

That's unclear.  The configure.in patch is the part that is unnecessary.

Ryan
______________________________________________________________
Ryan Bloom				rbb@apache.org
Covalent Technologies			rbb@covalent.net
--------------------------------------------------------------

Mime
View raw message