From owner-freebsd-hackers@FreeBSD.ORG Tue Jan 5 13:40:00 2010 Return-Path: Delivered-To: hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id F2F8D106566C for ; Tue, 5 Jan 2010 13:40:00 +0000 (UTC) (envelope-from david@catwhisker.org) Received: from bunrab.catwhisker.org (adsl-63-193-123-122.dsl.snfc21.pacbell.net [63.193.123.122]) by mx1.freebsd.org (Postfix) with ESMTP id A1F308FC1C for ; Tue, 5 Jan 2010 13:40:00 +0000 (UTC) Received: from bunrab.catwhisker.org (localhost [127.0.0.1]) by bunrab.catwhisker.org (8.13.3/8.13.3) with ESMTP id o05DTLQ5018890 for ; Tue, 5 Jan 2010 05:29:21 -0800 (PST) (envelope-from david@bunrab.catwhisker.org) Received: (from david@localhost) by bunrab.catwhisker.org (8.13.3/8.13.3/Submit) id o05DTLpM018889 for hackers@freebsd.org; Tue, 5 Jan 2010 05:29:21 -0800 (PST) (envelope-from david) Date: Tue, 5 Jan 2010 05:29:21 -0800 From: David Wolfskill To: hackers@freebsd.org Message-ID: <20100105132921.GN86359@bunrab.catwhisker.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="D5HQcwfjqNcvOcn2" Content-Disposition: inline User-Agent: Mutt/1.4.2.1i Cc: Subject: Patch to Makefile.inc1 to mention which kernel config is being installed X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Jan 2010 13:40:01 -0000 --D5HQcwfjqNcvOcn2 Content-Type: multipart/mixed; boundary="35KUG7rixYqtz4l+" Content-Disposition: inline --35KUG7rixYqtz4l+ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In the usual case where a machine is (only) used to build its own kernel, this is admittedly of no benefit. But for "build machines" that are used to build kernels for other machines (as well as themselves), it kinda bugged me that I was being told which kernels were being built, but not which one was being installed. Sure, I could recall that the first one built was being installed ... but why not have Makefile.inc1 tell us, just as it tells us which is being built? Before the patch, grepping through the typescript of a build for "^>>>" would yield: >>> Building an up-to-date make(1) >>> World build started on Mon Jan 4 04:47:37 PST 2010 >>> Rebuilding the temporary build tree >>> stage 1.1: legacy release compatibility shims >>> stage 1.2: bootstrap tools >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3: cross tools >>> stage 4.1: building includes >>> stage 4.2: building libraries >>> stage 4.3: make dependencies >>> stage 4.4: building everything >>> World build completed on Mon Jan 4 05:00:40 PST 2010 >>> Kernel build for GENERIC started on Mon Jan 4 05:00:40 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for GENERIC completed on Mon Jan 4 05:01:32 PST 2010 >>> Kernel build for ALBERT started on Mon Jan 4 05:01:32 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for ALBERT completed on Mon Jan 4 05:02:20 PST 2010 >>> Kernel build for JANUS started on Mon Jan 4 05:02:20 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for JANUS completed on Mon Jan 4 05:03:07 PST 2010 >>> Installing kernel >>> Making hierarchy >>> Installing everything >>> Removing old files (only deletes safe to delete libs) >>> Old files removed >>> Removing old directories >>> Old directories removed >>> stage 2.2: rebuilding the object tree >>> stage 4.4: building everything >>> Removing old libraries >>> Old libraries removed After the patch: >>> Building an up-to-date make(1) >>> World build started on Tue Jan 5 04:48:19 PST 2010 >>> Rebuilding the temporary build tree >>> stage 1.1: legacy release compatibility shims >>> stage 1.2: bootstrap tools >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3: cross tools >>> stage 4.1: building includes >>> stage 4.2: building libraries >>> stage 4.3: make dependencies >>> stage 4.4: building everything >>> World build completed on Tue Jan 5 05:01:26 PST 2010 >>> Kernel build for GENERIC started on Tue Jan 5 05:01:26 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for GENERIC completed on Tue Jan 5 05:03:28 PST 2010 >>> Kernel build for ALBERT started on Tue Jan 5 05:03:28 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for ALBERT completed on Tue Jan 5 05:05:12 PST 2010 >>> Kernel build for JANUS started on Tue Jan 5 05:05:12 PST 2010 >>> stage 1: configuring the kernel >>> stage 2.2: rebuilding the object tree >>> stage 2.3: build tools >>> stage 3.1: making dependencies >>> stage 3.2: building everything >>> Kernel build for JANUS completed on Tue Jan 5 05:06:53 PST 2010 >>> Installing kernel GENERIC >>> Making hierarchy >>> Installing everything >>> Removing old files (only deletes safe to delete libs) >>> Old files removed >>> Removing old directories >>> Old directories removed >>> stage 2.2: rebuilding the object tree >>> stage 4.4: building everything >>> Removing old libraries >>> Old libraries removed Attached patch is against head; for the above, I had patched stable/7. Thoughts? Peace, david --=20 David H. Wolfskill david@catwhisker.org Depriving a girl or boy of an opportunity for education is evil. See http://www.catwhisker.org/~david/publickey.gpg for my public key. --35KUG7rixYqtz4l+ Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="Makefile.inc1.diff" Index: Makefile.inc1 =================================================================== --- Makefile.inc1 (revision 201492) +++ Makefile.inc1 (working copy) @@ -817,7 +817,7 @@ false .endif @echo "--------------------------------------------------------------" - @echo ">>> Installing kernel" + @echo ">>> Installing kernel ${INSTALLKERNEL}" @echo "--------------------------------------------------------------" cd ${KRNLOBJDIR}/${INSTALLKERNEL}; \ ${CROSSENV} PATH=${TMPPATH} \ --35KUG7rixYqtz4l+-- --D5HQcwfjqNcvOcn2 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.0 (FreeBSD) iEYEARECAAYFAktDPrAACgkQmprOCmdXAD00agCdEDkCxBRGnB0XEp+hWMjoyAwa zmsAnjvqPMpE+Rk/T9SZK3f0w242HWGn =j3ae -----END PGP SIGNATURE----- --D5HQcwfjqNcvOcn2--