On Dec 2 13:36, Christopher Faylor wrote: > On Wed, Dec 02, 2009 at 07:21:40PM +0100, Corinna Vinschen wrote: > >On Dec 2 18:21, Thomas Wolff wrote: > >> Corinna Vinschen wrote in another thread about setting LANG: > >> >>... Andy and Thomas, please work > >> >>out the best solution together. It should work in sh and csh. Then > >> >>post it as reply to http://cygwin.com/ml/cygwin/2009-12/msg00090.html so > >> >>John can put it into the base-files package. > >> Our worked-out proposal is as follows: > >> > >> > >> /etc/profile.d/lang.sh: > >> > >> # if no locale variable is set, indicate terminal charset via LANG > >> test -z "${LC_ALL:-${LC_CTYPE:-$LANG}}" && export LANG=C.UTF-8 > >> > >> /etc/profile.d/lang.csh: > >> > >> # if no locale variable is set, indicate terminal charset via LANG > >> ( test $?LC_ALL = 0 || test -z "$LC_ALL" ) && ( test $?LC_CTYPE = 0 || > >> test -z "$LC_CTYPE" ) && ( test $?LANG = 0 || test -z "$LANG" ) && setenv > >> LANG C.UTF-8 > > > >Thanks to both of you. > > I wasn't paying attention before, so I apologize for not commenting > sooner but couldn't all of the above be one test statement or at least > handled inside '{}' rather than '()'. Maybe bash optimizes that away > but, if it doesn't, then forking subshells will be rather expensive on > Cygwin.
The statement you're referring to only affects tcsh, not bash. But you're right, nevertheless. I would simplify this to /etc/profile.d/lang.csh: if ( $?LC_ALL == 0 && $?LC_CTYPE == 0 && $?LANG == 0 ) setenv LANG C.UTF-8 This works without forking any child process. And the user might actually want to set one of these variables to an empty string. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader cygwin AT cygwin DOT com Red Hat -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple