Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

openblas: clear out .pc symlinks before installation #30335

Closed
mkoeppe opened this issue Aug 11, 2020 · 11 comments
Closed

openblas: clear out .pc symlinks before installation #30335

mkoeppe opened this issue Aug 11, 2020 · 11 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 11, 2020

When switching from system BLAS to SPKG openblas, the following errors have been reported:

Copying package files from temporary location /Users/yzh/sagecopies/sage/worktree-2020/local/var/tmp/sage/build/openblas-0.3.9/inst to /Users/yzh/sagecopies/sage/worktree-2020/local
cp: /Users/yzh/sagecopies/sage/worktree-2020/local/./lib/pkgconfig/cblas.pc: Permission denied
cp: /Users/yzh/sagecopies/sage/worktree-2020/local/./lib/pkgconfig/blas.pc: Permission denied
cp: /Users/yzh/sagecopies/sage/worktree-2020/local/./lib/pkgconfig/lapack.pc: Permission denied
************************************************************************
Error copying files for openblas-0.3.9.

The correct fix is described in #29585 comment:9
and #29387 + #29386

This ticket is a quick workaround for Sage 9.2.

CC: @yuan-zhou @jhpalmieri @dimpase @embray

Component: build

Author: Matthias Koeppe

Branch: 29b1ff8

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/30335

@mkoeppe mkoeppe added this to the sage-9.2 milestone Aug 11, 2020
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 11, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 11, 2020

New commits:

29b1ff8build/pkgs/openblas/spkg-install.in: Clear out possible .pc symlinks before installation

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 11, 2020

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 11, 2020

Commit: 29b1ff8

@dimpase
Copy link
Member

dimpase commented Aug 11, 2020

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Aug 11, 2020

comment:3

lgtm

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 11, 2020

comment:4

Thanks!

@vbraun
Copy link
Member

vbraun commented Aug 14, 2020

@embray
Copy link
Contributor

embray commented Aug 31, 2020

comment:6

Ah, no, spkg-install scripts should not make changes to $SAGE_LOCAL. If it's absolutely necessary to you can do that in an spkg-preinst script.

@embray
Copy link
Contributor

embray commented Aug 31, 2020

Changed commit from 29b1ff8 to none

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 31, 2020

comment:7

Follow-up: #30480

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants