* bug#19501: Default to 'configure --enable-silent-rules'
@ 2015-01-04 8:16 Paul Eggert
[not found] ` <handler.19501.B.142035941412016.ack@debbugs.gnu.org>
0 siblings, 1 reply; 2+ messages in thread
From: Paul Eggert @ 2015-01-04 8:16 UTC (permalink / raw)
To: 19501
[-- 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
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-01-11 9:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-04 8:16 bug#19501: Default to 'configure --enable-silent-rules' Paul Eggert
[not found] ` <handler.19501.B.142035941412016.ack@debbugs.gnu.org>
2015-01-11 9:22 ` Paul Eggert
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.