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. Regards, Navdeep > > Modified: > head/usr.sbin/bsdinstall/scripts/services > > Modified: head/usr.sbin/bsdinstall/scripts/services > ============================================================================== > --- head/usr.sbin/bsdinstall/scripts/services Mon Dec 9 22:40:22 2013 > (r259142) > +++ head/usr.sbin/bsdinstall/scripts/services Mon Dec 9 22:56:19 2013 > (r259143) > @@ -29,7 +29,7 @@ > : ${DIALOG_OK=0} > > if [ -f $BSDINSTALL_TMPETC/rc.conf.services ]; then > - eval `sed -e s/YES/on/I -e s/NO/off/I > $BSDINSTALL_TMPETC/rc.conf.services` > + eval `sed -e s/YES/on/i -e s/NO/off/i > $BSDINSTALL_TMPETC/rc.conf.services` > else > # Default service states. Everything is off if not enabled. > sshd_enable="on" > _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"