Date: Wed, 16 Jul 2014 19:57:07 +0000 From: Brooks Davis <brooks@freebsd.org> To: Sean Fagan <sef@ixsystems.com> Cc: hackers@freebsd.org Subject: Re: Expanding on NO_ROOT: Categorizing installed files Message-ID: <20140716195707.GH60425@spindle.one-eyed-alien.net> In-Reply-To: <DFB8DF7E-B83E-455A-9F37-E31BB196DD14@ixsystems.com> References: <FE374F8B-ABF7-4497-B1C7-68E63612DC43@ixsystems.com> <20140716170758.GE60425@spindle.one-eyed-alien.net> <DFB8DF7E-B83E-455A-9F37-E31BB196DD14@ixsystems.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--orO6xySwJI16pVnm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jul 16, 2014 at 12:13:48PM -0700, Sean Fagan wrote: >=20 > On Jul 16, 2014, at 10:07 AM, Brooks Davis <brooks@freebsd.org> wrote: >=20 > > The vast majorify of the diff is make debugging garbage that looks like > > it was committed by accident. I won't provided any detailed review > > of the current patch except to say that there are a lot of apparently > > redundent instances of setting META_CATEGORY in Makefiles and still > > quite a lot of instances of .EXPORTVAR: META_CATEGORY. >=20 > Okay. Again, my apologies about that; I've excised it, and removed all > of the EXPORTVAR directives, even the ones that had been commented out. > (Note that the patch includes a change to xinstall, that is commented out= ; that's > because I think I still like the concept of it, to at least some degree, = and didn't > want to forget where I put it. But it *is* commented out, and so the fun= ctionality > is simply not there.) >=20 > Without my idiotic error, the diff is much smaller -- just under 50,000 b= ytes. However, > rather than include it again, I've replaced the file at http://earth.kith= rup.com/~sef/auto-diffs.txt > for everyone's perusal. Some comments with limited context: In Makefile.inc1 you write: LIB32WMAKEENV+=3D META_CATEGORY=3Dcompat32 Is that so downstream makefiles can modify it or should it be in LIB32WMAKEFLAGS? I don't think bin/Makefile should need a META_CATEGORY line. Similarly lib/Makefile, libexec/Makefile, sbin/Makefile, secure/Makefile, ... cddl/Makefile.inc uses META_CATEGORY?=3D but many others use META_CATEGORY= =3D and I don't understand why which suggests some comments. gnu/usr.bin/cc/Makefile sets META_CATEGORY=3Ddev when ../Makefile.inc alrea= dy does. I think I'd put META_CATEGORY=3Ddev in lib/clang/clang.lib.mk rather than in the environment from lib/clang/Makefile. Setting META_CATAGORY in lib/csu/amd64/Makefile is probably not quite right. Since it needs to be base rather than lib I think setting it in lib/csu/Makefile.inc is probably correct. You'll also need to add -P (or what ever we decide on) to INSTALL there or patch all of lib/csu/*/Makefile. It doesn't look like the changes to lib/libc/Makefile are needed. Similarly lib/libelf/Makefile. There is debug stuff in rescue/Makefile. Writing: +.if exists(../Makefile.inc) +.include "../Makefile.inc" +.endif doesn't make sense. You can use .sinclude or just not bother to test since the directory shouldn't move around. Whitespace around =3D or ?=3D which assigning to META_CATEGORY is quite inc= onsistent. -- Brooks --orO6xySwJI16pVnm Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlPG2RIACgkQXY6L6fI4GtQYhQCeK9waqDmsPTrTDbauSp3fjyrU 1xsAoK6fPMKlUP0gsx/h7yVcOQLGswiO =a03v -----END PGP SIGNATURE----- --orO6xySwJI16pVnm--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140716195707.GH60425>