netcdf4-python debug installs #943
Comments
|
I saw that this was introduced in #673, which links to "issue #666", but I guess it's in some old issue tracker because it's a different thing. If https://www.python.org/dev/peps/pep-0518/#build-system-table |
|
Turns out I was one step away: diff --git a/setup.py b/setup.py
index c2a76da..e2aec1f 100644
--- a/setup.py
+++ b/setup.py
@@ -2,6 +2,7 @@ import os, sys, subprocess
import os.path as osp
from setuptools import setup, Extension
from distutils.dist import Distribution
+from Cython.Build import cythonize
setuptools_extra_kwargs = {
"install_requires": ["numpy>=1.7","cftime"],
@@ -609,5 +610,5 @@ setup(name="netCDF4",
"Topic :: System :: Archiving :: Compression",
"Operating System :: OS Independent"],
packages=['netCDF4'],
- ext_modules=ext_modules,
+ ext_modules=cythonize(ext_modules, include_path=['include'], gdb_debug=True),
**setuptools_extra_kwargs) |
|
As a last note, for some reason I was getting a really long compilation error when the (perhaps because I'm using Python with debugging symbols enabled and I had to install numpy and cython from source?) This was fixed by removing the |
|
@Juanlu001 this is the setup_requires sandbox vs cython clash most likely (also reported here: pypa/setuptools#1784 ) which happens when your own package uses Cython and has a setup_requires, rather than install_requires, dependency on another Cython-using package. One way to work around this is to get |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I am trying to debug #814 and for that I would like to cythonize the sources with debugging information as suggested in https://cython.readthedocs.io/en/latest/src/userguide/debugging.html#debugging-your-cython-program. However, since afaf0fa#diff-2eeaed663bd0d25b7e608891384b7298,
cythonizeis not used and I don't see where to pass the option. Trying to reintroduce it in a naïve way:results in an error:
The text was updated successfully, but these errors were encountered: