-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Could not build wheels for XXX which use PEP 517 #1516
Comments
Doing some more Googling on this I came up with the solution described here. One can create a from distutils.core import setup
import numpy
from Cython.Build import cythonize
from setuptools.extension import Extension
global setup_kwargs
extensions = [
Extension('seiya.cube.cubify',
['seiya/cube/cubify.pyx'],
include_dirs=[numpy.get_include()])
]
setup_kwargs = {}
setup_kwargs.update({'ext_modules': cythonize(extensions, annotate=True)})
setup(**setup_kwargs) and remove the need of using IMHO poetry really needs to define a way to deal with extensions. There are many projects that depend on a custom build system and this kind of workarounds are getting old ... |
@sdispater Any comments on this? I'm seeing problems with this workaround with a recent project that requires compilation of extensions and being able to install using |
+1 from me and probably related to #2214 I use |
Thanks @abn for fixing this. However, I cannot completely make it work. If I add
to my
However, if I do
Then it works. Is this expected and if so, should the build-system documentation be updated? I'm running this on a virtual environment using pyenv with Python 3.8.2. Note that the virtual environment has setuptools installed but that doesn't seem to make a difference. |
@albireox what's the version of pip in your environment? Can you try updating it? |
With the latest pip 20.1 the problem still happens (that's what I had been using originally). |
Maybe related to #2276 (comment) |
Same problem working through https://www.pythoncheatsheet.org/blog/python-projects-with-poetry-and-vscode-part-1/ installing pendulum. Conda installs pendulum fine. I use Conda as a pip fallback for things like scipy on Windows. |
@albireox the issue is the isolated environment created by pip when building th wheel. #2666 might help mitigate this issue and so will #2826 since that will mean we rely on virtualenv everytime to build packages. If he issue persists, please create a new issue, so we can track it and discuss it correctly. (#3001 maybe?). |
@abn I tried this again, without the |
Any solution yet? |
@ag-hcoder: I believe the problem here is |
I use python 3.8.0, and this problem still occurs
|
Any updates on this error? |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Issue
#11 provides a workaround for custom build systems (e.g., Cython extensions) that works well with poetry. However, when you try to do
pip install .
on a project that has a custombuild.py
the build process fails withThis means, among other things, that one cannot use ReadTheDocs to document the code. This happens with my code but I have also tested it with pendulum.
Is there a workaround for this?
The text was updated successfully, but these errors were encountered: