From owner-svn-src-all@freebsd.org Tue Jun 4 21:57:45 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 00EFA15BE39D; Tue, 4 Jun 2019 21:57:45 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4A73087AC5; Tue, 4 Jun 2019 21:57:43 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (localhost [127.0.0.1]) by gndrsh.dnsmgr.net (8.13.3/8.13.3) with ESMTP id x54LvfWi049943; Tue, 4 Jun 2019 14:57:41 -0700 (PDT) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.13.3/8.13.3/Submit) id x54LvfNV049942; Tue, 4 Jun 2019 14:57:41 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201906042157.x54LvfNV049942@gndrsh.dnsmgr.net> Subject: Re: svn commit: r348540 - head In-Reply-To: <059B57C6-874A-4E27-A7A7-4EEBDE45EA66@gmail.com> To: Enji Cooper Date: Tue, 4 Jun 2019 14:57:41 -0700 (PDT) CC: rgrimes@freebsd.org, Maxim Sobolev , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 4A73087AC5 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.96 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.96)[-0.960,0] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Jun 2019 21:57:45 -0000 > > On Jun 3, 2019, at 12:06 PM, Rodney W. Grimes wrote: > > > >> Author: sobomax > >> Date: Mon Jun 3 15:34:00 2019 > >> New Revision: 348540 > >> URL: https://svnweb.freebsd.org/changeset/base/348540 > >> > >> Log: > >> Rollback the rest of the botched r348521. Re-work would be posted to > >> reviews. Sorry. > >> > >> Reported by: Enji Cooper > >> > >> Modified: > >> head/Makefile.inc1 > >> > >> Modified: head/Makefile.inc1 > >> ============================================================================== > >> --- head/Makefile.inc1 Mon Jun 3 15:32:42 2019 (r348539) > >> +++ head/Makefile.inc1 Mon Jun 3 15:34:00 2019 (r348540) > >> @@ -887,7 +887,7 @@ MTREEFLAGS+= -W > >> INSTALLFLAGS+= -h sha256 > >> .endif > >> .if defined(DB_FROM_SRC) || defined(NO_ROOT) > >> -IMAKE_INSTALL= INSTALL="${INSTALL} ${INSTALLFLAGS}" > >> +IMAKE_INSTALL= INSTALL="install ${INSTALLFLAGS}" > > > > This is a correct change, > > ngie please explain exactly why this is an exception to INSTALL?=install. > > There are edge cases I?ve had to work around in the build system in the past, where using $INSTALL would not work here, e.g., use ?INSTALL=true?. This would break that use. This holds valid for all places in the three that use INSTALL today, anything that INSTALL is set to must be something that knows how to ignore the flags it may get. This holds true not only for INSTALL but also for any other like variable name. Furthermore true is infact one of those commands that ignores all args and all flags so how can your example break that I want this to do nothing, but not fail? > There?s also absolutely no guarantee that ?$INSTALL? knows how to interpret "$INSTALLFLAGS?; FreeBSD?s install(1) however, used during make distribution, installworld, installkernel, etc, does support these flags. > The same goes for mtree as well; nmtree isn?t the same as the older version of mtree that?s on older (pre-9.x) systems (IIRC). It?s better to use known build/install tools to avoid maintenance/debugging burden. It would be far far better to not have this 1 hard coded install and mtree, and arguing that it is about not parsing -flags is not going to hold water, as that is already true of all the other places that this is done. > Thanks, > -Enji -- Rod Grimes rgrimes@freebsd.org