mineshaftgap / d4m-nfs

Docker for Mac with NFS for performance improvements over osxfs
296 stars 26 forks source link

Match screen-256color as $TERM for tmux related exit #35

Closed nbr closed 7 years ago

nbr commented 7 years ago

From tmux's FAQ (https://github.com/tmux/tmux/blob/48a3dba6b9863abea9db47a2004009cb6cd68332/FAQ):

Inside tmux TERM must be "screen" or similar (such as "screen-256color").

https://github.com/IFSight/d4m-nfs/pull/33 did not consider that users may also set $TERM to screen-256color.

This change will match both cases.

Currently, if $TERM is set to screen-256color, the script will proceed under tmux and fail as described in #33

if-kenn commented 7 years ago

Ah, good catch