From f25ef7dea86fcf20d08bbead3b451cf593baf5ff Mon Sep 17 00:00:00 2001
From: Thomas Farstrike <thomasverstreken@protonmail.com>
Date: Wed, 11 Mar 2026 21:40:06 +0100
Subject: [PATCH] Simplify run_desktop.sh

---
 scripts/run_desktop.sh | 43 ++++++++----------------------------------
 1 file changed, 8 insertions(+), 35 deletions(-)

diff --git a/scripts/run_desktop.sh b/scripts/run_desktop.sh
index f8fcfc3b..1497cf57 100755
--- a/scripts/run_desktop.sh
+++ b/scripts/run_desktop.sh
@@ -1,9 +1,9 @@
 #!/bin/bash
-scriptdir=$(readlink -f "$0")
-scriptdir=$(dirname "$scriptdir")
+
+scriptdir=$(cd "$(dirname "$0")" && pwd -P)
 script="$1"
 if [ -f "$script" ]; then
-	script=$(readlink -f "$script")
+	script="$(cd "$(dirname "$script")" && pwd -P)/$(basename "$script")"
 fi
 
 echo "Usage:"
@@ -14,54 +14,28 @@ echo "$0 appname # starts the app by appname, for example: com.example.helloworl
 #export SDL_WINDOW_FULLSCREEN=true
 
 export HEAPSIZE=8M # default, same a PSRAM on many ESP32-S3 boards
-#export HEAPSIZE=9M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=10M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=11M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=12M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=13M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=14M # 9MB is not enough for slides, 10MB is okay for 5, 16 for 10, 64 for 100s
-#export HEAPSIZE=15M # 9MB is not enough for slides, 10MB is okay for 5, 15 ok for all
-
-# 15 works infinite with 8 images
-# 12 seems to work fine with all images now, doing only gc.collect()
-# 10-11 works infinite with 7 images but as soon as I add the next one (big PNG slide 2) it hangs memory alloc
-
-# Makes semse because the error is:
-# MemoryError: memory allocation failed, allocating 2518043 bytes
-# So every new slide needs 2.5MB extra RAM!
-# A 1280x720 slide seems to allocate 5530721 bytes, so 6 bytes per pixel... a bit much but okay...
-
-# Fixed by adding lv.image.cache_drop(None) # This helps a lot!
-
-#export HEAPSIZE=16M # 8-10MB is not enough for 1280x720 slides, 16MB seems enough in windowed mode
-
 #export HEAPSIZE=64M # fine for fullscreen 1280x720 slides
 
-
-# print os and set binary
 os_name=$(uname -s)
 if [ "$os_name" = "Darwin" ]; then
 	echo "Running on macOS"
 	binary="$scriptdir"/../lvgl_micropython/build/lvgl_micropy_macOS
 else
-	# other cases can be added here
 	echo "Running on $os_name"
 	binary="$scriptdir"/../lvgl_micropython/build/lvgl_micropy_unix
 fi
-
-binary=$(readlink -f "$binary")
+binary="$(cd "$(dirname "$binary")" && pwd -P)/$(basename "$binary")"
 chmod +x "$binary"
 
 pushd "$scriptdir"/../internal_filesystem/
 
 if [ -f "$script" ]; then
 	echo "Running script $script"
-	"$binary"  -v -i "$script"
+	"$binary" -v -i "$script"
 else
 	CONFIG_FILE="data/com.micropythonos.settings/config.json"
-	if [ ! -z "$script" ]; then
+	if [ -n "$script" ]; then
 		echo "run_desktop.sh: running app $script"
-		# Check if config.json exists
 		if [ -f "$CONFIG_FILE" ]; then
 			if grep -q '"auto_start_app"' "$CONFIG_FILE"; then
 				echo "Updating auto_start_app field using sed"
@@ -71,8 +45,7 @@ else
 				sed -i.backup -E 's/[[:space:]]*}[[:space:]]*$/,"auto_start_app": "'$script'"}/' "$CONFIG_FILE"
 			fi
 		else
-			# If config.json doesn't exist, create it with auto_start_app
-			mkdir -p data/com.micropythonos.settings
+			mkdir -p "$(dirname "$CONFIG_FILE")"
 			echo '{"auto_start_app": "'$script'"}' > "$CONFIG_FILE"
 		fi
 	else
@@ -81,7 +54,7 @@ else
 			sed -i.backup -E 's/[[:space:]]*,?[[:space:]]*"auto_start_app"[[:space:]]*:[[:space:]]*"[^"]*"[[:space:]]*//g; s/\{[[:space:]]*,/\{/g; s/,[[:space:]]*\}/\}/g' "$CONFIG_FILE"
 		fi
 	fi
-	"$binary" -X heapsize=$HEAPSIZE  -v -i -c "$(cat main.py)"
+	"$binary" -X heapsize=$HEAPSIZE -v -i -c "$(cat main.py)"
 fi
 
 popd