From 6e5ff0b8c3babe2ea462cd1818dd2647d08a716e Mon Sep 17 00:00:00 2001
From: Mikhail Novosyolov <mikhailnov@nixtux.ru>
Date: Fri, 22 Apr 2022 14:42:14 +0300
Subject: [PATCH] watcher: log xpra runs

---
 doskast-server-watcher.sh   | 6 +++++-
 doskast-trigger-connect.cgi | 1 -
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/doskast-server-watcher.sh b/doskast-server-watcher.sh
index 00d1427..45350ff 100755
--- a/doskast-server-watcher.sh
+++ b/doskast-server-watcher.sh
@@ -17,7 +17,11 @@ _main_server_watcher(){
 		local target
 		target="$(head -n 1 "$dir/$line" | _safe_param_stdin)"
 		test -n "$target"
-		xpra attach tcp://"$target":4648 &
+		# log how xpra is run into ~/.xsession-errors*
+		( set -x ; xpra attach tcp://"$target":4648 ) &
+		# XXX TODO: this fails for now, fix permissions and try to avoid 777
+		# or maybe clean up old files by systemd-tmpfiles
+		#unlink "$dir/$line"
 	done
 }
 
diff --git a/doskast-trigger-connect.cgi b/doskast-trigger-connect.cgi
index 4dc5ddb..7379b1f 100755
--- a/doskast-trigger-connect.cgi
+++ b/doskast-trigger-connect.cgi
@@ -7,7 +7,6 @@ set -f
 set -u
 
 _main_trigger_connect(){
-	local umask_old
 	local tmp
 	tmp="$(umask 0022 && mktemp --tmpdir="$dir" connect.XXXXX)"
 	test -f "$tmp"