Bug 256493 - science/py-phono3py: Remove unnecessary dependency on math/lapacke
Summary: science/py-phono3py: Remove unnecessary dependency on math/lapacke
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: Yuri Victorovich
Keywords: patch
Depends on:
Reported: 2021-06-09 02:48 UTC by John Hein
Modified: 2021-06-09 14:59 UTC (History)
2 users (show)

See Also:
jcfyecrayz: maintainer-feedback? (yuri)

[patch] remove unnecessary dependency on math/lapacke (785 bytes, patch)
2021-06-09 02:53 UTC, John Hein
jcfyecrayz: maintainer-approval? (yuri)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Hein 2021-06-09 02:48:49 UTC
science/py-phono3py depends on openblas.  openblas satisfies the LAPACKE dependency (see install.rst).

The dependency on math/lapacke should be removed.
Comment 1 John Hein 2021-06-09 02:53:07 UTC
Created attachment 225654 [details]
[patch] remove unnecessary dependency on math/lapacke

 testport - ok (stable/11 amd64)
 portlint - ok
Comment 2 Yuri Victorovich freebsd_committer 2021-06-09 03:32:27 UTC
Committed, thanks!
Comment 3 commit-hook freebsd_committer 2021-06-09 03:33:15 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=93284e2c88bae6736692aad5b93ac25b34e0c20e

commit 93284e2c88bae6736692aad5b93ac25b34e0c20e
Author:     Yuri Victorovich <yuri@FreeBSD.org>
AuthorDate: 2021-06-09 03:30:02 +0000
Commit:     Yuri Victorovich <yuri@FreeBSD.org>
CommitDate: 2021-06-09 03:32:22 +0000

    science/py-phono3py: Remove unnecessary build dependency on math/lapacke

    PR:             256493
    Reported by:    John Hein <jcfyecrayz@liamekaens.com>

 science/py-phono3py/Makefile | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
Comment 4 John Hein 2021-06-09 14:59:40 UTC
(In reply to commit-hook from comment #3)

My mistake on bumping PORTREVISION (not needed because the dependency change was just a BUILD_DEPENDS) in my patch.  Thanks for not committing that.