On 07/06/2011 06:52 AM, Steven Dake wrote:
> From: Florian Haas <[email protected]>
> 
> Having to force {ibverbs,rdmacm}_{LIBS,CFLAGS} looks positively odd;
> so this may warrant further review. However, they are definitely not
> needed if building without RDMA support.

Florian,

I agree it does look odd, and I was hopeful the rdma guys would take
pkgconfig patches.  But the pkgconfig patches were never merged and we
depend on the PKGCONFIG macros in configure.

Reviewed-by: Steven Dake <[email protected]>

> ---
>  corosync.spec.in |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/corosync.spec.in b/corosync.spec.in
> index d5bdeb6..34e1658 100644
> --- a/corosync.spec.in
> +++ b/corosync.spec.in
> @@ -57,10 +57,12 @@ BuildRoot: %(mktemp -ud 
> %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
>  ./autogen.sh
>  %endif
>  
> +%if %{with rdma}
>  export ibverbs_CFLAGS=-I/usr/include/infiniband \
>  export ibverbs_LIBS=-libverbs \
>  export rdmacm_CFLAGS=-I/usr/include/rdma \
>  export rdmacm_LIBS=-lrdmacm \
> +%endif
>  %{configure} \
>       --enable-nss \
>  %if %{with testagents}

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to