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

add version.h generated from the numbers in CMakeLists.txt #1029

Merged
merged 8 commits into from Jan 18, 2022

Conversation

paskino
Copy link
Contributor

@paskino paskino commented Jan 12, 2022

closes #874

Copy link
Member

@KrisThielemans KrisThielemans left a comment

Choose a reason for hiding this comment

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

could add another git_version.h file from git describe as discussed in #874, or leave that for later.

CMakeLists.txt Outdated Show resolved Hide resolved
src/common/tests/CMakeLists.txt Outdated Show resolved Hide resolved
src/common/tests/test_version.cpp Outdated Show resolved Hide resolved
paskino and others added 3 commits January 17, 2022 11:17
Co-authored-by: Kris Thielemans <KrisThielemans@users.noreply.github.com>
Co-authored-by: Kris Thielemans <KrisThielemans@users.noreply.github.com>
Co-authored-by: Kris Thielemans <KrisThielemans@users.noreply.github.com>
@paskino
Copy link
Contributor Author

paskino commented Jan 17, 2022

I wonder what would be the use of git_version.h. It'd add confusion, IMHO.

@paskino paskino merged commit 996fe59 into SyneRBI:master Jan 18, 2022
@paskino paskino deleted the add_C_version_h branch January 18, 2022 15:33
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.

Sirf version to be available at C++ level
2 participants