From e256742f75e1127266e4ce1e2a871a81a72bc84a Mon Sep 17 00:00:00 2001
From: Ian Kelling <iank@fsf.org>
Date: Thu, 16 May 2019 12:20:52 -0400
Subject: [PATCH] bug fixes

---
 btrbk-run           | 10 ++++++++--
 mount-latest-subvol |  2 +-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/btrbk-run b/btrbk-run
index 8d6eb28..31649a4 100644
--- a/btrbk-run
+++ b/btrbk-run
@@ -77,7 +77,7 @@ fi
 pre="${0##*/}:"
 cron=false
 orig_args=("$@")
-temp=$(getopt -l cpull-reexec,help cl:m:npqs:t:vh "$@") || usage 1
+temp=$(getopt -l cron,pull-reexec,help cl:m:npqs:t:vh "$@") || usage 1
 eval set -- "$temp"
 while true; do
   case $1 in
@@ -120,9 +120,15 @@ if $verbose; then
 fi
 ### end options parsing
 
+# TODO: i wonder if there should be an option to send to the default
+# targets, plus any given on the command line.
 
 # set default targets
-if [[ ! -v targets && ! $source && $HOSTNAME == "$MAIL_HOST" ]]; then
+if [[ ! -v targets && ! $source ]]; then
+  if [[ $HOSTNAME != "$MAIL_HOST" ]] && $cron ; then
+    echo "We are not MAIL_HOST, nothing to do"
+    exit 0
+  fi
   case $HOSTNAME in
     kw|x2)
       if ping -q -c1 -w1 iank.vpn.office.fsf.org &>/dev/null; then
diff --git a/mount-latest-subvol b/mount-latest-subvol
index a277b55..9bb324b 100644
--- a/mount-latest-subvol
+++ b/mount-latest-subvol
@@ -330,7 +330,7 @@ for vol in q a o i; do
       if [[ $bsub ]]; then
         tmp=$(mktemp)
         # in testing, same subvol is 136 bytes. allow some overhead
-        btrfs send --no-data -p $bsub $leaf | head -c 1000 > $tmp
+        btrfs send --no-data -p $bsub $leaf | head -c 1000 > $tmp || [[ $? == 141 ]]
         if (( $(stat -c%s $tmp) < 1000)); then
           # example output for an empty diff:
           # Found a valid Btrfs stream header, version 1
-- 
2.30.2