Date: Thu, 30 May 2013 10:45:28 -0500 From: Brooks Davis <brooks@freebsd.org> To: Konstantin Belousov <kostikbel@gmail.com> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, John Baldwin <jhb@freebsd.org> Subject: Re: svn commit: r251140 - head Message-ID: <20130530154528.GC36644@lor.one-eyed-alien.net> In-Reply-To: <20130530153750.GB3047@kib.kiev.ua> References: <201305301409.r4UE9wSP009396@svn.freebsd.org> <201305301053.23282.jhb@freebsd.org> <20130530152933.GB36644@lor.one-eyed-alien.net> <20130530153750.GB3047@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
--f0KYrhQ4vYSV2aJu Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 30, 2013 at 06:37:51PM +0300, Konstantin Belousov wrote: > On Thu, May 30, 2013 at 10:29:33AM -0500, Brooks Davis wrote: > > On Thu, May 30, 2013 at 10:53:23AM -0400, John Baldwin wrote: > > > On Thursday, May 30, 2013 10:09:58 am Brooks Davis wrote: > > > > Author: brooks > > > > Date: Thu May 30 14:09:58 2013 > > > > New Revision: 251140 > > > > URL: http://svnweb.freebsd.org/changeset/base/251140 > > > >=20 > > > > Log: > > > > Always define INSTALL_DDIR and define it such that it contains no= extra > > > > / characters rather than removing them later on. This should fix > > > > release builds. > > > > =20 > > > > PR: conf/178963 > > > > Reviewed by: gjb, hrs > > > >=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 Thu May 30 14:07:14 2013 (r251139) > > > > +++ head/Makefile.inc1 Thu May 30 14:09:58 2013 (r251140) > > > > @@ -522,7 +523,7 @@ _includes: > > > > @echo "----------------------------------------------------------= ----" > > > > @echo ">>> stage 4.1: building includes" > > > > @echo "----------------------------------------------------------= ----" > > > > - ${_+_}cd ${.CURDIR}; ${WMAKE} SHARED=3Dsymlinks par-includes > > > > + ${_+_}cd ${.CURDIR}; ${WMAKE} par-includes > > > > _libraries: > > > > @echo > > > > @echo "----------------------------------------------------------= ----" > > >=20 > > > Did you mean to change this? It seems unrelated. > >=20 > > Oops, I forgot that I had that in for testing. Reverted in 251141. > >=20 > > That said, the SHARED=3D(copies|symlinks) code should probably go on the > > chopping block. It's a lousy idea except for populating WORLDTMP and > > here it saves at most 1.5% in /usr/obj. Worse, the implementation is > > hand coded in each Makefile so it adds complexity for a mostly useless > > feature. >=20 > I disagree with the statement about the useless, at least for WORLDTMP > it saves the full buildworld when a change in the usermode headers is > done during the development. E.g., changing include/* or sys/sys/* > still allows to build libc with buildenv, instead of spending 40+ > minutes on the buildworld. Ok, that's fair. I won't touch the includes bits. I seriously doubt 40+ minutes given that -DNO_CLEAN should work fine in such a case. Do you agree that it's less than useful in the never exercised cases in share/examples and share/sendmail? -- Brooks --f0KYrhQ4vYSV2aJu Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iD8DBQFRp3QXXY6L6fI4GtQRAuHnAKCJ9xdpsqEWnw1hWF+M8r7Gs21vWgCfRPl1 iPKZTWuL237Vcn8DE4Prf/U= =aGQY -----END PGP SIGNATURE----- --f0KYrhQ4vYSV2aJu--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130530154528.GC36644>