From owner-svn-src-all@FreeBSD.ORG Fri Jan 30 15:54:28 2009 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A9C431065670; Fri, 30 Jan 2009 15:54:28 +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 71EAA8FC1D; Fri, 30 Jan 2009 15:54:28 +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 n0UFp4WD049530; Fri, 30 Jan 2009 08:51:05 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Fri, 30 Jan 2009 08:51:30 -0700 (MST) Message-Id: <20090130.085130.-4349483.imp@bsdimp.com> To: amdmi3@amdmi3.ru From: "M. Warner Losh" In-Reply-To: <20090130015518.GA20404@hades.panopticon> References: <200901130653.n0D6rrNX092719@svn.freebsd.org> <20090130015518.GA20404@hades.panopticon> 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: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, obrien@FreeBSD.org Subject: Re: svn commit: r187132 - head/usr.bin/make X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Jan 2009 15:54:29 -0000 In message: <20090130015518.GA20404@hades.panopticon> Dmitry Marakasov writes: : * David E. O'Brien (obrien@FreeBSD.org) wrote: : : I think this should be backed out. Those verbose messages: : - Are completely unexpected, this violates POLA. : - Do break recognizeable make output people are used to : - Really uglify make output for some custom makefiles (for example, : generated by cmake: : : --- po/it.gmo --- : [ 2%] Generating it.gmo : --- po/kn.gmo --- : [ 3%] Generating kn.gmo : --- po/lv.gmo --- : [ 3%] Generating lv.gmo : --- po/nb.gmo --- : [ 3%] Generating nb.gmo : : There's golden unix way rule: silence is golden. So please back this : out, as this will really annoy many people. This makes at least two requests... I hate them too. really really really hate them. -Q is default in all my trees. The real problem is that it exposes way too many internal target names that are totally baffling, even to me who has a lot of build experience. Also, it isn't clear how to use them. O'Brien says they were disabled in 1994 for no good reason without discussion, so he's turning them back on, without discussion. The project is a very different place than it was then, and doing this sort of thing is anti-social. Warner : > Author: obrien : > Date: Tue Jan 13 06:53:53 2009 : > New Revision: 187132 : > URL: http://svn.freebsd.org/changeset/base/187132 : > : > Log: : > Don't enable -Q by default - I've fixed the rescue build issue. : > : > Modified: : > head/usr.bin/make/job.c : > head/usr.bin/make/main.c : > : > Modified: head/usr.bin/make/job.c : > ============================================================================== : > --- head/usr.bin/make/job.c Tue Jan 13 06:52:51 2009 (r187131) : > +++ head/usr.bin/make/job.c Tue Jan 13 06:53:53 2009 (r187132) : > @@ -2362,8 +2362,7 @@ Job_Init(int maxproc) : > makeErrors = 0; : > : > lastNode = NULL; : > - : > - if ((maxJobs == 1 && fifoFd < 0) || beQuiet || beVerbose == 0) { : > + if ((maxJobs == 1 && fifoFd < 0) || is_posix || beQuiet) { : > /* : > * If only one job can run at a time, there's no need for a : > * banner, no is there? : > : > Modified: head/usr.bin/make/main.c : > ============================================================================== : > --- head/usr.bin/make/main.c Tue Jan 13 06:52:51 2009 (r187131) : > +++ head/usr.bin/make/main.c Tue Jan 13 06:53:53 2009 (r187132) : > @@ -126,7 +126,7 @@ Boolean is_posix; /* .POSIX target seen : > Boolean mfAutoDeps; /* .MAKEFILEDEPS target seen */ : > Boolean beSilent; /* -s flag */ : > Boolean beVerbose; /* -v flag */ : > -Boolean beQuiet = TRUE; /* -Q flag */ : > +Boolean beQuiet; /* -Q flag */ : > Boolean compatMake; /* -B argument */ : > int debug; /* -d flag */ : > Boolean ignoreErrors; /* -i flag */ : > @@ -519,6 +519,7 @@ rearg: : > break; : > case 'Q': : > beQuiet = TRUE; : > + beVerbose = FALSE; : > MFLAGS_append("-Q", NULL); : > break; : > case 'q': : > _______________________________________________ : > svn-src-all@freebsd.org mailing list : > http://lists.freebsd.org/mailman/listinfo/svn-src-all : > To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org" : : -- : Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D : amdmi3@amdmi3.ru ..: jabber: amdmi3@jabber.ru http://www.amdmi3.ru :