From owner-freebsd-arch@FreeBSD.ORG Fri Jan 30 16:33:17 2009 Return-Path: Delivered-To: arch@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C4CB8106567A; Fri, 30 Jan 2009 16:33:17 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id 544238FC3E; Fri, 30 Jan 2009 16:33:17 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.2/8.14.1) with ESMTP id n0UGUShh050004; Fri, 30 Jan 2009 09:30:28 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Fri, 30 Jan 2009 09:30:52 -0700 (MST) Message-Id: <20090130.093052.-2022808221.imp@bsdimp.com> To: obrien@FreeBSD.org, arch@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <20090130.085130.-4349483.imp@bsdimp.com> References: <200901130653.n0D6rrNX092719@svn.freebsd.org> <20090130015518.GA20404@hades.panopticon> <20090130.085130.-4349483.imp@bsdimp.com> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: Subject: Re: svn commit: r187132 - head/usr.bin/make X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Jan 2009 16:33:23 -0000 OMG! There's too much make output now, and the recent changes broke useful make features. All but one were eventually fixed. I just fixed the -s breakage. I'm planning on committing the following, if it passes code review and item #3 below. First, there was absolutely no reason to introduce -Q. -v is otherwise unused and matches the 'opt-in' debugging that's present in the rest of make and the build system and other utilities like cp and mv which will tell you what they are doing only if asked. Second, the extra always on debug introduces a performance penalty. Usually, one has to have good justification for doing this. Third, I'd like to collect votes on this. Do you like the new debugging, or would you rather see it just when asked. I'll let the majority rule as to whether to commit this patch. Please let me know privately. I'll tally the results and commit based on it. Finally, if there's any real, persuasive argument that should override the vote, I'd like to hear it. Thank you for your time. Warner Index: job.c =================================================================== --- job.c (revision 187843) +++ job.c (working copy) @@ -2362,7 +2362,7 @@ makeErrors = 0; lastNode = NULL; - if ((maxJobs == 1 && fifoFd < 0) || is_posix || beQuiet) { + if ((maxJobs == 1 && fifoFd < 0) || is_posix || !beVerbose) { /* * If only one job can run at a time, there's no need for a * banner, no is there? Index: main.c =================================================================== --- main.c (revision 187921) +++ main.c (working copy) @@ -126,7 +126,6 @@ Boolean mfAutoDeps; /* .MAKEFILEDEPS target seen */ Boolean beSilent; /* -s flag */ Boolean beVerbose; /* -v flag */ -Boolean beQuiet; /* -Q flag */ Boolean compatMake; /* -B argument */ int debug; /* -d flag */ Boolean ignoreErrors; /* -i flag */ @@ -517,11 +516,6 @@ printGraphOnly = TRUE; debug |= DEBUG_GRAPH1; break; - case 'Q': - beQuiet = TRUE; - beVerbose = FALSE; - MFLAGS_append("-Q", NULL); - break; case 'q': queryFlag = TRUE; /* Kind of nonsensical, wot? */ @@ -532,7 +526,7 @@ MFLAGS_append("-r", NULL); break; case 's': - beQuiet = TRUE; + beVerbose = FALSE; beSilent = TRUE; MFLAGS_append("-s", NULL); break; @@ -542,7 +536,6 @@ break; case 'v': beVerbose = TRUE; - beQuiet = FALSE; MFLAGS_append("-v", NULL); break; case 'x': Index: globals.h =================================================================== --- globals.h (revision 187843) +++ globals.h (working copy) @@ -76,7 +76,6 @@ extern Boolean ignoreErrors; /* True if should ignore all errors */ extern Boolean beSilent; /* True if should print no commands */ extern Boolean beVerbose; /* True if should print extra cruft */ -extern Boolean beQuiet; /* True if want quiet headers with -j */ extern Boolean noExecute; /* True if should execute nothing */ extern Boolean allPrecious; /* True if every target is precious */ extern Boolean is_posix; /* .POSIX target seen */