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
NumPy: rework, bump, cleanup, extra libs #3694
Conversation
@Begasus can you pls check blis on x86? |
How did you obtain the missing symbol error? I tried to run |
I'll try to add your patch and retest it. |
Now it works ok, at least no problem at importing it. |
dev-python/numpy/numpy-1.16.2.recipe
Outdated
pythonVersion=${PYTHON_VERSIONS[$i]} | ||
eval "PROVIDES_${pythonPackage}=\"\ | ||
${portName}_$pythonPackage = $portVersion\n\ | ||
cmd:f2py\n\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this mean that the two python versions will conflict?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True
dev-python/numpy/numpy-1.16.2.recipe
Outdated
cat > site.cfg << EOF | ||
[ALL] | ||
libraries = execinfo,lapack,openblas | ||
library_dirs = /system/`echo ${relativeDevelopLibDir}` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
library_dirs = /system/`echo ${relativeDevelopLibDir}` | |
library_dirs = /system/$relativeDevelopLibDir |
It'd be shorter to just use the variables directly :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you do the same with all other variables?
dev-python/numpy/numpy-1.16.2.recipe
Outdated
SOURCE_DIR="numpy-$portVersion" | ||
PATCHES="numpy-$portVersion.patch" | ||
|
||
ARCHITECTURES="any" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be any. $secondaryArchSuffix will only work with a secondary architecture defined anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blis can't be build for x86_gcc2, so agreed, any shouldn't be used here
ARCHITECTURES="any" | ||
|
||
PROVIDES=" | ||
$portName = $portVersion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why use $portName here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The recipe template contains it.
dev-python/numpy/numpy-1.16.2.recipe
Outdated
" | ||
|
||
BUILD_REQUIRES=" | ||
haiku_devel$secondaryArchSuffix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some mixup? :)
- haiku_devel$secondaryArchSuffix
+ haiku${secondaryArchSuffix}_devel
Seems to be ok, at least it works, unlike the earlier, so lets merge it. |
Refer: https://discuss.haiku-os.org/t/package-python3-numpy-not-working-on-nightly-build-hrev52992/8380
Import works now, but both version (python2 and 3) fails with missing symbols:
Both symbol defined in
libopenblas-r0.3.5.so
:Disabling blas results Python crash at import.