search

inducer / pyopencl

OpenCL integration for Python, plus shiny features
http://mathema.tician.de/software/pyopencl
Other
1.05k stars 240 forks source link

setup.py: add headers as dependencies #596

Closed matthiasdiener closed 2 years ago

matthiasdiener commented 2 years ago

Without this, modifications to header files do not trigger a rebuild.

See also https://docs.python.org/3/distutils/setupscript.html#other-options

inducer commented 2 years ago

Sorry, no. distutils is not smart enough to gather dependencies in the way a reasonable build tool might. In addition, this is a half-measure: There are many other headers that really ought to be watched for changes.

546 uses cmake to build, which solves this without having to clutter up setup.py.

inducer commented 2 years ago

To further explain: The only reliable way to build of which I'm aware is to start from scratch every time, i.e.

rm -Rf build; python setup.py develop