Re: libacm fails to build

[Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>]

On Thu, Mar 09, 2017 at 10:35:30PM +0000, Bart Van Assche wrote:
> On Thu, 2017-03-09 at 10:10 -0700, Jason Gunthorpe wrote:
> > On Thu, Mar 09, 2017 at 11:25:27AM +0100, Philipp Thomas wrote:
> > 
> > > I'm trying to build latest rdma-core but it fails in ibacm because a
> > > library is missing:
> > 
> > The suse cmake rpm macros are broken, this is worked around in the
> > shipped rdma-core.spec:
> > 
> > # Tumbleweed's cmake RPM macro adds -Wl,--no-undefined to the module flags
> > # which is totally inappropriate and breaks building 'ENABLE_EXPORTS' style
> > # module libraries (eg ibacmp).
> > %define CMAKE_FLAGS -DCMAKE_MODULE_LINKER_FLAGS=""
> > 
> > Ideally you would consult internally to globally fix the rpm cmake
> > macro, otherwise you have to use the above hack..
> 
> Something else is also broken, namely the SuSE section in rdma-core.spec. I
> just figured out that the following is needed to build an RPM on SuSE systems:
> 
> diff --git a/rdma-core.spec b/rdma-core.spec
> index f7a440d7..aa486371 100644
> +++ b/rdma-core.spec
> @@ -28,7 +28,7 @@ BuildRequires: ninja,make
>  %define __builder ninja
>  # cmake_install,make_jobs is specified by opensuse
>  %define cmake cmake
> -%define cmake_install DESTDIR=%{buildroot} ninja install
> +%define cmake_install DESTDIR=%{buildroot} make install

I'm not sure what tree you are looking at? The above '-' line never
appeared in mainline?

The opensuse rpm macros themselves are supposed to provide
cmake_install, so the spec file should not define it. Did they change
their macros again or something?

We provide it for the RH cases because they use a different cmake rpm
macro set. :(

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html