From mboxrd@z Thu Jan  1 00:00:00 1970
From: compudj@krystal.dyndns.org (Mathieu Desnoyers)
Date: Tue, 29 Nov 2011 19:17:27 -0500
Subject: [lttng-dev] [lttng-tools PATCH 1/2] Cleanup configure.ac
In-Reply-To: <1322611444-19408-1-git-send-email-alexandre.montplaisir@gmail.com>
References: <1322611444-19408-1-git-send-email-alexandre.montplaisir@gmail.com>
Message-ID: <20111130001727.GA15401@Krystal>

* Alexandre Montplaisir (alexandre.montplaisir at gmail.com) wrote:
> Use AS_IF() and AS_ECHO() macros whenever possible.
> Print the configuration options at the end of the script
> (so they don't get lost in the sea of the configure output).

Merged, thanks!

Mathieu

> 
> Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir at gmail.com>
> ---
>  configure.ac |   23 +++++++++++++----------
>  1 files changed, 13 insertions(+), 10 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index b73cf0f..01e15d3 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -85,21 +85,14 @@ AC_CHECK_DECL([caa_likely], [],
>  
>  # Check liblttng-ust-ctl library
>  AC_ARG_ENABLE(lttng-ust,
> -	[  --disable-lttng-ust     build without LTTng-UST (Userspace Tracing) support.],
> +	AS_HELP_STRING([--disable-lttng-ust],[build without LTTng-UST (Userspace Tracing) support]),
>  	lttng_ust_support=$enableval, lttng_ust_support=yes)
>  
> -[
> -if test "x$lttng_ust_support" = "xno"; then
> -	echo "LTTng-UST support disabled."
> -else
> -]
> +AS_IF([test "x$lttng_ust_support" = "xyes"], [
>  	AC_CHECK_LIB([lttng-ust-ctl], [ustctl_create_session], [],
>  		[AC_MSG_ERROR([Cannot find LTTng-UST. Use [LDFLAGS]=-Ldir to specify its location, or specify --disable-lttng-ust to build lttng-tools without LTTng-UST support.])]
>  	)
> -[
> -	echo "LTTng-UST support enabled."
> -fi
> -]
> +])
>  
>  AM_CONDITIONAL([HAVE_LIBLTTNG_UST_CTL], [ test "x$ac_cv_lib_lttng_ust_ctl_ustctl_create_session" = "xyes" ])
>  
> @@ -146,3 +139,13 @@ AC_CONFIG_FILES([
>  ])
>  
>  AC_OUTPUT
> +
> +# Mini-report on what will be built
> +AS_ECHO("")
> +
> +AS_ECHO_N("Lttng-UST support: ")
> +AS_IF([test "x$lttng_ust_support" = "xyes"], [AS_ECHO("Enabled")],
> +	[AS_ECHO("Disabled")])
> +
> +AS_ECHO("")
> +
> -- 
> 1.7.7.3
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com