[tz] [PATCH 3/3] tzselect: use /bin/sh by default

Alan Barrett apb at cequrux.com
Sun Oct 28 07:15:57 UTC 2012


On Sat, 27 Oct 2012, Paul Eggert wrote:
>On 10/27/2012 02:06 AM, Mike Frysinger wrote:
>> +	    eval echo \""$b$c) \${$c}"\" >&2
>
>That looks pretty tricky.

Yes, and there's no reason for $b$c to be expanded twice.  But 
it's easy to make it clearer:

	eval continent=\"\${$c}\"
	echo "$b$c) ${continent}" >&2

> How about if we just default to Bash instead?

If portable /bin/sh is not feasible, then please use the 
intersection of ksh and bash syntax, and make it easy for the 
choice between ksh and bash to be made at install time.  (I have a 
minor preference for ksh over bash, because it's more likely to be 
available on non-linux systems, but if the choice is configurable 
then that's no big deal.)

--apb (Alan Barrett)


More information about the tz mailing list