-
Notifications
You must be signed in to change notification settings - Fork 145
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
auditwheel repair marked internal so files as shared library dependencies #212
Comments
Thanks for the report. As for the issue itself, I did not have time to investigate this yet. Is it reproducible on x86_64 ? TensorFlow being quite complex, do they use something other than auditwheel to repair/rename wheels in official builds ? |
Thank you @mayeut for identifying that problem, if ld64.so.2 is not listed, I shouldn't need to run auditwheel repair in the first place. I will try to create the problem on x86 and see if i can (after the holidays) |
I was unable to recreate the problem with auditwheel repair on x86. |
Yes, I did verify the fix works. Sorry for not posting it here.
|
I'm trying to determine a minimal reproduction outside of tensorflow, so far no luck but will keep trying. |
@wdirons How big is the Tensorflow wheel that reproduces this? |
@di , the nightly build of TensorFlow for CPU is 147MB and is available here: https://powerci.osuosl.org/job/TensorFlow_PPC64LE_CPU_Nightly_Artifact/lastSuccessfulBuild/artifact/tensorflow_pkg/ (being a nightly build the file name changes nightly, I haven't yet done a release build from the quay manylinux 2014 container) |
@di , this can be recreated building numpy (v1.18.1) on ppc64le. Here is the generated whl file:
|
The above fix is now merged and I verified it works using the master branch of auditwheel. Thank you all!
|
Thanks to everyone that worked on adding manylinux 2014 support to auditwheel.
I'm trying to get build a manylinux 2014 whl file of TensorFlow for ppc64le, using the docker image
quay.io/pypa/manylinux2014_ppc64le
This image contains auditwheel 3.0.0.0rc1
auditwheel show says I'm close to being compliant, I just have one external dependency to resolve:
auditwheel repair appears to resolve the issue:
However when I run auditwheel show on the generated file, it list two internal so files as library dependencies:
Any idea what might be wrong in this case? Thank you
gist with debug (-vv) enabled:
auditwheel show of original wheel: https://gist.github.com/wdirons/2ee22c009890e3436caf1c982ab5e81d
auditwheel repair: https://gist.github.com/wdirons/550039b96f9fe8ddbccea5cbec228c3f
auditwheel show of repaired wheel: https://gist.github.com/wdirons/7bb37290e722ede677cd13a8bac0b6a1
The text was updated successfully, but these errors were encountered: