[pacman-dev] [PATCH] Add 'silent-rules' to automake setup in configure.ac

[Dave Reisner]

On Wed, Dec 07, 2011 at 10:11:18AM -0600, Dan McGee wrote:
> On Wed, Dec 7, 2011 at 10:08 AM, Dave Reisner <d at falconindy.com> wrote:
> > On Wed, Dec 07, 2011 at 09:58:49AM -0600, Dan McGee wrote:
> >> This is awesome, and I don't know why we haven't already done this. It
> >> gives us the much more less verbose make output in a few different ways:
> >>
> >> * If you run `make V=0`, you will get the quiet output.
> >> * If you run `./configure --enable-silent-rules`, the quiet output is
> >>   the default; verbose output can be had by passing V=1 to make.
> >>
> >>     make[3]: Entering directory `/home/dmcgee/projects/pacman/lib/libalpm'
> >>       CC     add.lo
> >>       CC     be_local.lo
> >>       CC     be_package.lo
> >>       CC     be_sync.lo
> >>       CC     delta.lo
> >>     .....
> >>
> >> Signed-off-by: Dan McGee <dan at archlinux.org>
> >> ---
> >>  configure.ac |    2 +-
> >>  1 files changed, 1 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/configure.ac b/configure.ac
> >> index d4ec677..39e34c8 100644
> >> --- a/configure.ac
> >> +++ b/configure.ac
> >> @@ -57,7 +57,7 @@ AC_CONFIG_SRCDIR([config.h.in])
> >>  AC_CONFIG_HEADERS([config.h])
> >>
> >>  AC_CANONICAL_HOST
> >> -AM_INIT_AUTOMAKE
> >> +AM_INIT_AUTOMAKE([silent-rules])
> >>
> >>  LIB_VERSION=`expr lib_current - lib_age`.lib_age.lib_revision
> >>  LIB_VERSION_INFO="lib_current:lib_revision:lib_age"
> >> --
> >> 1.7.8
> >>
> >>
> >
> > Oh... I figured you knew about this and didn't give a shit. Yay for
> > enabling this!!!
> 
> Haha, definitely did not- I would have at least put it in the
> configure file so people COULD use it if they wanted, since it still
> isn't on by default.
> 
> -Dan
> 

Honestly, I'd go one step further and enable it by default, since 'make
V=1' is always available ;). We'll see warnings and errors regardless.

d