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.

Sign up for GitHub

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

Jump to bottom

Clean up CMake, fail build for unsupported platforms #41

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Clean up CMake, fail build for unsupported platforms #41

wants to merge 3 commits into from

Conversation

axel-h
Copy link
Member

@axel-h axel-h commented May 5, 2023

Clean up CMake files a bit:

  • check platform variables directly (instead of the strings)
  • fail build for unsupported platforms instead of producing something undefined or errors due to missing settings

@axel-h axel-h force-pushed the patch-axel-7 branch 12 times, most recently from 8962a87 to 66a4651 Compare January 29, 2024 13:25
@axel-h axel-h marked this pull request as draft January 29, 2024 17:45
@axel-h axel-h force-pushed the patch-axel-7 branch 3 times, most recently from 6e61833 to 771372e Compare February 1, 2024 01:00
Axel Heider added 3 commits March 16, 2024 23:39
trivial: fix style
d81b6ae 
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
CMake: fail if platform configuration is missing
9ae7f11 
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
CMake: check platform variables directly
b132db5 
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@axel-h