Changeset 10916 in hybrid for branches/releng-9.0

Timestamp:

May 19, 2012, 4:28:15 PM (13 years ago)

Author:

rick

Message:

Make dryrun default as I always forget the type the command.

File:

• branches/releng-9.0/nanobsd/files/tools/sync-etc-changes-to-cf (modified) (5 diffs)

branches/releng-9.0/nanobsd/files/tools/sync-etc-changes-to-cf

@@ -14 +14 @@
   -v     - Verbose diffs
 Arguments:
-  dryrun - Check the pending changes
+  dryrun - Check the pending changes [default]
   sync   - Sync the pending changes
 EOF
@@ -20 +20 @@
 }
 
+SYNC=false
 while [ -n "$1" ]; do
   case "$1" in
@@ -31 +32 @@
       SYNC=true; shift
       ;;
+    "-h")
+      usage; exit 0;;
     *)
       echo "Invalid Argument -- $1"
@@ -37 +40 @@
    esac
 done
-[ -z "$SYNC" ] && usage
 
-trap "umount /cfg; mount -ur /" 1 2 15 EXIT
-mount -r /cfg
-$SYNC && mount -uwo noatime /
+# To sync we need to now the content of /cfg
+mount -r /cfg || exit 1
 
+# Leave disks in consistent state on exit
+trap "umount /cfg; mount -ur /"
+trap "exit 1" 1 2 15
+
+$SYNC && mount -uwo noatime / || exit 1
 for file in `find /conf/base/etc -type f`; do
   live_file=${file##/conf/base}
@@ -56 +62 @@
 done
 
-umount /cfg
-mount -ur /
-trap 1 2 15 EXIT
+if ! $SYNC; then
+  echo "# Dryrun done, commit changes by calling: $0 sync"
+fi