From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/666 Path: news.gmane.org!not-for-mail From: Rich Felker Newsgroups: gmane.linux.lib.musl.general Subject: Re: [PATCH] makefile: Make default output compact and similar to Linux kernel. Date: Fri, 16 Mar 2012 18:55:45 -0400 Message-ID: <20120316225545.GB22075@brightrain.aerifal.cx> References: <1331799923-14402-1-git-send-email-gf@unixsol.org> <20120315164209.GE184@brightrain.aerifal.cx> <20120316110023.bc4724d8.idunham@lavabit.com> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: dough.gmane.org 1331938536 26160 80.91.229.3 (16 Mar 2012 22:55:36 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Fri, 16 Mar 2012 22:55:36 +0000 (UTC) To: musl@lists.openwall.com Original-X-From: musl-return-667-gllmg-musl=m.gmane.org@lists.openwall.com Fri Mar 16 23:55:35 2012 Return-path: Envelope-to: gllmg-musl@plane.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by plane.gmane.org with smtp (Exim 4.69) (envelope-from ) id 1S8g3Z-0008Fa-U9 for gllmg-musl@plane.gmane.org; Fri, 16 Mar 2012 23:55:34 +0100 Original-Received: (qmail 15996 invoked by uid 550); 16 Mar 2012 22:55:33 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Original-Received: (qmail 15988 invoked from network); 16 Mar 2012 22:55:32 -0000 Content-Disposition: inline In-Reply-To: <20120316110023.bc4724d8.idunham@lavabit.com> User-Agent: Mutt/1.5.21 (2010-09-15) Xref: news.gmane.org gmane.linux.lib.musl.general:666 Archived-At: On Fri, Mar 16, 2012 at 11:00:23AM -0700, Isaac Dunham wrote: > On Thu, 15 Mar 2012 12:42:09 -0400 > Rich Felker wrote: > > > On Thu, Mar 15, 2012 at 10:25:23AM +0200, Georgi Chorbadzhiyski wrote: > > > This patch changes default make output to be similar to Linux kernel. > > > An example output looks like this: > > > CC src/unistd/write.c > > > LD lib/libc.so > > > BUILD tools/musl-gcc > > > INSTALL /usr/local/musl/lib/libc.a > ... > > > > Any opinions on this? I really dislike this kind of output, but if > > there's a demand for it I'll consider it. > > > On the plus side, it makes spotting warnings/errors easier. > It does look a little cleaner, as well. For warnings, I agree. For errors, they should stop compilation anyway. As for myself, I always build with -Werror but with a carefully tuned set of warning options that disable a number of warnings which I consider broken by design (e.g. ones that necessarily have false positives that can't be "fixed", like unused function arguments) or which impose style rules I disagree with. :-) > On the down side > -it complicates makefiles Indeed. A feature like this really belongs in make itself (e.g. make --pretty or something) rather than in every single program's makefile. > -it hides what's happening (and often the flags gcc gets are the > biggest problem) With most programs, the reason I dislike the "V=0" style is that it makes it impossible to re-run just the command that failed after changing something in the source, or to rerun it with slightly different compiler options. Often re-running make will recurse through 10 different directories before getting back to where it stopped, and of course the situation is even worse trying to find where in the makefile to override CFLAGS for just one source file (e.g. when gcc is OOM'ing building some gigantic file with -O3 and you want to retry with -O0 just to get a working build). For musl this is not such a big deal since the makefile is sane. > -you can review the errors in full context later with > make 2>&1 | tee errors.log - > > So my vote is that it isn't worth the effort. My view is that it wouldn't be as harmful in musl as in many other projects, but I still find it mildly annoying at best and slightly problematic at worst. Rich