svn commit: r259143 - head/usr.sbin/bsdinstall/scripts

[Navdeep Parhar]

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"
>