Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 CMake build system for valkey #1196
Add CMake build system for valkey #1196
Changes from 13 commits
f06b624
9e53867
1dbc492
8589132
b5993a0
c64b5d8
f166512
7734d9c
9c81b77
be704ac
5281bb2
f59c5fc
563358a
bfc2991
83a5840
ec9ec96
b5d5a91
86ae036
e9fb7f1
046fda4
0b6db86
da9b98d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
not blocking this PR but can we also document the default settings for these flags?
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.
Will be fixed as part of this small PR: #1276
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.
can we consider a custom versioning scheme for the unstable branch such as
${CPACK_PACKAGE_NAME}-unstable-${CMAKE_SYSTEM_NAME}
? It will look like${CPACK_PACKAGE_NAME}-255.255.255-${CMAKE_SYSTEM_NAME}
otherwise. I am a bit more in favor of explicitness. Separate PR is totally fine.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.
+1 for
${CPACK_PACKAGE_NAME}-unstable-${CMAKE_SYSTEM_NAME}
for unstable branch