[344] in Hesiod
Re: PATCH: hesiod3.0.0
daemon@ATHENA.MIT.EDU (Greg Hudson)
Tue Dec 24 03:50:20 1996
Date: Tue, 24 Dec 1996 03:50:01 -0500
From: Greg Hudson <ghudson@MIT.EDU>
To: Larry Schwimmer <rosebud@cyclone.Stanford.EDU>
Cc: hesiod@MIT.EDU
In-Reply-To: "[341] in Hesiod"
> * distclean missed config.h
I will take this change, of course.
> * Install binaries so they can be stripped. Mode 755
> is pretty standard for packages.
I'm inclined to leave the modes as 555, since binaries are not
intended to be written to in most circumstances and operating systems
generally read-only files without the owner write bit set. (It's true
that third-party packages do tend to install binaries mode 755 and
data mode 644, but that's by no means universal.)
Ideally, there would be some convenient way to tell "make install" to
install with the -s option. The Gnu Coding Standards provide some
guidance in the form of INSTALL_PROGRAM and INSTALL_DATA, but I'm
reluctant to abide by those guidelines for a couple of reasons:
* In my experience, setting INSTALL_PROGRAM to include the -s
option tends to fail on many autoconf-using packages,
because they use INSTALL_PROGRAM to install scripts as well
as executable binaries. So it's not a universally useful
interface.
* It's not a very good interface in my opinion; asking the
installer to run "make install INSTALL_PROGRAM='install -c
-s'" doesn't appeal to me, especially since multi-word
command-line variable assignments don't work with recursive
makes in some versions of gmake. I would rather see
something of the form "make install STRIP=-s". (I'm not
even sure that INSTALL_PROGRAM was intended to be
overridable by the caller; the GCS only specifies that you
are to use INSTALL_PROGRAM
* The Gnu Coding Standards and autoconf tend to assume that
all files will be installed writable, a practice I disagree
with.
So, for lack of some good form of guidance I can agree with, I'm going
to wait at least for a while, and possibly communicate with the
autoconf maintainers, before making any changes in this area.
> * honor CFLAGS passed to configure.
Looking over the autoconf documentation, I think this is probably a
good idea (although as someone else commented, your configure.in
changes have problems). At our site, we want to compile everything
with -g, but that's a site-specific preference and we can take care of
it in our config.site file. I've taken your Makefile.in change
replacing "CDEBUGFLAGS=-g" with "CDEBUGFLAGS=@CFLAGS@".
