Tau - improve log handling

On macOS and Linux close stdout, stderr and stdin which have been inherited by the calling process (likely Daemon) before opening our own pointing to the tau log path. This reduces chances from the tau kill switch from hanging on these platforms.

We also need to move our config to the file runtime.exs for the shell variables to be available to configure the logger.

Author: samaaron

app/server/beam/tau/boot-lin.sh

@@ -2,5 +2,21 @@
 set -e # Quit script on error
 SCRIPT_DIR="$(dirname "$0")"
 cd ${SCRIPT_DIR}
+
+# Close standard output file descriptor
+exec 1<&-
+# Close standard error file descriptor
+exec 2<&-
+
+# Close standard in file descriptor
+exec <&-
+
+# Open standard output file for read and write.
+exec 1>>"$9"
+
+# Redirect standard error to standard output
+exec 2>&1
+
 echo "Booting Sonic Pi on Linux..."
-TAU_ENABLED=$1 TAU_INTERNAL=$2 TAU_MIDI_ENABLED=$3 TAU_LINK_ENABLED=$4 TAU_IN_PORT=$5 TAU_API_PORT=$6 TAU_SPIDER_PORT=$7 TAU_DAEMON_PORT=$8 _build/"${MIX_ENV:-prod}"/rel/tau/bin/tau start
+
+TAU_ENABLED=$1 TAU_INTERNAL=$2 TAU_MIDI_ENABLED=$3 TAU_LINK_ENABLED=$4 TAU_IN_PORT=$5 TAU_API_PORT=$6 TAU_SPIDER_PORT=$7 TAU_DAEMON_PORT=$8 TAU_LOG_PATH=$9 _build/"${MIX_ENV:-prod}"/rel/tau/bin/tau start

app/server/beam/tau/boot-mac.sh

@@ -2,5 +2,21 @@
 set -e # Quit script on error
 SCRIPT_DIR="$(dirname "$0")"
 cd ${SCRIPT_DIR}
+
+# Close standard output file descriptor
+exec 1<&-
+# Close standard error file descriptor
+exec 2<&-
+
+# Close standard in file descriptor
+exec <&-
+
+# Open standard output file for read and write.
+exec 1>>"$9"
+
+# Redirect standard error to standard output
+exec 2>&1
+
 echo "Booting Sonic Pi on macOS..."
-TAU_ENABLED=$1 TAU_INTERNAL=$2 TAU_MIDI_ENABLED=$3 TAU_LINK_ENABLED=$4 TAU_IN_PORT=$5 TAU_API_PORT=$6 TAU_SPIDER_PORT=$7 TAU_DAEMON_PORT=$8 _build/"${MIX_ENV:-prod}"/rel/tau/bin/tau start
+
+TAU_ENABLED=$1 TAU_INTERNAL=$2 TAU_MIDI_ENABLED=$3 TAU_LINK_ENABLED=$4 TAU_IN_PORT=$5 TAU_API_PORT=$6 TAU_SPIDER_PORT=$7 TAU_DAEMON_PORT=$8 TAU_LOG_PATH=$9 _build/"${MIX_ENV:-prod}"/rel/tau/bin/tau start

app/server/beam/tau/boot-win.bat

@@ -17,4 +17,4 @@ set TAU_LOG_PATH=%9%
 
 IF NOT DEFINED MIX_ENV SET "MIX_ENV=prod"
 
-_build\%MIX_ENV%\rel\tau\bin\tau start
+_build\%MIX_ENV%\rel\tau\bin\tau start > "%9%" 2>&1

app/server/beam/tau/config/config.exs renamed to app/server/beam/tau/config/runtime.exs

@@ -1,4 +1,4 @@
-import Config
+ import Config
 
 config :tau,
   handle_otp_reports: true,
@@ -8,5 +8,5 @@ config :logger,
   backends: [{LoggerFileBackend, :error_log}]
 
 config :logger, :error_log,
-  path: System.get_env("TAU_LOG_PATH"),
+  path: System.fetch_env!("TAU_LOG_PATH"),
   level: :info