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>