git.rcrnstn.net - rcrnstn/fzy

Browse code

fzy-tmux: Use named pipe for conveying exit code

On a couple of occasions I've seen the following error being reported by
the fzy-tmux script:
> head: cannot open '/tmp/tmp.qUbDwrPLxQ/ret' for reading: No such file or directory
> /usr/bin/fzy-tmux: line 41: exit: : numeric argument required

The issue seems to be that PATH_FIFO_RET is not yet created but we
already try to read from it, causing the read to fail. With this change
we use a named pipe which we create before the actual tmux invocation,
preventing this case from happening.

Credit for the fix goes to @zsugabubus.

Closes: #143

Daniel Mueller authored on 07/07/2020 01:17:29
Showing 1 changed files

contrib/fzy-tmux index ba7c146..3ddf57b 100755

contrib/fzy-tmux

History View file @ c907a20

@@ -23,7 +23,7 @@ main() {
                              fatal "Unable to create a temporary directory"
                          fi
                     -    mkfifo "${PATH_FIFO_IN}" "${PATH_FIFO_OUT}"
                     +    mkfifo "${PATH_FIFO_IN}" "${PATH_FIFO_OUT}" "${PATH_FIFO_RET}"
                          export TMUX=$(_echo "${TMUX}" | cut -d , -f 1,2)
                          eval "tmux \