Opened 4 years ago

Closed 4 years ago

#24362 closed defect (fixed)

numpy fails to build when MKLROOT environment variable is set

Reported by: tmonteil Owned by:
Priority: critical Milestone: sage-8.2
Component: packages: standard Keywords:
Cc: jdemeyer, fbissey Merged in:
Authors: Thierry Monteil Reviewers: François Bissey
Report Upstream: N/A Work issues:
Branch: 5ae04cd (Commits, GitHub, GitLab) Commit: 5ae04cde3cbffb845be2bd69c7192d7ef4ec11c7
Dependencies: Stopgaps:

Status badges

Description

numpy fails to build on a 40-core 64bit PC running Debian 8. I used sage's gcc (with export SAGE_INSTALL_GCC='yes'). Log attached.

Attachments (1)

numpy-1.13.3.p0.log (255.3 KB) - added by tmonteil 4 years ago.

Download all attachments as: .zip

Change History (9)

Changed 4 years ago by tmonteil

comment:1 Changed 4 years ago by tmonteil

Unfortunately, I can reproduce the issue with make -j1 (i was secretly hoping a race condition).

comment:2 Changed 4 years ago by tmonteil

  • Branch set to u/tmonteil/numpy_fails_to_build

comment:3 Changed 4 years ago by git

  • Commit set to 5ae04cde3cbffb845be2bd69c7192d7ef4ec11c7

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5ae04cdTrac 24362 : unset MKLROOT environment variable

comment:4 Changed 4 years ago by tmonteil

  • Authors set to Thierry Monteil
  • Summary changed from numpy fails to build to numpy fails to build when MKLROOT environment variable is set

Apparently, i found the culprit: MKL blas was installed on the computer, but unsetting MKL is not enough, we also have to unset MKLROOT.


New commits:

5ae04cdTrac 24362 : unset MKLROOT environment variable

comment:5 Changed 4 years ago by fbissey

Yes numpy is stupid that way. It knows better than your settings unless you disable any other possibility.

comment:6 Changed 4 years ago by tmonteil

  • Status changed from new to needs_review

make ptestlong leads to "All tests passed!"

comment:7 Changed 4 years ago by fbissey

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

Looks good to me.

comment:8 Changed 4 years ago by vbraun

  • Branch changed from u/tmonteil/numpy_fails_to_build to 5ae04cde3cbffb845be2bd69c7192d7ef4ec11c7
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.