Re: svn commit: r314239 - head/rescue/rescue

2017-02-25 Thread Ngie Cooper (yaneurabeya)

> On Feb 24, 2017, at 23:52, Andriy Gapon  wrote:
> 
> On 25/02/2017 05:11, Ngie Cooper wrote:
>> Author: ngie
>> Date: Sat Feb 25 03:11:08 2017
>> New Revision: 314239
>> URL: https://svnweb.freebsd.org/changeset/base/314239
>> 
>> Log:
>>  Add shutdown/poweroff support to rescue(8)
>> 
>>  shutdown is a safer way to power off than reboot (in general), because of
>>  the added shutdown process that it executes via /etc/rc.shutdown . It was
>>  odd that it was missing from rescue(8) since reboot and friends were
>>  added in past commits.
> 
> Just a note that rescue is typically used in situations where rc.shutdown is 
> of
> little use.  E.g., most likely there won't be any daemons running, etc.

Yeah.. or arguably ones of value. I still like the fact that it’s running 
“service  stop” on /etc/rc.d scripts in reverse to shut things down 
cleanly, as opposed to reboot(8) which just reboots. This can make a bit of a 
difference if you just set up a system and still have mount points setup, etc.

But in most cases, yes, I would totally agree.

-Ngie


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: svn commit: r314239 - head/rescue/rescue

2017-02-24 Thread Andriy Gapon
On 25/02/2017 05:11, Ngie Cooper wrote:
> Author: ngie
> Date: Sat Feb 25 03:11:08 2017
> New Revision: 314239
> URL: https://svnweb.freebsd.org/changeset/base/314239
> 
> Log:
>   Add shutdown/poweroff support to rescue(8)
>   
>   shutdown is a safer way to power off than reboot (in general), because of
>   the added shutdown process that it executes via /etc/rc.shutdown . It was
>   odd that it was missing from rescue(8) since reboot and friends were
>   added in past commits.

Just a note that rescue is typically used in situations where rc.shutdown is of
little use.  E.g., most likely there won't be any daemons running, etc.

>   While here, alias poweroff to shutdown for parity with 
> sbin/shutdown/Makefile
>   
>   MFC after:  2 weeks
>   Sponsored by:   Dell EMC Isilon
> 
> Modified:
>   head/rescue/rescue/Makefile
> 
> Modified: head/rescue/rescue/Makefile
> ==
> --- head/rescue/rescue/Makefile   Sat Feb 25 01:53:45 2017
> (r314238)
> +++ head/rescue/rescue/Makefile   Sat Feb 25 03:11:08 2017
> (r314239)
> @@ -99,7 +99,7 @@ CRUNCH_PROGS_sbin= badsect  
> \
>   mount_udf mount_unionfs newfs   \
>   newfs_msdos nos-tun ping reboot \
>   restore rcorder route routed rtquery rtsol savecore \
> - spppcontrol swapon sysctl tunefs umount
> + shutdown spppcontrol swapon sysctl tunefs umount
>  
>  .if ${MK_ATM} != "no"
>  CRUNCH_PROGS_sbin+= atmconfig
> @@ -166,6 +166,7 @@ CRUNCH_ALIAS_restore= rrestore
>  CRUNCH_ALIAS_dump= rdump
>  CRUNCH_ALIAS_fsck_ffs= fsck_4.2bsd fsck_ufs
>  CRUNCH_ALIAS_geom= glabel gpart
> +CRUNCH_ALIAS_shutdown= poweroff
>  
>  # dhclient has historically been troublesome...
>  CRUNCH_PROGS_sbin+= dhclient
> 


-- 
Andriy Gapon
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r314239 - head/rescue/rescue

2017-02-24 Thread Ngie Cooper
Author: ngie
Date: Sat Feb 25 03:11:08 2017
New Revision: 314239
URL: https://svnweb.freebsd.org/changeset/base/314239

Log:
  Add shutdown/poweroff support to rescue(8)
  
  shutdown is a safer way to power off than reboot (in general), because of
  the added shutdown process that it executes via /etc/rc.shutdown . It was
  odd that it was missing from rescue(8) since reboot and friends were
  added in past commits.
  
  While here, alias poweroff to shutdown for parity with sbin/shutdown/Makefile
  
  MFC after:2 weeks
  Sponsored by: Dell EMC Isilon

Modified:
  head/rescue/rescue/Makefile

Modified: head/rescue/rescue/Makefile
==
--- head/rescue/rescue/Makefile Sat Feb 25 01:53:45 2017(r314238)
+++ head/rescue/rescue/Makefile Sat Feb 25 03:11:08 2017(r314239)
@@ -99,7 +99,7 @@ CRUNCH_PROGS_sbin= badsect
\
mount_udf mount_unionfs newfs   \
newfs_msdos nos-tun ping reboot \
restore rcorder route routed rtquery rtsol savecore \
-   spppcontrol swapon sysctl tunefs umount
+   shutdown spppcontrol swapon sysctl tunefs umount
 
 .if ${MK_ATM} != "no"
 CRUNCH_PROGS_sbin+= atmconfig
@@ -166,6 +166,7 @@ CRUNCH_ALIAS_restore= rrestore
 CRUNCH_ALIAS_dump= rdump
 CRUNCH_ALIAS_fsck_ffs= fsck_4.2bsd fsck_ufs
 CRUNCH_ALIAS_geom= glabel gpart
+CRUNCH_ALIAS_shutdown= poweroff
 
 # dhclient has historically been troublesome...
 CRUNCH_PROGS_sbin+= dhclient
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"