From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from omta002.cacentral1.a.cloudfilter.net (omta002.cacentral1.a.cloudfilter.net [3.97.99.33]) by sourceware.org (Postfix) with ESMTPS id 8DB573858D28 for ; Wed, 8 Dec 2021 06:26:31 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 8DB573858D28 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=SystematicSw.ab.ca Authentication-Results: sourceware.org; spf=none smtp.mailfrom=systematicsw.ab.ca Received: from shw-obgw-4004a.ext.cloudfilter.net ([10.228.9.227]) by cmsmtp with ESMTP id uhuZmbMo8ztEjuqPPmoYka; Wed, 08 Dec 2021 06:26:31 +0000 Received: from [192.168.1.105] ([68.147.0.90]) by cmsmtp with ESMTP id uqPOmmCjUd5UnuqPOm7fIT; Wed, 08 Dec 2021 06:26:31 +0000 X-Authority-Analysis: v=2.4 cv=FrgWQknq c=1 sm=1 tr=0 ts=61b05017 a=T+ovY1NZ+FAi/xYICV7Bgg==:117 a=T+ovY1NZ+FAi/xYICV7Bgg==:17 a=IkcTkHD0fZMA:10 a=mDV3o1hIAAAA:8 a=TImcKGuyeGIbufSLrCcA:9 a=QEXdDO2ut3YA:10 a=9S8CEPD2kXIA:10 a=GXUAyPBqdsIA:10 a=_FVE-zBwftR9WsbkzFJk:22 Message-ID: <19c1cb99-6f8c-d0d0-7f1d-406c2682c0ff@SystematicSw.ab.ca> Date: Tue, 7 Dec 2021 23:26:30 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.2 Reply-To: newlib@sourceware.org Subject: Re: [PATCH] Enable automake silent rules Content-Language: en-CA To: newlib@sourceware.org References: <20211207125624.5465-1-jon.turney@dronecode.org.uk> From: Brian Inglis Organization: Systematic Software In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4xfKD46d/BZypfaqcHF/kWPSTj6jvwpUQzWSIOh0N/bdY6G5bJiBtd71uoi0CHRTSpEgqgu0bkljoxVIJSIqfh3bLGMYpH7HX8tpI5rwKWEUMB+GtBJT9I kl5ZX7GiqsJUWEvpEYhAqH/Jz9j1jgPgWHrDFxTcTYh9mZvv8FPB3lxqBKmzhBIiedH1GKNA6c0UBKckCF+/tVW/b3D8sEF1e+8= X-Spam-Status: No, score=-1165.2 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KAM_SHORT, NICE_REPLY_A, RCVD_IN_BARRACUDACENTRAL, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: newlib@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Newlib mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Dec 2021 06:26:33 -0000 On 2021-12-07 13:03, C Howland wrote: > On Tue, 7 Dec 2021 at 14:51, Mike Frysinger wrote: >> On 07 Dec 2021 11:10, C Howland wrote: >>> Should this really be made the new default? >> yes >>> I personally don't want it as a default because I want to see >>> command line arguments. Easy for me to get different, but only >>> knowing how. That leads to the next question. >> then use the standard `make V=1` or `./configure --disable-silent-rules` Does "$ export V=1" provide the OP's preferred behaviour? >>> Given that silent were made default, it is a big change in terms >>> of what people see. How would you know to change it? (New >>> feature, most people are far from experts in automake.) >>> Therefore, the directions most definitely would need something >>> added that explains it. (Without knowing about it, I can see it >>> taking someone a long time to chase down why they suddenly lost >>> the make output.) Something along the lines of Jon's explanation >>> in the patch email needs to end up in README and probably also >>> printed by a configure help. >> you can make this argument about any change. we shouldn't be paralyzed by >> historical behavior and never make progress. the automake manual provides >> background info about why chatty make is not a good thing. > I did not at all say that we should not do it. I just pointed out there > are reasons some people might not want it as the new default, while > acknowledging that others might (e.g you and Corinna) and that it is a very > visible one that needs additional consideration. >> https://www.gnu.org/software/automake/manual/automake.html#Silencing-Make >> the option is standard in the GNU ecosystem and adoption rate is only going >> up. it's already in the --help of configure. no one has to know or be an >> expert in automake to find it. > OK, good. But this underscores the need to know something for a change > for such a large impact. (Why would one look under configure when make > output has a very large change?) >> adding the option to the newlib/README sounds reasonable since it already >> has a bunch of these. want to send a patch ? > The whole point of my response is that a README entry should be part of the > patch to begin with--you should be supplying it. (I didn't know about the > help in configure, what else would I miss?) Try "$ less +/^Optional .../configure" to see Optional Features --enable-/--disable- and Optional Packages --with-/--without- where available and supported. As with the best configure, build, and test suites, most maintainers would ideally like to see an options summary, build and test command summaries e.g CC c.c/CCLD/GEN/PASS/FAIL c to show progress, with the full commands and paths on any failures, and a results summary. -- Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada This email may be disturbing to some readers as it contains too much technical detail. Reader discretion is advised. [Data in binary units and prefixes, physical quantities in SI.]