Re: F23 Self Contained Change: RPM MPI Requires Provides

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 09.07.2015 20:18, Orion Poplawski wrote:
Also, isn't the -x86_64 redundant?  Also though I guess we don't have an mpi
variable MPI_NAME.
Yes it is redundant, but it is the prettiest variable I could find, given the lack of MPI_NAME.
Also, your trick of using:

   for module in $(module avail 2>&1 | grep "^mpi/"); do

to find the available mpi modules needs a "-t" option for Lmod.  Fortunately
this also works with environment-modules:

  for module in $(module -t avail 2>&1 | grep "^mpi/"); do
Ok
Also, it doesn't seem to get all of the requires quite right.  For
scorep-openmpi I have:

Provides: libscorep_adapter_compiler_event.so.2()(64bit)(openmpi-x86_64)

but:

Requires: libscorep_adapter_compiler_event.so.2()(64bit)

is being emitted.  This appears to be coming from:

./fileattrs/libsymlink.attr:%__libsymlink_requires
%{_rpmconfigdir}/elfdeps --provides --soname-only

So it looks like we need to contend with that as well.

# cat fileattrs/libsymlink.attr
# Make libfoo.so symlinks require the soname-provide of the target library
%__libsymlink_requires          %{_rpmconfigdir}/elfdeps --provides --soname-only
%__libsymlink_magic             ^symbolic link to `.*lib.*\.so\..*'$
%__libsymlink_exclude_path      ^.*[[:digit:]]$


Perhaps with:

%global __libsymlink_exclude_path     ^%{_prefix}/lib(64)?/(openmpi|mpich)/.*$

in mpi packages.
Or directly in mpi.attrs?

Thanks,
Sandro

--
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux