httpd-test-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Justin Erenkrantz <>
Subject Re: [PATCH] flood: install target
Date Tue, 10 Sep 2002 16:39:58 GMT
On Tue, Sep 10, 2002 at 07:04:08PM +0200, Jacek Prucia wrote:
> Here's my take at flood install target. Since I'm total automake newbie,
> you might want to take a closer look at this. It does what is desired --
> it copies flood to $PREFIX/bin. There's no special place for examples
> and DESIGN, CHANGES, CONFIG files in therms of autoconf, so they are
> left without instalation. Such files are often found in
> /usr/share/doc/$package_name/, but it is entirelly up to binary (deb,
> rpm) package maintainer.

No need to install them for now.  All we want to do is install the
flood binaries with the (possibly) in-tree APR/APR-util.

> This patch has one problem. Install dir is cluttered with lotsa apr/apu
> stuff (*-config scripts, includes, libraries) -- even when flood is
> built with --disable-shared. I've had enough of autoconf magic for
> today, so I'll tackle this tommorow (unless somebody else beats me
> first).

That's to be expected.

The only caveat is to build flood statically - rather than
dynamically (not sure if you can do that).  Yet, even if that
happens, you still need to let APR and APR-util install - and they
will install all of the ap{ru}-config files.

The proper way to get around that is to install APR and APR-util
on their own and pass --with-apr and --with-apr-util to configure.
If they use an intree APR/APR-util, this cruft is what they get.

> After reading Greg Stein post at dev@httpd, I think we schould roll
> flood-1.1 instead of flood-1.0, as Aaron suggested. FLOOD_1_0 lies in
> repo far to long to bump revisions for free. Some people might be pissed

Whatever. =)

We never released 1.0 or told a soul that it existed, so I don't
see a reason to be picky.

(*hears Aaron scream*)

> diff -urN flood.orig/ flood/
> --- flood.orig/	2002-07-22 07:48:02.000000000 +0200
> +++ flood/	2002-09-10 18:35:18.000000000 +0200
> @@ -5,7 +5,7 @@
>  dnl Override the default prefix with /pkg/flood-0.1
> -AC_PREFIX_DEFAULT(/pkg/flood-0.1)
> +AC_PREFIX_DEFAULT(/usr/local)

To be precise, I think we could change it to /usr/local/flood.  But,
we shouldn't install things to /usr/local by default.  As you can
tell, I refuse to use /usr/local or encourage its use in others.  =)

Otherwise, looks fine.  -- justin

View raw message