Re: [9front] -d option for fshalt

[Alex Musolino <alex@musolino.id.au>]

> -		echo usage: $0 '[-r]' >[1=2]
> +		echo usage: $0 '[-rd]' >[1=2]

Good catch.  I opted for '[-d] [-r]' to match what I have in
the man page.  The were separated in the first place so as not
to provide a misleading example; Fshalt will *not* parse the
individual flags out of a single '-rd' argument.

Updated patch follows.

diff -r ff6b294d07a9 rc/bin/fshalt
--- a/rc/bin/fshalt	Mon Jun 01 00:07:01 2020 +0200
+++ b/rc/bin/fshalt	Thu Jun 04 16:22:02 2020 +0930
@@ -1,16 +1,20 @@
 #!/bin/rc
-# fshalt [-r] - sync (flush) and, if possible, halt all file servers
-#	and optionally reboot
 rfork en
+dump=no
 reboot=no
 scram=no
-switch ($#*) {
-case 0
-case 1
-	reboot=yes
-case *
-	echo usage: $0 '[-r]' >[1=2]
-	exit usage
+
+while (~ $1 -*) {
+	switch ($1) {
+	case -d
+		dump=yes
+	case -r
+		reboot=yes
+	case *
+		echo usage: $0 '[-d] [-r]' >[1=2]
+		exit usage
+	}
+	shift
 }
 
 path=(/bin)
@@ -52,6 +56,13 @@
 # put this in a shell function so this rc script doesn't get read
 # when it's no longer accessible
 fn x {
+	if (~ $dump yes) {
+		echo
+		echo -n dumping...
+		for (i in $c $h)
+			echo dump >>$i
+	}
+
 	echo
 	echo -n halting...
 	for (i in $c $h)
diff -r ff6b294d07a9 sys/man/8/fshalt
--- a/sys/man/8/fshalt	Mon Jun 01 00:07:01 2020 +0200
+++ b/sys/man/8/fshalt	Thu Jun 04 16:22:02 2020 +0930
@@ -4,6 +4,8 @@
 .SH SYNOPSIS
 .B fshalt
 [
+.B -d
+] [
 .B -r
 ]
 .br
@@ -20,13 +22,15 @@
 and
 .IR hjfs (4)
 servers.
-If given
-.BR -r ,
-.I fshalt
-will then reboot the machine.
-Else it will invoke
+The
+.BR -d
+option causes a dump command to be issued to the local filesystem
+before the sync and halt commands.  Once any filesystems have been
+halted,
 .I scram
-to shut down the machine.
+will be invoked to shut down the machine, unless the
+.BR -r
+option is specified, in which case the machine is instead rebooted.
 The halting and rebooting is done by copying all necessary
 commands into a
 .IR ramfs (4)