[PATCH] Use automake silent build mode only if supported

2010-09-20 Thread Tommi Rantala
Fixes build for those of us who are stuck with an old version of
automake.

* configure.ac (AM_INIT_AUTOMAKE, AM_SILENT_RULES): Remove silent-rules
from AM_INIT_AUTOMAKE, and instead call AM_SILENT_RULES macro if it is
defined.

Signed-off-by: Tommi Rantala 
---
 configure.ac |4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index 8a1c9da..77bc7d0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4,10 +4,12 @@ AC_INIT([strace],[4.5.20])
 AC_CONFIG_SRCDIR([strace.c])
 AC_CONFIG_AUX_DIR([.])
 AM_CONFIG_HEADER([config.h])
-AM_INIT_AUTOMAKE([foreign check-news dist-bzip2 no-dist-gzip silent-rules])
+AM_INIT_AUTOMAKE([foreign check-news dist-bzip2 no-dist-gzip])
 AM_MAINTAINER_MODE
 AC_CANONICAL_HOST
 
+m4_ifdef([AM_SILENT_RULES],[AM_SILENT_RULES([yes])])
+
 AC_MSG_CHECKING([for supported operating system])
 case "$host_os" in
 *linux*)
-- 
1.7.3.rc0.6.g7505a


--
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: [PATCH] Use automake silent build mode only if supported

2010-09-20 Thread Roland McGrath
Frankly I'd rather just require the appropriate automake version.

--
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel


Re: [PATCH] Use automake silent build mode only if supported

2010-09-20 Thread Dmitry V. Levin
Hi,

On Mon, Sep 20, 2010 at 03:04:43PM +0300, Tommi Rantala wrote:
> Fixes build for those of us who are stuck with an old version of
> automake.
> 
> * configure.ac (AM_INIT_AUTOMAKE, AM_SILENT_RULES): Remove silent-rules
> from AM_INIT_AUTOMAKE, and instead call AM_SILENT_RULES macro if it is
> defined.
[...]
> --- a/configure.ac
> +++ b/configure.ac
> @@ -4,10 +4,12 @@ AC_INIT([strace],[4.5.20])
>  AC_CONFIG_SRCDIR([strace.c])
>  AC_CONFIG_AUX_DIR([.])
>  AM_CONFIG_HEADER([config.h])
> -AM_INIT_AUTOMAKE([foreign check-news dist-bzip2 no-dist-gzip silent-rules])
> +AM_INIT_AUTOMAKE([foreign check-news dist-bzip2 no-dist-gzip])
>  AM_MAINTAINER_MODE
>  AC_CANONICAL_HOST
>  
> +m4_ifdef([AM_SILENT_RULES],[AM_SILENT_RULES([yes])])
> +

It was discussed in January already.
In short, initial suggestion submitted to the list was to add
m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])

My first idea was to add
m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES])
instead, thus keeping default behaviour unchanged.

Later we agreed to just require the automake version that
has the macro.


-- 
ldv


pgp4aHeHbLWch.pgp
Description: PGP signature
--
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev___
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel