foreground

Author: mkalkbrenner

README.md

@@ -218,8 +218,16 @@ platforms/linux/install-gnome-autostart.sh -- -c examples/t2.yml -n -i
 ```
 
 This writes `~/.config/autostart/ppuc-pinmame.desktop` with absolute paths to the
-generated `ppuc/ppuc-pinmame` binary and the selected config file. Pass the same
-arguments you normally use on the command line after `--`.
+generated launcher and starts `ppuc-pinmame` in a terminal window so the process
+stays attached to the foreground session. Pass the same arguments you normally
+use on the command line after `--`.
+
+By default the installer prefers `gnome-terminal`, then `kgx`, then
+`x-terminal-emulator`. Override that with:
+
+```shell
+platforms/linux/install-gnome-autostart.sh --terminal gnome-terminal -- -c examples/t2.yml -n -i
+```
 
 Remove the autostart entry with:
 

platforms/linux/autostart/ppuc-pinmame.desktop.in

@@ -4,8 +4,7 @@ Version=1.0
 Name=PPUC PinMAME
 Comment=Start ppuc-pinmame automatically when the GNOME session begins
 Terminal=false
-Path=@PPUC_WORKDIR@
-Exec=@PPUC_EXEC@
+Exec=@AUTOSTART_EXEC@
 X-GNOME-Autostart-enabled=true
 StartupNotify=false
 Categories=Game;

platforms/linux/install-gnome-autostart.sh

@@ -5,21 +5,24 @@ set -euo pipefail
 usage() {
   cat <<'USAGE'
 Usage:
-  platforms/linux/install-gnome-autostart.sh [--repo-root PATH] [--autostart-name NAME] -- <ppuc-pinmame args...>
+  platforms/linux/install-gnome-autostart.sh [--repo-root PATH] [--autostart-name NAME] [--terminal CMD] -- <ppuc-pinmame args...>
 
 Examples:
   platforms/linux/install-gnome-autostart.sh -- -c examples/t2.yml -n -i
   platforms/linux/install-gnome-autostart.sh --repo-root /opt/ppuc -- -c /opt/ppuc/examples/t2.yml -n -i
+  platforms/linux/install-gnome-autostart.sh --terminal kgx -- -c examples/t2.yml -n -i
 
 Notes:
   - The script writes ~/.config/autostart/<name>.desktop for GNOME/XDG autostart.
+  - It also writes a small launcher script under ~/.local/share/ppuc/autostart/.
   - The repo build output is expected at <repo-root>/ppuc/ppuc-pinmame.
   - Any relative paths after -- are resolved from <repo-root> before being written.
 USAGE
 }
 
 repo_root="$(pwd)"
 autostart_name="ppuc-pinmame"
+terminal_cmd=""
 
 while [[ $# -gt 0 ]]; do
   case "$1" in
@@ -33,6 +36,11 @@ while [[ $# -gt 0 ]]; do
       [[ $# -gt 0 ]] || { echo "Missing value for --autostart-name" >&2; exit 1; }
       autostart_name="$1"
       ;;
+    --terminal)
+      shift
+      [[ $# -gt 0 ]] || { echo "Missing value for --terminal" >&2; exit 1; }
+      terminal_cmd="$1"
+      ;;
     --help|-h)
       usage
       exit 0
@@ -62,6 +70,9 @@ binary="$workdir/ppuc-pinmame"
 template="$repo_root/platforms/linux/autostart/ppuc-pinmame.desktop.in"
 autostart_dir="${XDG_CONFIG_HOME:-$HOME/.config}/autostart"
 desktop_file="$autostart_dir/$autostart_name.desktop"
+data_home="${XDG_DATA_HOME:-$HOME/.local/share}"
+launcher_dir="$data_home/ppuc/autostart"
+launcher_script="$launcher_dir/$autostart_name.sh"
 
 if [[ ! -x "$binary" ]]; then
   echo "Expected executable not found: $binary" >&2
@@ -93,16 +104,57 @@ for arg in "$@"; do
   esac
 done
 
-printf -v quoted_exec '%q ' "$binary" "${resolved_args[@]}"
-quoted_exec="${quoted_exec% }"
+if [[ -z "$terminal_cmd" ]]; then
+  if command -v gnome-terminal >/dev/null 2>&1; then
+    terminal_cmd="gnome-terminal"
+  elif command -v kgx >/dev/null 2>&1; then
+    terminal_cmd="kgx"
+  elif command -v x-terminal-emulator >/dev/null 2>&1; then
+    terminal_cmd="x-terminal-emulator"
+  else
+    echo "No supported terminal emulator found. Use --terminal <command>." >&2
+    exit 1
+  fi
+fi
+
 mkdir -p "$autostart_dir"
+mkdir -p "$launcher_dir"
+
+{
+  echo "#!/bin/bash"
+  echo "set -euo pipefail"
+  echo "cd $(printf '%q' "$workdir")"
+  printf 'exec %q' "$binary"
+  for arg in "${resolved_args[@]}"; do
+    printf ' %q' "$arg"
+  done
+  printf '\n'
+} > "$launcher_script"
+
+chmod 755 "$launcher_script"
+
+case "$terminal_cmd" in
+  gnome-terminal)
+    printf -v autostart_exec '%q --wait -- %q' "$terminal_cmd" "$launcher_script"
+    ;;
+  kgx)
+    printf -v autostart_exec '%q -- %q' "$terminal_cmd" "$launcher_script"
+    ;;
+  x-terminal-emulator)
+    printf -v autostart_exec '%q -e %q' "$terminal_cmd" "$launcher_script"
+    ;;
+  *)
+    printf -v autostart_exec '%q -e %q' "$terminal_cmd" "$launcher_script"
+    ;;
+esac
 
 sed \
-  -e "s|@PPUC_WORKDIR@|$workdir|g" \
-  -e "s|@PPUC_EXEC@|$quoted_exec|g" \
+  -e "s|@AUTOSTART_EXEC@|$autostart_exec|g" \
   "$template" > "$desktop_file"
 
 chmod 644 "$desktop_file"
 
 echo "Installed GNOME autostart entry: $desktop_file"
-echo "Command: $quoted_exec"
+echo "Launcher: $launcher_script"
+echo "Terminal: $terminal_cmd"
+echo "Command: $binary ${resolved_args[*]}"

platforms/linux/uninstall-gnome-autostart.sh

@@ -13,7 +13,8 @@ Examples:
 
 Notes:
   - The script removes ~/.config/autostart/<name>.desktop.
-  - It only removes the per-user GNOME/XDG autostart entry and leaves the build output untouched.
+  - It also removes the generated launcher script under ~/.local/share/ppuc/autostart/.
+  - It only removes per-user autostart artifacts and leaves the build output untouched.
 USAGE
 }
 
@@ -41,11 +42,23 @@ done
 
 autostart_dir="${XDG_CONFIG_HOME:-$HOME/.config}/autostart"
 desktop_file="$autostart_dir/$autostart_name.desktop"
+data_home="${XDG_DATA_HOME:-$HOME/.local/share}"
+launcher_script="$data_home/ppuc/autostart/$autostart_name.sh"
 
-if [[ ! -e "$desktop_file" ]]; then
-  echo "Autostart entry not found: $desktop_file"
-  exit 0
+removed_any=0
+
+if [[ -e "$desktop_file" ]]; then
+  rm -f "$desktop_file"
+  echo "Removed GNOME autostart entry: $desktop_file"
+  removed_any=1
+fi
+
+if [[ -e "$launcher_script" ]]; then
+  rm -f "$launcher_script"
+  echo "Removed launcher script: $launcher_script"
+  removed_any=1
 fi
 
-rm -f "$desktop_file"
-echo "Removed GNOME autostart entry: $desktop_file"
+if [[ "$removed_any" -eq 0 ]]; then
+  echo "Autostart entry not found for name: $autostart_name"
+fi