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
Fix clickhouse-keeper-client symlink #54587
Conversation
Let's keep the symlink, so both cc: @pufit |
I'm afraid simple symlink won't do,
|
Here's what I've done and it works
So, symlinks should be fixed, and not removed. Good catch, thanks! |
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.
Please, bring the symlink back with the correct src clickhouse-keeper
90b66e7
to
7a12488
Compare
@Felixoid You're right, that's definitely a better solution. |
This is an automated comment for commit 7a12488 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
Backport #54587 to 23.8: Fix clickhouse-keeper-client symlink
Since #50964 got merged there's
command which makes the
clickhouse-keeper-client
symlink obsolete.Moreover
clickhouse-keeper
package contains following files:which means that the symlink would be broken anyway (unless some other package provides
/usr/bin/clickhouse
binary)Same applies for
clickhouse-keeper-converter
symlink.Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Remove redundant
clickhouse-keeper-client
symlink