Page MenuHomeFreeBSD
Differential D20866

math/openblas: Install header files too.
ClosedPublic
Actions

Authored by arrowd on Jul 6 2019, 4:45 PM.
Tags
None
Referenced Files
F139435889: D20866.diff
Fri, Dec 12, 1:43 AM
F139413680: D20866.id63022.diff
Thu, Dec 11, 7:38 PM
Unknown Object (File)
Thu, Dec 11, 1:08 AM
Unknown Object (File)
Tue, Dec 9, 12:43 AM
Unknown Object (File)
Thu, Nov 27, 5:35 PM
Unknown Object (File)
Mon, Nov 24, 12:07 PM
Unknown Object (File)
Sun, Nov 23, 9:45 PM
Unknown Object (File)
Sun, Nov 23, 11:42 AM
View All Files
Subscribers
lbartoletti
lwhsu
mat

Details

Reviewers
None
Group Reviewers
Contributor Reviewers (ports)
Commits
rP515970: math/openblas: Install header files.

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 25220
Build 23897: arc lint + arc unit

Event Timeline

arrowd created this revision.Jul 6 2019, 4:45 PM
Harbormaster completed remote builds in B25220: Diff 59470.Jul 6 2019, 4:45 PM
lbartoletti added a subscriber: lbartoletti.Aug 10 2019, 5:51 AM
lwhsu added a subscriber: lwhsu.Sep 29 2019, 12:11 AM

maintainer approved at https://bugs.freebsd.org/238228

arrowd updated this revision to Diff 62848.Oct 2 2019, 6:00 PM

Add prefix to OpenBLAS headers with common names (like config.h or common.h).

Harbormaster completed remote builds in B26836: Diff 62848.Oct 2 2019, 6:00 PM
arrowd updated this revision to Diff 63022.Oct 8 2019, 5:11 AM
  • And bump port revision.
Harbormaster completed remote builds in B26922: Diff 63022.Oct 8 2019, 5:11 AM
arrowd updated this revision to Diff 63399.Oct 17 2019, 11:26 AM
  • Patch config.h entries too. Submitted by Artyom Davidov <ard_1@mail.ru>
Harbormaster completed remote builds in B27083: Diff 63399.Oct 17 2019, 11:26 AM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 29 2019, 10:25 AM
Closed by commit rP515970: math/openblas: Install header files. (authored by arrowd). · Explain Why
This revision was automatically updated to reflect the committed changes.
arrowd added a commit: rP515970: math/openblas: Install header files..
mat added a subscriber: mat.Oct 29 2019, 10:34 AM
mat added inline comments.
head/math/openblas/Makefile
187–191 ↗(On Diff #63748)

Shouldn't all these be ${MV} and not ${CP}?

arrowd marked an inline comment as done.Oct 29 2019, 12:30 PM

Revision Contents
Changeset List

PathSize
math/
openblas/
2 lines
40 lines

Diff 59470

View Options

math/openblas/Makefile

Loading...
View Options

math/openblas/pkg-plist

Loading...