>> I did a quick survey of a number of other terminal emulators: tmux, >> xterm, GNU screen, rxvt and dvtm all check the SHELL environment >> variable. Checking SHELL is expected behaviour, and it's a useful one at >> that. >> >> Eric > > Checking SHELL is common behaviour and any user setting SHELL will > expect their terminal emulator to use it, rather than the value from > /etc/passwd. > For those worrying about security, /bin/sh prevents setting SHELL in > restricted mode.
Ok, I will apply the patch.
