You should follow the modern guidelines to specify the dependencies etc, with pyproject.toml <https://packaging.python.org/en/latest/guides/section-build-and-publish/> AFAIK, # distutils tags in Cython header is the usual setuptools way.
I would also say, switch from setuptools to meson or pybind11/nanobind Dima On January 12, 2026 11:26:51 AM CST, Nils Bruin <[email protected]> wrote: >OK, it seems like a "# distutils: libraries = pari" at the top of the pyx >file does the trick, as suggested by the cython docs. Still interested in >hearing if it's a good idea to solve it like that. > >On Monday, 12 January 2026 at 09:13:12 UTC-8 Nils Bruin wrote: > >> I ran into an issue with https://github.com/nbruin/RiemannTheta where >> installing it into modern sage completes without an error but where I get: >> >> sage: import riemann_theta.riemann_theta >> ImportError: >> /usr/local/sage/sage-git/local/var/lib/sage/venv-python3.13/lib64/python3.13/site-packages/riemann_theta/ >> riemann_theta.cpython-313-x86_64-linux-gnu.so: undefined symbol: pari_err >> >> Clearly, libpari doesn't get linked. Previously it did, so this is >> probably due to some tightening of dependency determination in cython or >> python's setup machinery. >> >> The setup.py for this project is below. This used to work. Is there an >> obvious way in which I should change it? Particularly is there something >> sage-specific that can help here? The package in question really depends >> quite closely on a lot of sage so it would need to be built within its venv >> anyway. >> >> -------------------------------------- >> import setuptools >> from Cython.Build import cythonize >> import numpy >> >> with open("README.md", "r") as fh: >> long_description = fh.read() >> >> setuptools.setup( >> name='RiemannTheta', >> version="1.0.0", >> author="Nils Bruin, Sohrab Ganjian", >> author_email="[email protected]", >> license="GPL2+", >> description="Evaluate Riemann Theta function numerically in Sagemath", >> long_description=long_description, >> long_description_content_type="text/markdown", >> url="", >> packages=setuptools.find_packages(), >> ext_modules=cythonize("riemann_theta/riemann_theta.pyx"), >> include_dirs=[numpy.get_include()], >> zip_safe=False, >> ) >> ----------------------------------------------- >> >> I guess there need to be some "libraries" and possibly "library_dirs" >> added to the config but hardcoding them is probably not a good idea. >> Suggestions? (ChatGPT suggests running "pari-config" but I think it just >> came up with that because it looked convenient) >> >> There may be other third party packages out there that quietly broke due >> to some backwards incompatible changes in the python/sage build system. >> > >-- >You received this message because you are subscribed to the Google Groups >"sage-devel" group. >To unsubscribe from this group and stop receiving emails from it, send an >email to [email protected]. >To view this discussion visit >https://groups.google.com/d/msgid/sage-devel/5b7b0d27-58d4-46d8-96de-864d5d912af9n%40googlegroups.com. -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion visit https://groups.google.com/d/msgid/sage-devel/4DB8FF52-654C-4BA1-AF02-073E78436D38%40gmail.com.
