Opened 4 years ago

Closed 4 years ago

#20130 closed defect (fixed)

Use blas pc-files

Reported by: vbraun Owned by:
Priority: major Milestone: sage-7.1
Component: build Keywords:
Cc: fbissey Merged in:
Authors: Volker Braun Reviewers: François Bissey
Report Upstream: N/A Work issues:
Branch: 555b258 (Commits) Commit: 555b25811afe8da5d83c3ed6461e423a70f961f7
Dependencies: Stopgaps:

Description (last modified by vbraun)

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

Change History (12)

comment:1 Changed 4 years ago by vbraun

  • Branch set to u/vbraun/use_blas_pc_files

comment:2 Changed 4 years ago by vbraun

  • Authors set to Volker Braun
  • Commit set to 18d59bf4dda7b0521395b917e30edb5f026f524f
  • Component changed from PLEASE CHANGE to build
  • Type changed from PLEASE CHANGE to defect

New commits:

18d59bfMove third-party packages to pkg-config blas

comment:3 Changed 4 years ago by vbraun

  • Cc fbissey added

comment:4 Changed 4 years ago by vbraun

  • Description modified (diff)

comment:5 Changed 4 years ago by jdemeyer

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

comment:6 Changed 4 years ago by git

  • Commit changed from 18d59bf4dda7b0521395b917e30edb5f026f524f to de7efd23ffb7c9ed223bf769e55f585f47566d1a

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

de7efd2Fix typo: use lapack.pc not blas.pc

comment:7 Changed 4 years ago by git

  • Commit changed from de7efd23ffb7c9ed223bf769e55f585f47566d1a to f1302b23fb533a17e91f66bda951c3394eed1463

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

f1302b2Bump patchlevels

comment:8 Changed 4 years ago by vbraun

  • Status changed from new to needs_review

comment:9 Changed 4 years ago by fbissey

  • Reviewers set to François Bissey
  • Status changed from needs_review to positive_review

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.

comment:10 Changed 4 years ago by git

  • Commit changed from f1302b23fb533a17e91f66bda951c3394eed1463 to 555b25811afe8da5d83c3ed6461e423a70f961f7
  • Status changed from positive_review to needs_review

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

555b258Iml version/patchlevel fix

comment:11 Changed 4 years ago by vbraun

  • Status changed from needs_review to positive_review

Yes, lapack is still broken in fflas-ffpack

comment:12 Changed 4 years ago by vbraun

  • Branch changed from u/vbraun/use_blas_pc_files to 555b25811afe8da5d83c3ed6461e423a70f961f7
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.