-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
The provided wheels faultily report to be pure Python #16
Comments
bump? |
This bug is commonly encountered when using symengine on Fedora-based systems (including RHEL), under virtualenv or tox (which uses virtualenv under the hood, I have heard). The below Dockerfile shows how simple it is to reproduce, and it makes symengine impossible to uses under these circumstances. FROM fedora:40
RUN dnf install -y pip
RUN pip install virtualenv
RUN virtualenv myenv
RUN ./myenv/bin/pip install symengine
RUN ./myenv/bin/python -c 'import symengine' This test case can be reproduced by installing docker, placing the above text in a
|
Switching to scikit-build would be the best way forward |
I would like to raise the issue symengine/symengine.py#474 to this repo because I believe that the fix for this requires some changes to how the Python wheels are built.
There is a lot in that discussion so let me try to summarize the key points:
symengine
some problems occur with symengine wheels on specific platforms and environment setups (see PDM and Poetry can't install symengine properly symengine.py#474 (comment))lib
andlib64
are symlinked and whether the wheel reports to be pure Python or not (see PDM and Poetry can't install symengine properly symengine.py#474 (comment) and following comments).The
symengine
wheels report as pure Python, although they are not, which I believe has to be fixed for this issue to be resolved. Unfortunately, I do not have the expertise to fix this in the build system ofsymengine
but maybe the maintainers of this repository could help with that.I see that there have been efforts in the past to part the build setup to
scikit-build
, which might prove to be a good path forward:Cc @martinruefenacht
The text was updated successfully, but these errors were encountered: