Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Dec 2013 18:10:38 -0500
From:      Eitan Adler <eadler@freebsd.org>
To:        Navdeep Parhar <np@freebsd.org>
Cc:        "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>
Subject:   Re: svn commit: r259143 - head/usr.sbin/bsdinstall/scripts
Message-ID:  <CAF6rxg=ZOsqESMsjev=n=rhrb4%2BD91jY8VrTsCg8Pv3qTRcyUg@mail.gmail.com>
In-Reply-To: <52A64CB1.4060201@FreeBSD.org>
References:  <201312092256.rB9MuJ1i027962@svn.freebsd.org> <52A64CB1.4060201@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Dec 9, 2013 at 6:05 PM, Navdeep Parhar <np@freebsd.org> wrote:
> On 12/09/13 14:56, Eitan Adler wrote:
>> Author: eadler
>> Date: Mon Dec  9 22:56:19 2013
>> New Revision: 259143
>> URL: http://svnweb.freebsd.org/changeset/base/259143
>>
>> Log:
>>   sed(1): Use /i instead of /I
>>
>>       the i modifier was added in r259132 since POSIX recently agreed to add
>>       it. Switch uses of /I to /i.
>
> There doesn't seem to be any upside to this sort of change.  The
> downside obviously is that it won't work with older sed's that don't
> understand /i.
>
> If you are considering sweeping
over the entire tree looking for sed
> invocations and "fixing" them then please reconsider.

Understood.


-- 
Eitan Adler
Source, Ports, Doc committer
Bugmeister, Ports Security teams



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxg=ZOsqESMsjev=n=rhrb4%2BD91jY8VrTsCg8Pv3qTRcyUg>