Re: [PATCH] test: skip test with COLUMNS=1 under mksh

From: Stefano Lattarini <stefano.lattarini@gmail.com>
Date: Apr 27, 2012, 22:35 UTC
Message-ID: <4F9B1F38.8040406@gmail.com>
In-Reply-To: <xmqqaa1x9mdi.fsf@junio.mtv.corp.google.com>

Hi Junio, Zbigniew.

On 04/27/2012 06:21 PM, Junio C Hamano wrote:

Show 18 quoted lines

> Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> writes:
> 
>> mksh does not allow $COLUMNS to be set below 12. Quoting mksh(1)
>> $COLUMNS is "always set, defaults to 80, unless the value as reported
>> by stty(1) is non-zero and sane enough". This applies also to setting
>> it directly for one command:
>>
>> $ COLUMNS=30 python -c 'import os; print os.environ["COLUMNS"]'
>> 30
>> $ COLUMNS=20 python -c 'import os; print os.environ["COLUMNS"]'
>> 20
>> $ COLUMNS=10 python -c 'import os; print os.environ["COLUMNS"]'
>> 98
> 
> I wonder if this is an easier workaround, though.
> 
> 	env COLUMNS=10 $cmd
>

It works for me:

  $ mksh -c 'COLUMNS=10 env | grep COLUMNS'
  COLUMNS=113

  $ mksh -c 'env COLUMNS=10 env | grep COLUMNS'
  COLUMNS=10

  $ dpkg -l mksh
  ...
  ii  mksh  40.2-2  MirBSD Korn Shell

HTH,
  Stefano