F23 Self Contained Change: RPM MPI Requires Provides -> Review request

[Sandro Mani]

On 08.07.2015 22:28, Orion Poplawski wrote:
> It appears to be sufficient to define this macro anywhere, not just in 
> elf.attr. So I think it could be added in to a rpm macros file in 
> openmpi/mpich-devel. 
Ah cool, actually it seems to also work if I add it directly to 
mpi.attrs, meaning that all pieces are nicely confined in one place.
> Also I think you need (64)? or (|64).
Correct, fixed. I'm seeing this also in cmake:

$ grep '(64)' /usr/lib/rpm/fileattrs/cmake.attr
%__cmake_path ^/usr/lib(64)/cmake/.*/.*(Config\.cmake|-config\.cmake)$

suppose it should be fixed there too.



I've now posted the review request for the rpm-mpi-hooks package at 
https://bugzilla.redhat.com/show_bug.cgi?id=1241282

A quick double check of the mpi.req and mpi.prov as part of the review 
would also be appreciated.

Once the package is approved, openmpi and mpich will need to 
BuildRequire: rpm-mpi-hooks, and openmpi-devel and mpich-devel will need 
to Require: rpm-mpi-hooks.

And finally, once that is done too, all *-openmpi and *-mpich packages 
will need to be rebuilt.

Thanks,
Sandro