This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Base-Files (was Re: Unset TMP/TEMP in profile?)


John?  Ping?

On Dec  2 22:10, Corinna Vinschen wrote:
> 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.

It would be helpful to get a new base-files package this weekend, if
possible.


Thanks,
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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]