X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 	tests=AWL,BAYES_00,SPF_HELO_PASS
X-Spam-Check-By: sourceware.org
Message-ID: <4B181F65.5010605@towo.net>
Date: Thu, 03 Dec 2009 21:28:21 +0100
From: Thomas Wolff <towo AT towo DOT net>
User-Agent: Thunderbird 2.0.0.23 (Windows/20090812)
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: Base-Files (was Re: Unset TMP/TEMP in profile?)
References: <4B14573C DOT 3040409 AT alice DOT it>  <a13b5a590911301753w74ccf298g669e55fc58d36c50 AT mail DOT gmail DOT com>  <20091201091515 DOT GC30043 AT calimero DOT vinschen DOT de>  <b610e770a6f8ab524e0df3c9ce625e84 DOT squirrel AT mail DOT morrison DOT mine DOT nu>  <a13b5a590912010832t52d7ed9cjd2c093ac7d6ea9a4 AT mail DOT gmail DOT com>  <89d87180772edf0cb5974b7d21ed1f50 DOT squirrel AT mail DOT morrison DOT mine DOT nu>  <20091202093042 DOT GO8059 AT calimero DOT vinschen DOT de>  <4B16A22A DOT 4070402 AT towo DOT net>  <20091202182140 DOT GT8059 AT calimero DOT vinschen DOT de> <20091202183652 DOT GA13591 AT ednor DOT casa DOT cgf DOT cx>
In-Reply-To: <20091202183652.GA13591@ednor.casa.cgf.cx>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

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.
>   
For bash and other sh-alike shells, test is a built-in, but not for tcsh 
- sorry I did not consider this.
So an if statement should be used. I don't see, however, how {} would 
help - if you mean to embed an invocation of test
in an if expression, that doesn't help because test would still be forked.
The expression if ( $?LC_ALL == 0 || "$LC_ALL" == "" ) on the other 
doesn't work if LC_ALL is undefined because
|| inside an expression is not a conditional or, at least not with 
respect to variable substitution.
So I propose to neglect the unlikely possibility of an empty variable 
setting here and just test whether any of them is defined:

if ( $?LC_ALL == 0 && $?LC_CTYPE == 0 && $?LANG == 0 ) setenv LANG C.UTF-8

Thomas

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