-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Export HalidePythonExtensionHelpers.cmake for installs #6941
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to build up some more comprehensive integration testing with our CMake package to catch packaging mistakes. At the very least, it shouldn't all be in my head.
PTAL |
PTAL |
Looks good, but I really feel the need to prioritize this now. |
* Export HalidePythonExtensionHelpers.cmake for installs * oops * fixes * Fix broken code in target_export_script() * oops #2 * Add WITH_SOABI to stubs as well as AOT * More fixes * Update CMakePresets.json * Update CMakePresets.json
* Export HalidePythonExtensionHelpers.cmake for installs * oops * fixes * Fix broken code in target_export_script() * oops halide#2 * Add WITH_SOABI to stubs as well as AOT * More fixes * Update CMakePresets.json * Update CMakePresets.json
The functions in this will be helpful to end-users and should be exported, installed, and documented. (Note that both are actually related to Generators and Python; it might make more sense just to move them to the existing HalideGeneratorHelpers file?)