-
Notifications
You must be signed in to change notification settings - Fork 149
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: consider TMUX_PLUGIN_MANAGER_PATH when searching tpm binary #835
Conversation
https://github.com/SteveLauC/topgrade/releases/tag/pr835 There is a build for linux with this patch included, @rubiin, welcome to give it a test. |
I will give it a try asap |
Tried but doesnt do anything |
Hi, would you like to show me where Topgrade should look for the update binary if $ export TMUX_PLUGIN_MANAGER_PATH=/tmp Then where is the update binary? |
You have
and the
so Topgrade should look for |
https://github.com/tmux-plugins/tpm/blob/master/docs/automatic_tpm_installation.md that is to say the root tpm folder so Topgrade should look for $TMUX_PLUGIN_MANAGER_PATH/bin/install_plugins/tpm/bin/update_plugins |
Attached Topgrade build updated, welcome to give it another test. |
sure will get back to you |
What does this PR do
Before this PR, when updating tmux plugins, we only look for the update binary in its default location. The actual location can be altered via environment variable
TMUX_PLUGIN_MANAGER_PATH
so it should be considered as well.fixes #832
Standards checklist
CONTRIBUTING.md