Skip to content
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

Merged
merged 9 commits into from Aug 16, 2022

Conversation

steven-johnson
Copy link
Contributor

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?)

Copy link
Member

@alexreinking alexreinking left a 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.

cmake/HalidePythonExtensionHelpers.cmake Outdated Show resolved Hide resolved
cmake/HalidePythonExtensionHelpers.cmake Show resolved Hide resolved
packaging/CMakeLists.txt Outdated Show resolved Hide resolved
@steven-johnson
Copy link
Contributor Author

PTAL

@steven-johnson
Copy link
Contributor Author

PTAL

@alexreinking
Copy link
Member

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.

Looks good, but I really feel the need to prioritize this now.

@steven-johnson steven-johnson merged commit 63d563f into main Aug 16, 2022
alexreinking pushed a commit that referenced this pull request Aug 16, 2022
* 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
@steven-johnson steven-johnson deleted the srj/install-py-helpers branch August 16, 2022 22:15
ardier pushed a commit to ardier/Halide-mutation that referenced this pull request Mar 3, 2024
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants