Date: Tue, 4 Jun 2019 12:22:25 -0700 From: Enji Cooper <yaneurabeya@gmail.com> To: rgrimes@freebsd.org Cc: Maxim Sobolev <sobomax@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r348540 - head Message-ID: <059B57C6-874A-4E27-A7A7-4EEBDE45EA66@gmail.com> In-Reply-To: <201906031906.x53J6mPi044273@gndrsh.dnsmgr.net> References: <201906031906.x53J6mPi044273@gndrsh.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Jun 3, 2019, at 12:06 PM, Rodney W. Grimes = <freebsd@gndrsh.dnsmgr.net> wrote: >=20 >> Author: sobomax >> Date: Mon Jun 3 15:34:00 2019 >> New Revision: 348540 >> URL: https://svnweb.freebsd.org/changeset/base/348540 >>=20 >> Log: >> Rollback the rest of the botched r348521. Re-work would be posted to >> reviews. Sorry. >>=20 >> Reported by: Enji Cooper >>=20 >> Modified: >> head/Makefile.inc1 >>=20 >> Modified: head/Makefile.inc1 >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- 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+=3D -W >> INSTALLFLAGS+=3D -h sha256 >> .endif >> .if defined(DB_FROM_SRC) || defined(NO_ROOT) >> -IMAKE_INSTALL=3D INSTALL=3D"${INSTALL} ${INSTALLFLAGS}" >> +IMAKE_INSTALL=3D INSTALL=3D"install ${INSTALLFLAGS}" >=20 > This is a correct change, > ngie please explain exactly why this is an exception to = INSTALL?=3Dinstall. There are edge cases I=E2=80=99ve had to work around in the = build system in the past, where using $INSTALL would not work here, = e.g., use =E2=80=9CINSTALL=3Dtrue=E2=80=9D. This would break that use. There=E2=80=99s also absolutely no guarantee that =E2=80=9C$INSTAL= L=E2=80=9D knows how to interpret "$INSTALLFLAGS=E2=80=9D; FreeBSD=E2=80=99= s install(1) however, used during make distribution, installworld, = installkernel, etc, does support these flags. The same goes for mtree as well; nmtree isn=E2=80=99t the same = as the older version of mtree that=E2=80=99s on older (pre-9.x) systems = (IIRC). It=E2=80=99s better to use known build/install tools to avoid = maintenance/debugging burden. Thanks, -Enji=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?059B57C6-874A-4E27-A7A7-4EEBDE45EA66>