Provide sane defaults

diff --git a/ChangeLog b/ChangeLog
index ac3615d310eefd5e41eb71361e255c52bdac6a70..23b85ca195fbcfd9de226fe361cc30e544fdbb82 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,8 @@
   If mythtv-update-motd is run via interactive shell, always try to run,
   regardless of the flags.
 
+  Provide, hopefully sane, defaults in health-check.
+
 2023-07-27  Andrew Ruthven
   Install the systemd files into the UsrMerge location so we'll work with
   more distributions, thanks to Göran Uddeborgfor pointing this out.

diff --git a/bin/health-check b/bin/health-check
index 8dc0164bb3fe184d2fb6f36f8e70d6d2ad083c5c..bbcda107907e6776cc3a3ccaf3a480c1133e0c24 100755 (executable)
--- a/bin/health-check
+++ b/bin/health-check
@@ -10,12 +10,12 @@
 
 . /etc/default/mythtv-status
 
-if [ "$EMAIL" = "" ] || [ "$EMAIL" = "none" ]
+if [ "$EMAIL" = "none" ]
 then
   exit
 fi
 
-/usr/bin/mythtv-status --host "${HOST:-localhost}" --email "$EMAIL" "$EMAIL_ARGS"
+/usr/bin/mythtv-status --host "${HOST:-localhost}" --email "${EMAIL:-root}" "${EMAIL_ARGS:---email-only-on-alert}"
 
 # Keep things happy by always having a return code of 0.
 exit 0

diff --git a/debian/postinst b/debian/postinst
index 96cc9235fb1e279d5de661326180f6fd452d7922..73d37d192ec341ce3af96852e930c25124eaebc1 100644 (file)
--- a/debian/postinst
+++ b/debian/postinst
@@ -72,7 +72,7 @@ UPDATE_MOTD_ARGS="$UPDATE_MOTD_ARGS"
 # EMAIL:
 #  A comma separated list of email address to send status emails to.
 #
-#  Set to none (or leave empty) to disable daily emails.
+#  Set to none to disable daily emails, if empty then default to root.
 EMAIL=$EMAIL
 
 # EMAIL_ARGS: