Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 493318 - sci-libs/scipy with sci-libs/mkl (intel blas/cblas) - ?
Summary: sci-libs/scipy with sci-libs/mkl (intel blas/cblas) - ?
Status: VERIFIED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Related Packages
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2013-12-04 15:08 UTC by Olaf Leidinger
Modified: 2014-01-27 11:34 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
fix for the ebuild (scipy-0.13.1-r2.ebuild.diff,396 bytes, patch)
2013-12-04 15:10 UTC, Olaf Leidinger
Details | Diff
a better, more general, fix for the ebuild (scipy-0.13.1-r2.ebuild.diff,418 bytes, patch)
2013-12-04 16:30 UTC, Olaf Leidinger
Details | Diff
fix for current version of ebuild (double-library-entry.ebuild.patch,394 bytes, patch)
2014-01-27 11:34 UTC, Olaf Leidinger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olaf Leidinger 2013-12-04 15:08:46 UTC
The configure step fails, as it can't find blas. My system only contains intel's blas implementation. I used scipy 0.13.0, however, 0.12.* also has the problem.


Reproducible: Always

Steps to Reproduce:
1. emerge mkl, I used the latest version from science overlay, but that shouldn't matter
2. eselect cblas, blas and lapack to mkl. I used "mkl64-int64-dynamic" in every case.
3. emerge scipy



The error occurs due to a "failure" in site.cfg. The runtime library, libmkl_rt is included twice in the library section. The following patch generates a correct configuration file and thus, the package compiles fine.
Comment 1 Olaf Leidinger 2013-12-04 15:10:43 UTC
Created attachment 364592 [details, diff]
fix for the ebuild

This fixes the generated scite.cfg for me. It simply removes double library names.
Comment 2 Olaf Leidinger 2013-12-04 16:30:22 UTC
Created attachment 364598 [details, diff]
a better, more general, fix for the ebuild
Comment 3 Justin Lecher (RETIRED) gentoo-dev 2013-12-26 20:52:37 UTC
+*scipy-0.13.2-r1 (26 Dec 2013)
+
+  26 Dec 2013; Justin Lecher <jlec@gentoo.org> +scipy-0.13.2-r1.ebuild:
+  Fix for linking with mkl, #493318
+
Comment 4 Olaf Leidinger 2014-01-27 11:32:31 UTC
I'm sorry, this doesn't fix the problem for me. The issue is the same, the sed command in the ebuild doesn't remove the double library entries. I'll attach a patch for the ebuild.

My version gives:

pkg-config --libs-only-l cblas blas | sed -e  's/[ ]-l*\(pthread\|m\)[ ]*//g' -e 's/^-l//' -e 's/[ ]*-l/\n/g' -e 's/[ ]*$//' 
mkl_rt
mkl_rt

The current ebuild gives:

pkg-config --libs-only-l cblas blas | sed -e 's/[ ]-l*\(pthread\|m\)\([ ]\|$\)//g'  -e 's/^-l//' -e 's/[ ]*-l/,/g' -e 's/[ ]*$//'
mkl_rt,mkl_rt

And thus uniq doesn't do it's magic.
Comment 5 Olaf Leidinger 2014-01-27 11:34:47 UTC
Created attachment 368866 [details, diff]
fix for current version of ebuild

Patch for scipy-0.13.2-r1.ebuild to resolve above issue