Fix fbi leaking and goello announce issue

dubo-dubon-duponey · Feb 2, 2024 · 7279d82 · 7279d82
1 parent d000bc3
commit 7279d82
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/context/runtime/boot/mdns.sh b/context/runtime/boot/mdns.sh
@@ -9,7 +9,7 @@ mdns::records::add(){
   local host="$2"
   local name="${3:-$host}"
   local port="${4:-9}"
-  local text="${5:-[]}"
+  local text="${5:-[\"\"]}" # XXX Goello bug - if [] the announce if not visible
   _internal_mod_mdns_records+=("$(printf '{"Type": "%s", "Host": "%s", "Name": "%s", "Port": %s, "Text": %s}' "$type" "$host" "$name" "$port" "$text")")
 }
 

diff --git a/context/runtime/boot/onevent.sh b/context/runtime/boot/onevent.sh
@@ -23,8 +23,12 @@ get::url(){
 
 display(){
   local img="$1"
+  local prior
+  prior="$(pidof fbi)"
   # Should be smarter and verify permissions on the ttys or bail out
   fbi -a -noverbose -norandom -T 2 -once "$img"
+  # shellcheck disable=SC2086
+  kill -s KILL $prior
 }
 
 call(){