-
Notifications
You must be signed in to change notification settings - Fork 50
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
Meson #108
base: master
Are you sure you want to change the base?
Conversation
|
||
# Google test. | ||
gtest_dep = dependency('gtest', main : true, required : false) | ||
if not gtest_dep.found() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Meson supports has a GTest wrapDB?
# Configuring headers # | ||
####################### | ||
racon_version_commit = 'unknown' | ||
git_command = find_program('git', required: false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
have you tried vcs_tag
?
if not meson.is_subproject() | ||
racon_bin = executable( | ||
'racon', | ||
['src/main.cpp'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
files(['src/main.cpp'])
(does an implicit check whether the file exists)
…is added here which just wraps the one-liner usage.
…renamed the rule to run Meson to 'meson'. Those two generate different build directories, and the CMake build still defaults to 'build' for legacy purposes.
…nfigured from version.hpp.in, and now also includes commit ID, which is important for reproducibility.
… tests correctly.
This PR adds the Meson build support to the project, and also extends the version number with the exact commit ID, which should help reproducibility.