Use parent STDOUT for logging output

Fixes issues introduced by #169

Author: PaulSD

lib/log.sh

@@ -7,6 +7,11 @@
 # to be included in add-on scripts to reduce code duplication across add-ons.
 # ==============================================================================
 
+# Preserve the original STDOUT on a free fd so that we can log to it without
+# interfering with the STDOUT of subshells and child processes whose output we
+# want to capture for other purposes.
+exec {LOG_FD}>&1
+
 # ------------------------------------------------------------------------------
 # Log a message to output.
 #
@@ -15,7 +20,7 @@
 # ------------------------------------------------------------------------------
 bashio::log() {
     local message=$*
-    echo -e "${message}"
+    echo -e "${message}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -27,7 +32,7 @@ bashio::log() {
 # ------------------------------------------------------------------------------
 bashio::log.red() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_RED}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_RED}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -39,7 +44,7 @@ bashio::log.red() {
 # ------------------------------------------------------------------------------
 bashio::log.green() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_GREEN}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_GREEN}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -51,7 +56,7 @@ bashio::log.green() {
 # ------------------------------------------------------------------------------
 bashio::log.yellow() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_YELLOW}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_YELLOW}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -63,7 +68,7 @@ bashio::log.yellow() {
 # ------------------------------------------------------------------------------
 bashio::log.blue() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_BLUE}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_BLUE}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -75,7 +80,7 @@ bashio::log.blue() {
 # ------------------------------------------------------------------------------
 bashio::log.magenta() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_MAGENTA}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_MAGENTA}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -87,7 +92,7 @@ bashio::log.magenta() {
 # ------------------------------------------------------------------------------
 bashio::log.cyan() {
     local message=$*
-    echo -e "${__BASHIO_COLORS_CYAN}${message}${__BASHIO_COLORS_RESET}"
+    echo -e "${__BASHIO_COLORS_CYAN}${message}${__BASHIO_COLORS_RESET}" >&"$LOG_FD"
     return "${__BASHIO_EXIT_OK}"
 }
 
@@ -115,7 +120,7 @@ function bashio::log.log() {
     output="${output//\{MESSAGE\}/"${message}"}"
     output="${output//\{LEVEL\}/"${__BASHIO_LOG_LEVELS[$level]}"}"
 
-    echo -e "${output}"
+    echo -e "${output}" >&"$LOG_FD"
 
     return "${__BASHIO_EXIT_OK}"
 }