You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It seems that cython has a --depfile argument and cmake's add_custom_command has a DEPFILE argument meant for this sort of thing. I'm not aware of the historical context, but perhaps replacing the custom regex parsing logic in UseCython.cmake with the DEPFILE functionality would both reduce maintenance burden and increase robustness?
The text was updated successfully, but these errors were encountered:
Cool. It's actually useful for me to have that broken out to a separate repository as well, as I've recently written cmake code to replace the python-driven build machinery (which is otherwise completely opaque to the cmake dependency graph, and rebuilds native libraries I've already built in a nested cmake invocation).
It seems that
cython
has a--depfile
argument and cmake'sadd_custom_command
has aDEPFILE
argument meant for this sort of thing. I'm not aware of the historical context, but perhaps replacing the custom regex parsing logic inUseCython.cmake
with the DEPFILE functionality would both reduce maintenance burden and increase robustness?The text was updated successfully, but these errors were encountered: