Merge pull request #19 from Retro-I/systemd-startup-service

Systemd startup service

Author: felixholfelder

SETUP.md

@@ -24,17 +24,16 @@ Im Plymouth Poweroff-Service muss eine Kleinigkeit ausgetauscht werden, damit da
 In der Datei `/usr/lib/systemd/system/plymouth-poweroff.service` muss `--mode=shutdown` durch `--mode=reboot` ersetzt werden.\
 Klingt nach einer Art "Trick-17"... Aber es funktioniert ;)
 
-### Erstelle Autostart-Datei
-Damit die Software nach dem Systemstart von selbst startet, muss eine Autostart-Datei angelegt werden.\
-Unter `/etc/xdg/autostart/retroi.desktop` wird folgendes hinzugefügt:
+### User zur "audio" Gruppe hinzufügen
+Mit dem Befehl wird der aktuelle User zur `audio`-Gruppe hinzugefügt:
 ```
-[Desktop Entry]
-Name=Retro.I
-Type=Application
-Exec=sh -c '$RETROI_DIR/scripts/start.sh >> $HOME/autostart.log 2>&1'
-Terminal=true
+sudo usermod -aG audio $USER
 ```
 
+### Systemd-Datei für Systemstart erstellen
+Damit die Software nach dem Systemstart von selbst startet, muss eine Autostart-Datei angelegt werden.\
+In `/etc/systemd/system/retroi.service` wird der Inhalt der Datei [`retroi.service`](scripts/retroi.service) hinzugefügt.
+
 ### Taskbar ausblenden
 Die Datei `$HOME/.config/wf-panel-pi.ini` wird angepasst und folgendes hinzugefügt:
 ```

components/dialogs/SettingsShutdownDialog.py

@@ -53,7 +53,7 @@ def __init__(self):
                                 ft.IconButton(
                                     ft.icons.HIGHLIGHT_OFF,
                                     icon_size=75,
-                                    on_click=system_helper.stopp_app,
+                                    on_click=system_helper.stop_app,
                                 ),
                                 ft.Text(
                                     "App beenden",

helper/Audio.py

@@ -30,7 +30,7 @@ def init_sound(self):
     def mixer(self):
         with open(self.mixers_path, "w") as f:
             f.write(str(a.mixers()))
-        return a.Mixer()
+        return a.Mixer("Master")
 
     def update_sound(self, value):
         if 0 <= value <= 100:

helper/SystemHelper.py

@@ -35,7 +35,7 @@ def restart_system(self, _):
         time.sleep(3)
         os.system("sudo reboot")
 
-    def stopp_app(self, _):
+    def stop_app(self, _):
         PageState.page.window_destroy()
         time.sleep(0.5)
         os._exit(0)

scripts/retroi.desktop

@@ -0,0 +1,21 @@
+[Unit]
+Description=Retro.I Desktop App
+After=graphical.target
+
+[Service]
+Type=simple
+User=pi
+Group=pi
+WorkingDirectory=$RETROI_DIR
+ExecStart=$RETROI_DIR/.venv/bin/python $RETROI_DIR/main.py
+Restart=no
+Environment=DISPLAY=:0
+Environment=XAUTHORITY=/home/pi/.Xauthority
+Environment=XDG_RUNTIME_DIR=/run/user/$user_id
+Environment=PULSE_SERVER=unix:/run/user/$user_id/pulse/native
+TimeoutStopSec=1
+KillSignal=SIGKILL
+KillMode=control-group
+
+[Install]
+WantedBy=graphical.target

scripts/retroi.service

@@ -108,22 +108,46 @@ remove_splashscreen() {
   fi
 }
 
-create_autostart_file() {
-  autostart_path="/etc/xdg/autostart/retroi.desktop"
-
-  sudo tee "$autostart_path" > /dev/null <<EOF
-[Desktop Entry]
-Name=Retro.I
-Type=Application
-Exec=sh -c '$RETROI_DIR/scripts/start.sh >> $HOME/autostart.log 2>&1'
-Terminal=true
+apply_audio_group() {
+  sudo usermod -aG audio $USER
+}
+
+create_systemd_service() {
+  systemd_path="/etc/systemd/system/retroi.service"
+
+  user_id=$(id -u)
+
+  sudo tee "$systemd_path" > /dev/null <<EOF
+[Unit]
+Description=Retro.I Desktop App
+After=graphical.target
+
+[Service]
+Type=simple
+User=pi
+Group=pi
+WorkingDirectory=$RETROI_DIR
+ExecStart=$RETROI_DIR/.venv/bin/python $RETROI_DIR/main.py
+Restart=no
+Environment=DISPLAY=:0
+Environment=XAUTHORITY=/home/pi/.Xauthority
+Environment=XDG_RUNTIME_DIR=/run/user/$user_id
+Environment=PULSE_SERVER=unix:/run/user/$user_id/pulse/native
+TimeoutStopSec=1
+KillSignal=SIGKILL
+KillMode=control-group
+
+[Install]
+WantedBy=graphical.target
 EOF
 
   # Verify creation
-  if ! grep -q -- "^\[Desktop Entry\]" "$autostart_path"; then
-    echo "Autostart file could not be created correctly!" >&2
+  if ! grep -q -- "^\[Service\]" "$systemd_path"; then
+    echo "Systemd file could not be created correctly!" >&2
     return 1
   fi
+
+  sudo systemctl enable retroi.service
 }
 
 hide_taskbar() {
@@ -147,6 +171,11 @@ EOF
 remove_background_image() {
   pcmanfm --set-wallpaper "" --wallpaper-mode=color
 
+  mkdir -p "$HOME/.config/pcmanfm"
+
+  pcmanfm --reconfigure
+  sleep 0.5
+
   CONFIG_FILES=$(find "$HOME/.config/pcmanfm" -type f -name "desktop-items-*.conf" 2>/dev/null)
 
   if [ -z "$CONFIG_FILES" ]; then
@@ -374,7 +403,8 @@ read -p "Drücke <ENTER> um das Setup zu beginnen..."
 set_project_path
 run_step "Entferne Splashscreen" remove_splashscreen
 run_step "System-Splashscreen ändern" sudo -E bash -c "$RETROI_DIR/update-system-splash.sh"
-run_step "Erstelle Autostart-Datei" create_autostart_file
+run_step "User zur \"audio\" Gruppe hinzufügen" apply_audio_group
+run_step "Systemd-Datei für Systemstart erstellen" create_systemd_service
 run_step "Taskbar ausblenden" hide_taskbar
 run_step "Hintergrund entfernen" remove_background_image
 run_step "Mülleimer entfernen" remove_trash_basket