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

Update version #79

Merged
merged 2 commits into from
Aug 28, 2024
Merged

Update version #79

merged 2 commits into from
Aug 28, 2024

Conversation

mkmkme
Copy link
Contributor

@mkmkme mkmkme commented Aug 27, 2024

This PR contains two commits:

  1. Make LIBVALKEY_SONAME a string to remove clang-format off block and to make it available from the code in case it's needed
  2. Update version to 0.1.0 and make LIBVALKEY_SONAME of a format ${MAJOR}.${MINOR}

Previously it was a literal `1.2.1-dev` that had to be put under
`clang-format off` section because clang-format treated that expression
as subtraction and added spaces around minus sign.

Making it a string, however, required the users of `LIBVALKEY_SONAME`,
namely `Makefile` and `CMakeLists.txt`, to ignore the possible quotes in
the variable.

As a benefit, `LIBVALKEY_SONAME` can be used in the code as well in case
such thing is needed.

Signed-off-by: Mikhail Koviazin <[email protected]>
Additionally make SONAME just in a form of
`${MAJOR_VERSION}.${MINOR_VERSION}`.

Signed-off-by: Mikhail Koviazin <[email protected]>
Copy link
Collaborator

@bjosv bjosv left a comment

Choose a reason for hiding this comment

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

LGTM.
If we skip -dev, then I guess we don't need to handle "
But now it will work fine when/if we do add it..

@bjosv bjosv merged commit d90b75f into valkey-io:main Aug 28, 2024
41 checks passed
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.

2 participants