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.

Sign up for GitHub

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

Jump to bottom

Use blas pc-files #20130

Closed
vbraun opened this issue Feb 27, 2016 · 16 comments
Closed

Use blas pc-files #20130

vbraun opened this issue Feb 27, 2016 · 16 comments
Labels
c: build p: major / 3 t: bug
Milestone
sage-7.1

Comments

@vbraun
Copy link
Member

vbraun commented Feb 27, 2016

Get blas configuration in third-party packages from pkg-config

CC: @kiwifb

Component: build

Author: Volker Braun

Branch/Commit: 555b258

Reviewer: François Bissey

Issue created by migration from https://trac.sagemath.org/ticket/20130
@vbraun vbraun added this to the sage-7.1 milestone Feb 27, 2016
@vbraun
Copy link
Member Author

vbraun commented Feb 27, 2016

Branch: u/vbraun/use_blas_pc_files

@vbraun
Copy link
Member Author

vbraun commented Feb 27, 2016

Author: Volker Braun

@vbraun
Copy link
Member Author

vbraun commented Feb 27, 2016

Commit: 18d59bf

@vbraun
Copy link
Member Author

vbraun commented Feb 27, 2016

New commits:

18d59bfMove third-party packages to pkg-config blas

@vbraun

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link

comment:5

When you change a package, don't forget to bump the version number patch level.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Changed commit from 18d59bf to de7efd2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

de7efd2Fix typo: use lapack.pc not blas.pc

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

f1302b2Bump patchlevels

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Changed commit from de7efd2 to f1302b2

@kiwifb
Copy link
Member

kiwifb commented Feb 28, 2016

comment:9

Looks good to me in that it does all that was done before. I'll have to check if the detection of lapack is still broken in the fflas-ffpack upgrade.

@kiwifb
Copy link
Member

kiwifb commented Feb 28, 2016

Reviewer: François Bissey

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Changed commit from f1302b2 to 555b258

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2016

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

555b258Iml version/patchlevel fix

@vbraun
Copy link
Member Author

vbraun commented Feb 28, 2016

comment:11

Yes, lapack is still broken in fflas-ffpack

@vbraun
Copy link
Member Author

vbraun commented Feb 28, 2016

Changed branch from u/vbraun/use_blas_pc_files to 555b258

@vbraun vbraun closed this as completed in e55b4f9 Feb 28, 2016
@jdemeyer jdemeyer mentioned this issue Mar 2, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: build p: major / 3 t: bug
Projects
None yet
Milestone
sage-7.1
Development

No branches or pull requests
3 participants
@kiwifb @vbraun @jdemeyer