Date: Thu, 31 May 2001 17:53:17 +0300 From: Ruslan Ermilov <ru@FreeBSD.org> To: Bruce Evans <bde@zeta.org.au> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src UPDATING Message-ID: <20010531175317.B79754@sunbay.com> In-Reply-To: <Pine.BSF.4.21.0105312231160.25702-100000@besplex.bde.org>; from bde@zeta.org.au on Thu, May 31, 2001 at 10:55:20PM %2B1000 References: <8666eil46i.wl@cheerful.com> <Pine.BSF.4.21.0105312231160.25702-100000@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, May 31, 2001 at 10:55:20PM +1000, Bruce Evans wrote:
[...]
> -c is now the default for install(1), so both -c and ${COPY} have no
> effect unless ${COPY} is abused for something other than the user's
> preference for copying.
>
> I think the correct fix is to remove all instances of -c and COPY. COPY
> is a FreeBSD thing. It was added by rgrimes in FreeBSD-1.x. The .mk
> files in 4.4BSD-Lite2 use plain "install" for most generated files, so
> the default in 4.4BSD-Lite2 was to remove most generated files after
> installing them. I think the default for COPY was always -c in FreeBSD,
> and the only reason to have a macro for this was to support bug for bug
> compatible behaviour.
>
I fully agree. Since -c behavior is now the default, we can really drop
support for it completely from share/mk stuff and src/ Makefiles. But we
still need some way to pass -C globally for file-type installations.
How about if we replace the COPY knob with INSTALL_FILE (empty or -c by
default) to replace "INSTALL=install -C" /etc/defaults/make.conf hack?
Cheers,
--
Ruslan Ermilov Oracle Developer/DBA,
ru@sunbay.com Sunbay Software AG,
ru@FreeBSD.org FreeBSD committer,
+380.652.512.251 Simferopol, Ukraine
http://www.FreeBSD.org The Power To Serve
http://www.oracle.com Enabling The Information Age
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010531175317.B79754>