bug#19501: Default to 'configure --enable-silent-rules'

bug#19501: Default to 'configure --enable-silent-rules'

[Paul Eggert]

[-- Attachment #1: Type: text/plain, Size: 556 bytes --]

Tags: patch
Severity: wishlist

Attached is a patch to change the defaults for Emacs's build procedure to 
generate less output, to help builders and developers focus more easily on the 
diagnostics and warnings that are worth paying attention to.  For the shorter 
output, use 'make V=0' (the new default); for the traditional verbose output, 
use 'make V=1' (the old default).  This sort of thing has been used for some 
time for other GNU utilities, and seems to work fairly well.  I plan to install 
this into the Emacs master after some more testing.

[-- Attachment #2: 0001-Default-to-configure-enable-silent-rules.patch --]
[-- Type: text/x-diff, Size: 4128 bytes --]

From fe61981f51e02cf6da93c4ecdcac6db747ec445c Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Sun, 4 Jan 2015 00:09:17 -0800
Subject: [PATCH] Default to 'configure --enable-silent-rules'

This greatly shortens the 'make' output, making it more readable
and useful.  For example, on my platform it shortens a
4125-character line "gcc -std=gnu99 -c -Demacs -I. -I. -I../lib
... emacs.c" -- a line so long that it's hard to see what's going
on or where the diagnostics are -- to just "CC emacs.o".
* INSTALL, etc/NEWS: Document this.
* configure.ac: Add AM_SILENT_RULES([yes]).
(AM_DEFAULT_VERBOSITY): Remove now-unnecessary initialization.
---
 ChangeLog     | 10 ++++++++++
 INSTALL       |  7 +++----
 configure.ac  |  4 +++-
 etc/ChangeLog |  3 +++
 etc/NEWS      |  6 ++++--
 5 files changed, 23 insertions(+), 7 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 68d1391..624c65a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
 2015-01-04  Paul Eggert  <eggert@cs.ucla.edu>
 
+	Default to 'configure --enable-silent-rules'
+	This greatly shortens the 'make' output, making it more readable
+	and useful.  For example, on my platform it shortens a
+	4125-character line "gcc -std=gnu99 -c -Demacs -I. -I. -I../lib
+	... emacs.c" -- a line so long that it's hard to see what's going
+	on or where the diagnostics are -- to just "CC emacs.o".
+	* INSTALL: Document this.
+	* configure.ac: Add AM_SILENT_RULES([yes]).
+	(AM_DEFAULT_VERBOSITY): Remove now-unnecessary initialization.
+
 	Clarify 'make info'
 	* Makefile.in (info): Use GNU make conditional rather than
 	an '@' rule with a shell conditional, so that the builder can
diff --git a/INSTALL b/INSTALL
index 6850c7d..3d1a9a3 100644
--- a/INSTALL
+++ b/INSTALL
@@ -314,10 +314,9 @@ and is useful with GNU-compatible compilers.  On a recent GNU system
 there should be no warnings; on older and on non-GNU systems the
 generated warnings may still be useful.
 
-Use --enable-silent-rules to cause 'make' to chatter less.  This is
-helpful when combined with options like --enable-gcc-warnings that
-generate long shell-command lines.  'make V=0' also suppresses the
-chatter.
+Use --disable-silent-rules to cause 'make' to give more details about
+the commands it executes.  This can be helpful when debugging a build
+that goes awry.  'make V=1' also enables the extra chatter.
 
 Use --enable-link-time-optimization to enable link-time optimizer.  If
 you're using GNU compiler, this feature is supported since version 4.5.0.
diff --git a/configure.ac b/configure.ac
index e09b289..4cad214 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1011,9 +1011,11 @@ if test "${enableval}" != "no"; then
    fi
 fi)
 
+dnl Prefer silent make output.  For verbose output, use
+dnl 'configure --disable-silent-rules' or 'make V=1' .
+AM_SILENT_RULES([yes])
 dnl Port to Automake 1.11.
 dnl This section can be removed once we assume Automake 1.14 or later.
-: ${AM_DEFAULT_VERBOSITY=1}
 : ${AM_V=$AM_DEFAULT_VERBOSITY}
 : ${AM_DEFAULT_V=$AM_DEFAULT_VERBOSITY}
 AC_SUBST([AM_V])
diff --git a/etc/ChangeLog b/etc/ChangeLog
index c72c560..fe498ef 100644
--- a/etc/ChangeLog
+++ b/etc/ChangeLog
@@ -1,5 +1,8 @@
 2015-01-04  Paul Eggert  <eggert@cs.ucla.edu>
 
+	Default to 'configure --enable-silent-rules'
+	* NEWS: Document this.
+
 	batch write-region no longer says "Wrote FOO"
 	* NEWS: Document this.
 
diff --git a/etc/NEWS b/etc/NEWS
index ac42a9f..2ddb067 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -40,8 +40,10 @@ or by sticking with Emacs 24.4.
 ** The configure option `--with-pkg-config-prog' has been removed.
 Use './configure PKG_CONFIG=/full/name/of/pkg-config' if you need to.
 
-** The configure option '--enable-silent-rules' and the command
-'make V=0' now do a better job of suppressing chatter.
+** The configure option '--enable-silent-rules' is now the default.
+It and the command 'make V=0' now do a better job of suppressing chatter.
+To get the old behavior, configure with '--disable-silent-rules' or
+build with 'make V=1'.
 
 ---
 ** The `grep-changelog' script (and its manual page) are no longer included.
-- 
2.1.0

bug#19501: Default to 'configure --enable-silent-rules'

[Paul Eggert]

I installed the patch in the master as commit 
b995b4ae8f9e08719a92bbe3bd6ac7e8ff091309 and am marking this as done.