Page MenuHomeFreeBSD

math/lapack: remove superfluous and broken ranlib invocation
ClosedPublic

Authored by emaste on Aug 10 2021, 4:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 20, 12:41 PM
Unknown Object (File)
Sun, Sep 15, 9:13 AM
Unknown Object (File)
Fri, Sep 6, 5:53 PM
Unknown Object (File)
Thu, Sep 5, 12:38 AM
Unknown Object (File)
Tue, Sep 3, 11:17 PM
Unknown Object (File)
Aug 9 2024, 6:02 PM
Unknown Object (File)
Aug 4 2024, 1:57 PM
Unknown Object (File)
Aug 1 2024, 3:41 AM
Subscribers

Details

Summary
FreeBSD's base system ar and ranlib have a bug where they exit with
status 0 (success) even in case of fatal errors, which hid the fact that
math/lapacke was invoking ranlib on a non-existent file.  (Presumably the
ranlib invocation was correct when introduced, but broken by some
rework of lapack's upstream build system).

Use of ranlib is unncessary assuming the -s flag is passed to ar (it is
here), so just remove it.

PR: 257744
Sponsored by:   The FreeBSD Foundation
Test Plan

Verify math/lapacke builds and inspect build log to ensure ar contains -s flag

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste created this revision.
emaste added a reviewer: Ports Committers.
kevans added a subscriber: kevans.

Confirm it builds under poudriere testport then

Approved by: kevans (ports), portmgr (implicit, blanket: build fix)
This revision is now accepted and ready to land.Aug 10 2021, 8:42 PM
jrm accepted this revision as: jrm.EditedAug 10 2021, 8:45 PM
jrm added a subscriber: jrm.

Ran testport for 11amd64, 12i386, 12amd64, and 13amd64 for math/lapack, math/blas, math/cblas, math/lapacke, and math/xlapack. Everything looks fine.

(Also for posterity, PORTREVISION bump not necessary as the removed command had no effect.)