d3adb5 / scripts Goto Github PK

Whenever there are multiple users on the same machine using the screenshot
script, unless TMP_SCREENSHOT_PATH (or whatever it was) is set, they will both
be trying to write screenshots to /dev/shm/screenshot.png.

Since the screenshot file will be used elsewhere and there is no need to waste
RAM (or disk write cycles) with multiple screenshots, we can't simply remove it
at the end of the script, and mktemp will not suffice.

The record script should be able to query X11 for the resolution (and relative position) of the currently focused display as an alternative to using slop to select an area of the screen to record.

It would be useful even if just to have an option for users who are not on a multihead (multiple monitors) setup.

In some repositories (so far only work related ones), the tmux-git-peek script
is invoking some command that errors out and yields a return status of 123.

Either add some "empty" error management, or figure out the root cause of the
error. This probably has to do with preview commands, if FZF propagates errors
instead of containing them in the preview window, or with Git commands that for
one reason or another don't work properly.

It seems that despite the nohup line, when sh is a link to /bin/bash, the
process that launched fzfmenu_run will stick around until the child process
dies instead of terminating as soon as it's started it.

This doesn't seem to be a problem when /bin/sh is a link to dash, an
implementation of the POSIX shell. Perhaps this has to do with the standard and
Bash's relative noncompliance with it, or perhaps this is a particularity of
Dash.

When Bash is the shell running in POSIX compatible mode (i.e. launched through
/bin/sh), an exec in place of nohup will suffice to replace the process
with the one we intend to run. If that is also the case with Dash, the change
will be pushed at some point.