* Alexandre Montplaisir ([email protected]) 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 <[email protected]>
> ---
>  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
> [email protected]
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> 

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

_______________________________________________
lttng-dev mailing list
[email protected]
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to