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

make utilities/CMakeLists.txt respect RF24_NO_IRQ #859

Merged
merged 1 commit into from
Jul 23, 2022
Merged

Conversation

2bndy5
Copy link
Member

@2bndy5 2bndy5 commented Jul 23, 2022

This is needed to update the pyRF24 pkg to use latest master.

This should not negatively affect RF24 lib builds because RF24_NO_INTERRUPT is respected in the root CMakeLists.txt. But, the pyRF24 pkg doesn't actually use the root CMakeLists.txt. Instead, the pyRF24 pkg only uses utility/CMakeLists.txt to get the appropriate driver sources/includes/etc.

This is the CMake equivalent of #839 which did the same exclusion of interrupt.h when it wasn't needed.

This is needed to update the pyRF24 pkg to use latest master.

This does not affect RF24 lib builds because
RF24_NO_IRQ is respected in the root CMakeLists.txt.
But, the pyRF24 pkg doesn't actually use the root CMakeLists.txt.
Instead, the pyRF24 pkg only uses utility/CMakeLists.txt.
@TMRh20 TMRh20 merged commit 89358b2 into master Jul 23, 2022
@2bndy5 2bndy5 deleted the patch-for-pyrf24 branch July 23, 2022 14:44
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.

2 participants