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)

OpenBLAS_Portfile_LAPACKE.diff (2.7 KB) - added by michaelld (Michael Dickens) 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by michaelld (Michael Dickens)

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: newclosed

Thanks for the report. Patch added in r144497.

comment:3 Changed 9 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: closedreopened

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: reopenedclosed

Fixed in r145974.

Note: See TracTickets for help on using tickets.