X-Git-Url: https://iankelling.org/git/?a=blobdiff_plain;f=check-subvol-stale;h=8e7d7547457a4e02f01efeaf4720f4a9bf9a699a;hb=77917a8fbf2032a8b2634a1b3de0879ec45cf213;hp=8d9ac786c6b71644df4708f58b2f88f112caf692;hpb=1059b8ac11f567b9a8eec1e34a7ab6e791721dca;p=distro-setup

diff --git a/check-subvol-stale b/check-subvol-stale
index 8d9ac78..8e7d754 100644
--- a/check-subvol-stale
+++ b/check-subvol-stale
@@ -14,7 +14,7 @@
 # limitations under the License.
 
 
-[[ $EUID == 0 ]] || exec sudo -E "$BASH_SOURCE" "$@"
+[[ $EUID == 0 ]] || exec sudo -E "${BASH_SOURCE[0]}" "$@"
 
 set -eE -o pipefail
 trap 'echo "$0:$LINENO:error: \"$BASH_COMMAND\" returned $?" >&2' ERR
@@ -57,7 +57,7 @@ while true; do
   esac
 done
 
-if [[ ! $@ ]]; then
+if [[ ! $1 ]]; then
   echo "$0: error: expected mountpoint argument"
 fi
 
@@ -100,12 +100,13 @@ for d; do
     echo "$0: error could not find root subvol mount for $dev" >&2
     exit 1
   fi
-  svp=$root_dir/$subvol_dir
+  svp=$root_dir/$subvol_dir # subvolume path
   d "svp=$svp # subvolume path"
 
   snaps=($root_dir/btrbk/$subvol_dir.20*) # Assumes we are in the 21st century.
-  if [[ ! ${snaps[@]} ]]; then
+  if [[ ! ${snaps[*]} ]]; then
     # no snapshots yet
+    # TODO: make this an error and override with a cli flag
     echo "$0: warning: no snapshots found at $root_dir/btrbk/$subvol_dir.20*. this is expected for a brand new volume"
     continue
   fi
@@ -135,7 +136,8 @@ for d; do
       f=${s##*/}
       unix_time=$(date -d $(sed -r  's/(.{4})(..)(.{5})(..)(.*)/\1-\2-\3:\4:\5/' <<<${f#$vol.}) +%s)
       printf "%s %s\n" $unix_time $s
-    done | sort -r | head -n 1 | awk '{print $2}'
+      # sort will fail
+    done | sort -r | head -n 1 | awk '{print $2}' || [[ ${PIPESTATUS[1]} == 2 ]]
            )
   if [[ ! $last_snap ]]; then
     # should not happen.