Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Improve fish binding #2174

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
Loading
from
Open

Conversation

mingmingrr
Copy link

Fish 3.1 added $pipestatus, which should be preferable over $status in --last-pipe-status.
Fish 3.0 added $fish_pid, which should be preferable over (random) in --renderer_arg=client_id.
Older versions of fish can fall back on $status and (random).

%self is also available instead of $fish_pid prior to fish 3.0, but I've opted not to use it due to process expansion having been removed at some point, and there being no good way of checking whether %self is usable.

Additionally, test -z/-n and env ara replaced with fish builtins set -q eval where applicable.

Manually tested on NixOS / fish 3.3.1 / tmux 3.2a and Ubuntu / fish 2.1.2 / tmux 3.0a

`eval` breaks on pipes due to improper space handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.