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

Experiment: Add example CMake and meson build files #2071

Merged
merged 4 commits into from
Mar 1, 2022

Conversation

olafmersmann
Copy link
Contributor

CMake and meson are build systems that have both gained popularity. This PR adds example build files for both so that users might use them as starting points if their project already uses CMake or meson. It should also be a good option for users on Windows where CMake can generate Visual Studio project files.

@olafmersmann olafmersmann changed the base branch from master to development February 18, 2022 23:32
@olafmersmann
Copy link
Contributor Author

Should these be checked in CI?

default_options: ['warning_level=3', 'buildtype=release']
)

add_languages('c')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The idiomatic way to add a language is in the project function.

project('example_experiment', 'c', default_options: ['warning_level=3'])

(Hmm, why isn't the cmake build also setting -Wall -Wextra -Wpedantic?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the feedback! Changed the language specification to the more idiomatic way. Also tried to add warnings to the CMakeLists.txt, but afaik there's no easy and compiler agnostic way to do this.

@olafmersmann olafmersmann force-pushed the ome-c-build-options branch 3 times, most recently from 0f8c4c5 to 1f51c8f Compare February 20, 2022 20:52
@nikohansen nikohansen merged commit 7de2ca4 into numbbo:development Mar 1, 2022
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.

3 participants