Bug 285345 - net/openmpi: Fix pkg-plist when option SLURM is used (5.0.7)
Summary: net/openmpi: Fix pkg-plist when option SLURM is used (5.0.7)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Vladimir Druzenko
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-03-12 11:09 UTC by Laurent Chardon
Modified: 2025-03-12 15:02 UTC (History)
1 user (show)

See Also:
laurent.chardon: maintainer-feedback+

Attachments
net/openmpi fix pkg-plist for SLURM oprion (562 bytes, patch)
2025-03-12 11:09 UTC, Laurent Chardon 		laurent.chardon: maintainer-approval+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Laurent Chardon 2025-03-12 11:09:47 UTC 
Created attachment 258599 [details]
net/openmpi fix pkg-plist for SLURM oprion

Fix pkg-plist for option SLURM
Comment 1 Laurent Chardon 2025-03-12 11:11:26 UTC 
I didn't bump the PORTREVISION since there is no change in the default package.

Vladimir I cc'ed you because I know that you are aware of the issue.

Thanks!
Comment 2 Vladimir Druzenko freebsd_committer freebsd_triage 2025-03-12 13:28:02 UTC 
Don't need lib with option SLURM: libslurm.so:sysutils/slurm-wlm
Comment 3 commit-hook freebsd_committer freebsd_triage 2025-03-12 14:15:30 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=65c9bd0bf75590ab699b41d86c9ed00b11f4db79

commit 65c9bd0bf75590ab699b41d86c9ed00b11f4db79
Author:     Laurent Chardon <laurent.chardon@gmail.com>
AuthorDate: 2025-03-12 14:11:24 +0000
Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
CommitDate: 2025-03-12 14:14:40 +0000

    net/openmpi: Fix pkg-plist when non-default option SLURM is used

    PR:             285345
    Reported by:    leres (via email)
    Fixes:          d164009dd097 (Update 5.0.6 → 5.0.7)

 net/openmpi/pkg-plist | 2 --
 1 file changed, 2 deletions(-)
Comment 4 Vladimir Druzenko freebsd_committer freebsd_triage 2025-03-12 14:23:14 UTC 
Build with SLURM on:
Warning: you might not need LIB_DEPENDS on libslurm.so

After remove line:
SLURM_LIB_DEPENDS=     libslurm.so:sysutils/slurm-wlm
build without errors and warnings.
Comment 5 Laurent Chardon 2025-03-12 15:00:38 UTC 
Thanks. I will have a good look at that option in the near future because something smells fishy. I'm also looking into updating sysutils/slrm-wlm and making it work, because I have a feeling that it should.
Comment 6 Vladimir Druzenko freebsd_committer freebsd_triage 2025-03-12 15:02:17 UTC 
(In reply to Laurent Chardon from comment #5)
Ok!