You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Enhancement: Creating a socket with a fixed filename for mpv control, in $HOME/.config/invidtui no less, doesn't make sense. (If you wanted to do single instance enforcement, the way to do it would be with the standard PID file idiom.) Create the socket in env(TMPDIR) | '/tmp' + whatever you get back from the golang equivalent of mkdtemp. For bonus points, include the current PID in template. This way a crash doesn't require the user to manually remove the socket.
The text was updated successfully, but these errors were encountered:
Initially I did put the socket in /tmp, but later I put it in the config dir supposedly to better manage socket creation/detection/deletion, and also due to this #16 (comment).
Maybe multiple instances of invidtui could be allowed to run, each with its own socket.
Enhancement: Creating a socket with a fixed filename for mpv control, in $HOME/.config/invidtui no less, doesn't make sense. (If you wanted to do single instance enforcement, the way to do it would be with the standard PID file idiom.) Create the socket in env(TMPDIR) | '/tmp' + whatever you get back from the golang equivalent of mkdtemp. For bonus points, include the current PID in template. This way a crash doesn't require the user to manually remove the socket.
The text was updated successfully, but these errors were encountered: