Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 14 Jan 1997 17:54:48 +1100
From:      Bruce Evans <bde@zeta.org.au>
To:        bde@zeta.org.au, jkh@time.cdrom.com
Cc:        cvs-all@freefall.freebsd.org, CVS-committers@freefall.freebsd.org, cvs-gnu@freefall.freebsd.org, jkh@freefall.freebsd.org
Subject:   Re: cvs commit: src/gnu/usr.bin/texinfo/install-info Makefile
Message-ID:  <199701140654.RAA27339@godzilla.zeta.org.au>

next in thread | raw e-mail | index | archive | help
>> Anywhere early is reasonable.
>
>I must respectfully disagree.

There are many precedents for it.  The include and lib dirs are installed
first so that their headers can be perlified (this is broken for headers
installed from other directories).

>/usr/src/Makefile has two properties I
>don't want to screw up here, both of which would be casualties of this
>change:
>
>	1. The SUBDIR list is in alphabetical order, making it easy to
>	   find out if something has been omitted or not.

It's not in alphabetical order.  Apart from include and lib, eBones,
secure, and lkm are out of order for no apparent reason, and etc,
local and ports are out of order for bootstrapping.

>	2. The subdirs listed are all top level dirs in /usr/src.  If I
>	   were to add share/info to the list, it would get traversed
>	   *twice* - once at the beginning and once again during share
>	   (and your change doesn't respect NOSHARE, as the subsequent
>	   reference does).

Installing it twice takes 0.58 seconds for the first time and 0.28
seconds for the second time here :-).  NOSHARE is almost useless, since
many man and info pages are installed in /usr/share.  Installing the
dir file as well as the info pages won't hurt unless the source directory
is missing, and if it is missing then your change is equally broken:

	@(cd /usr/src/share/info; make install)

This has several other bugs:
- absolute pathname for the directory.  This is difficult to fix, since it
  is hard to determine the root of the source tree, if any.  (If it could
  be found easily, then it would already be used to locate non-absolute
  versions of *.mk.)
- `make' should be ${MAKE}.

>That's what I meant when I said that the proposed cure was worse than
>the disease.  I don't want to needlessly obfuscate /usr/src/Makefile

I agree that your proposed cure is worse :-).

Bruce



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199701140654.RAA27339>