Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CMake for linux #18

Merged
merged 24 commits into from
Nov 15, 2021
Merged

CMake for linux #18

merged 24 commits into from
Nov 15, 2021

Conversation

2bndy5
Copy link
Member

@2bndy5 2bndy5 commented Jun 12, 2021

The usage of CMake for this library follows suite with the same implementation in the RF24 repo (except the RF24_DRIVER option). The DEBUG macro is renamed DEBUG_LEVEL, so it can be changed from CMake CLI option and not confuse users with the BUILE_TYPE option (whose value is usually DEBUG, RELEASE, etc.)

New build-linux workflow that

  • cross-compiles & packages the library targeting armhf & arm64 architectures
    • packages targeting armhf & arm64 architectures will be automatically uploaded to future releases' assets
  • tests the python wrapper installation (not cross-compiled though)
  • tests the library examples (not the python examples though)

  • Additional improvements to the CI workflow scripts that limit triggers to only when relevant files are modified.
  • python wrapper now includes metadata recognized by pypi.org (specifically the license agreement)
  • The Ncurses examples are now automatically built if libncurses5-dev installation is detected. If libncurses5-dev is not present, then the ncurses examples are not built.

@2bndy5 2bndy5 self-assigned this Jun 12, 2021
@2bndy5 2bndy5 marked this pull request as ready for review November 15, 2021 00:05
@TMRh20 TMRh20 merged commit da2058a into master Nov 15, 2021
@2bndy5
Copy link
Member Author

2bndy5 commented Nov 15, 2021

🧹 🏆 You win!

@2bndy5 2bndy5 deleted the CMake-4-Linux branch June 24, 2022 14:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants