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

add support to changed special character handling #529

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

are-scenic
Copy link

Newer versions of tmux (observed on 3.4, on Ubuntu 24.04) in comparison to the older one (observed on 3.2a, on Ubuntu 22.04) changed behavior in how special characters within the option values are returned by the command tmux show-option -gqv. Now, it adds a leading backslash to each special character.

On Ubuntu 22.04 with tmux 3.2a:

$ tmux -V
tmux 3.2a
$ tmux show-option -gqv '@resurrect-dir'
~/.tmux/resurrect/$HOSTNAME

On Ubuntu 24.04 with tmux 3.4:

$ tmux -V
tmux 3.4
$ tmux show-option -gqv '@resurrect-dir'
~/.tmux/resurrect/\$HOSTNAME

This breaks the expansion of environment variables that is done by the function resurrect_dir() (in #/scripts/helpers.sh).

This commit adds support for this new representation of special characters, in addition to the older ones.

Newer versions of tmux (observed on 3.4, on Ubuntu 24.04) in comparison
to the older one (observed on 3.2a, on Ubuntu 22.04) changed behavior
in how special characters within the option values are returned
by the command `tmux show-option -gqv`. Now, it adds a leading backslash
to each special character.
This breaks the expansion of environment variables that is done by the
function `resurrect_dir()` (in #/scripts/helpers.sh).
This commit adds support for this new representation of special
characters, in addition to the older ones.

Signed-off-by: Arseniy Aharonov <[email protected]>
@are-scenic
Copy link
Author

@bruno-, could you please review the suggested modification?

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.

1 participant