Re: [Libunwind-devel] [PATCH 5/7] build: make libunwind-coredump build optional

[Tommi Rantala]

2012/9/4 Cody P Schafer <address@hidden>:
> Disable the building of libunwind-coredump except on x86_64 and x86
> (where implimentations exsist).

Technically checking just the architecture would not be enough, since
there are also some OS specific bits and pieces.

We could also provide no-op implementations for the missing
architectures, similar to (as far as I can tell) the setjmp library.
This might make distro people happier, as they could package the
coredump library for all architectures unconditionally.

> +no_force_coredump=true
> +AC_ARG_ENABLE(coredump,
> +             AS_HELP_STRING([--enable-coredump],[build libunwind-coredump 
> library]),
> +             [no_force_coredump=false; use_coredump=true],
> +             [no_force_coredump=false; use_coredump=false])
> +
> +if $no_force_coredump; then
> +  case "$host_arch" in
> +    x86|x86_64) use_coredump=true ;;
> +    *) use_coredump=false;;
> +  esac
> +fi

BTW, after applying this patch, the coredump library is no longer
built by default on my x86_64 machine. I believe that AC_ARG_ENABLE()
is not used entirely correctly here.

BR,
Tommi