Bug 1241737 - [Patch] Use %__libsymlink_path instead of %__libsymlink_exclude_path
Summary: [Patch] Use %__libsymlink_path instead of %__libsymlink_exclude_path
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: redhat-rpm-config
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Florian Festi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1238428
TreeView+ depends on / blocked
 
Reported: 2015-07-10 00:48 UTC by Sandro Mani
Modified: 2015-07-15 14:59 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-07-15 08:53:50 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch (1.51 KB, patch)
2015-07-10 00:48 UTC, Sandro Mani
no flags Details | Diff

Description Sandro Mani 2015-07-10 00:48:33 UTC
Created attachment 1050485 [details]
Patch

Description of problem:
For the F23 Change "RPM MPI Requires Provides" [1] the chosen strategy is to use a dependency generator hook to extend the requires/provides generated by elfdeps for mpi binaries. Two default dependency triggers need to be prevented from being run for files in %{_prefix}/lib(64)?/(openmpi|mpich), namely

fileattrs/elf.attr
fileattrs/libsymlink.attr


elf.attr can be blocked by adding to mpi.attr:

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


Currently this cannot be done for libsymlink.attr however because it already defines an exclude_path:

%__libsymlink_exclude_path      ^.*[[:digit:]]$

To allow mpi.attr to block the libsymlink triggers, I'd like to propose replacing the %__libsymlink_exclude_path with

%__libsymlink_path   ^.*\.so$
%__libsymlink_flags magic_and_path

Patch attached.


[1] https://fedoraproject.org/wiki/Changes/RpmMPIReqProv


Version-Release number of selected component (if applicable):
redhat-rpm-config-34-1.fc23.noarch

Comment 1 Florian Festi 2015-07-15 08:53:50 UTC
Added to redhat-rpm-config-35-1.fc24

Comment 2 Sandro Mani 2015-07-15 11:48:48 UTC
Thank you. Could you build this also for F23? The RpmMPIReqProv feature is targeting that release.

Comment 3 Florian Festi 2015-07-15 12:54:05 UTC
Build as redhat-rpm-config-35-1.fc23

Comment 4 Sandro Mani 2015-07-15 14:59:46 UTC
Many thanks


Note You need to log in before you can comment on or make changes to this bug.