Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 21 Feb 2009 18:40:33 -0800
From:      Sam Leffler <sam@freebsd.org>
To:        =?ISO-8859-1?Q?G=E1bor_K=F6vesd=E1n?= <gabor@freebsd.org>
Cc:        Perforce Change Reviews <perforce@freebsd.org>, "David E. O'Brien" <obrien@freebsd.org>
Subject:   Re: PERFORCE change 158046 for review
Message-ID:  <49A0BB21.1090706@freebsd.org>
In-Reply-To: <49A0B6AC.9000303@FreeBSD.org>
References:  <200902220119.n1M1JljJ020105@repoman.freebsd.org> <49A0B129.6030308@freebsd.org> <49A0B6AC.9000303@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Gábor Kövesdán wrote:
> Sam Leffler escribió:
>> Gabor Kovesdan wrote:
>>> http://perforce.freebsd.org/chv.cgi?CH=158046
>>>
>>> Change 158046 by gabor@gabor_server on 2009/02/22 01:19:13
>>>
>>>     - Add support for GREP_OPTIONS
>>>       
>>
>> How many apps take options via the environment?  Not sure I like 
>> this--I can imagine very confusing situations caused by 
>> unseen/unnoticed environment variables.
> You're right, it might by dangerous if it is misused, but GNU grep 
> uses it so we should support this to maintain compatibility. And in 
> some cases, it can be really useful, imagine that you always want to 
> colorize the output, you can just set GREP_OPTIONS="--color". I'd say 
> that those who don't like it, can just ignore this and don't use it 
> all, while others who have been using it can keep using it, that's 
> fair enough.
> (CC'd obrien@ as he was the proposer, who reminded me of this missing 
> feature.)
>
I looked at ls(1); it has env variables for specific features (like 
colors).  I can buy that more than an arbitrary set of options.  If we 
want apps to take cmd line args from the environ then imo we should have 
all apps work like that; this seems prone to errors and confusion.  The 
gnu argument doesn't work on me.  But then again I'm old school...

    Sam




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?49A0BB21.1090706>