Opened 9 years ago
Closed 9 years ago
#50232 closed defect (fixed)
openblas: destroot fails with variant lapack
Reported by: | michaelld (Michael Dickens) | Owned by: | NicosPavlov |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.4 |
Keywords: | haspatch | Cc: | |
Port: | OpenBLAS, OpenBLAS-devel |
Description
same summary as another ticket that was fixed, but for a different reason. Looks like OpenBLAS uses case sensitive directories for some items, including the lapacke headers installed in the post-destroot when +lapack is enabled. Which, by the way, is the default now as of a recent commit. It looks like the change needed is to replace "/lapacke/" with "/LAPACKE/". This works for me. Another option might be to used "find DIR -name "*.h"" and then just use those with "lapack" in the name. This port should fail when doing an install / update and using +lapack on a case sensitive file system right now.
Attachments (1)
Change History (6)
Changed 9 years ago by michaelld (Michael Dickens)
Attachment: | OpenBLAS_Portfile_LAPACKE.diff added |
---|
comment:1 Changed 9 years ago by mf2k (Frank Schima)
Keywords: | haspatch added; hasfix removed |
---|
comment:2 Changed 9 years ago by NicosPavlov
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 9 years ago by michaelld (Michael Dickens)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Related to #50653 and r145845. We have a strange issue: the release port uses 'lapacke' while the devel port uses 'LAPACKE'. We can split out the actual xinstall command in the release and devel, or we can come up with a patch for the release to tweak it to be uppercase, or some other clever technique within the port file such as a 'find | grep -i' to get the actual directory name.
comment:4 Changed 9 years ago by NicosPavlov
That's an interesting one.
It seems to me that we could shoot for simplicity and just split the install, or use a variable to avoid repeating the commands. The folder has been lowercase throughout several releases, even when lapacke itself had been updated.
comment:5 Changed 9 years ago by michaelld (Michael Dickens)
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Fixed in r145974.
Thanks for the report. Patch added in r144497.