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

Correcting the installed redis symlinks in 7.2.4-rc1 #282

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

bjosv
Copy link
Contributor

@bjosv bjosv commented Apr 10, 2024

This is a PR directly to the 7.2 branch.

The make variable ENGINE_NAME (from 3863227) was lost during branching of 7.2 and tag 7.2.4-rc1
resulting in the creation of faulty symlinks:

      INSTALL SYMLINK valkey-serverredis -> valkey-server
      INSTALL SYMLINK valkey-cliredis -> valkey-cli
      INSTALL SYMLINK valkey-benchmarkredis -> valkey-benchmark
      INSTALL SYMLINK valkey-check-rdbredis -> valkey-check-rdb
      INSTALL SYMLINK valkey-check-aofredis -> valkey-check-aof
      INSTALL SYMLINK valkey-sentinelredis -> valkey-sentinel

By just adding the variable we get a minimal diff compared to unstable.

The make variable ENGINE_NAME was lost during branching of 7.2.4-rc1
resulting in the creation of faulty symlinks:
  INSTALL SYMLINK valkey-serverredis -> valkey-server
  INSTALL SYMLINK valkey-cliredis -> valkey-cli
  INSTALL SYMLINK valkey-benchmarkredis -> valkey-benchmark
  INSTALL SYMLINK valkey-check-rdbredis -> valkey-check-rdb
  INSTALL SYMLINK valkey-check-aofredis -> valkey-check-aof
  INSTALL SYMLINK valkey-sentinelredis -> valkey-sentinel

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
@zuiderkwast zuiderkwast added the bug Something isn't working label Apr 10, 2024
@madolson madolson merged commit 5449089 into valkey-io:7.2 Apr 10, 2024
40 of 43 checks passed
@bjosv bjosv deleted the fix-symlinks branch April 11, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants