Merge pull request #237 from Paiusco/create-split-systemd-system-user…

…-service

Create CMake option for systemd system/user service

.github/workflows/push_pr_build_cmake.yml

@@ -48,6 +48,7 @@ jobs:
         run: |
           mv ${{github.workspace}}/build/ydotool ${{github.workspace}}/build/ydotool-${{matrix.os}}
           mv ${{github.workspace}}/build/ydotoold ${{github.workspace}}/build/ydotoold-${{matrix.os}}
+          mv ${{github.workspace}}/build/ydotoold.service ${{github.workspace}}/build/ydotoold-${{matrix.os}}.service
 
       - name: Archive production artifacts
         uses: actions/upload-artifact@v4
@@ -56,4 +57,5 @@ jobs:
           path: |
             ${{github.workspace}}/build/ydotool-${{matrix.os}}
             ${{github.workspace}}/build/ydotoold-${{matrix.os}}
+            ${{github.workspace}}/build/ydotoold-${{matrix.os}}.service
 

CMakeLists.txt

@@ -6,16 +6,6 @@ set(CMAKE_C_STANDARD 11)
 # Include GNU install directory module to detect where to install
 # files on Linux/Unix systems (e.g., lib vs lib64)
 include(GNUInstallDirs)
-find_package(PkgConfig)
-
-# Try to get the systemd directory for user units from the package
-# and fall back on upstream default without the '/usr' prefix
-pkg_check_modules(SYSTEMD systemd)
-if (SYSTEMD_FOUND)
-    pkg_get_variable(SD_UNITDIR systemd systemduserunitdir)
-else()
-    set(SD_UNITDIR "lib/systemd/user")
-endif()
 
 execute_process(COMMAND git describe --tags --long --always RESULT_VARIABLE RC_GIT_VER OUTPUT_VARIABLE GIT_VERSION)
 

Daemon/CMakeLists.txt

@@ -1,5 +1,15 @@
-configure_file(
-    "${CMAKE_CURRENT_SOURCE_DIR}/ydotoold.service.in"
-    "${PROJECT_BINARY_DIR}/ydotoold.service"
-    @ONLY)
-install(FILES "${PROJECT_BINARY_DIR}/ydotoold.service" DESTINATION ${SD_UNITDIR})
+option(SYSTEMD_USER_SERVICE "Install systemd user service file" ON)
+option(SYSTEMD_SYSTEM_SERVICE "Install systemd system service file" OFF)
+option(OPENRC "Install OpenRC service file" OFF) # To be implemented
+
+if(SYSTEMD_USER_SERVICE AND SYSTEMD_SYSTEM_SERVICE)
+    message(FATAL_ERROR "Both system and user services are enabled. Choose one.")
+endif()
+
+if(SYSTEMD_USER_SERVICE OR SYSTEMD_SYSTEM_SERVICE) 
+    add_subdirectory(systemd)
+endif()
+
+if(OPENRC)
+    add_subdirectory(openrc)
+endif()

Daemon/openrc/CMakeLists.txt

@@ -0,0 +1 @@
+# TBD

Daemon/systemd/CMakeLists.txt

@@ -0,0 +1,25 @@
+find_package(PkgConfig)
+
+# Try to get the systemd directory for system/user units from the package
+# and fall back on upstream default without the '/usr' prefix if not forced by
+# command line.
+pkg_check_modules(SYSTEMD systemd)
+if (SYSTEMD_FOUND)
+    if(SYSTEMD_USER_SERVICE)
+        pkg_get_variable(SD_UNITDIR systemd systemduserunitdir)
+    elseif(SYSTEMD_SYSTEM_SERVICE)
+        pkg_get_variable(SD_UNITDIR systemd systemdsystemunitdir)
+    endif()
+endif()
+
+if(NOT DEFINED SD_UNITDIR)
+    set(SD_UNITDIR "lib/systemd/user")
+endif()
+
+message("Systemd service file is going to be installed at ${SD_UNITDIR}")
+
+configure_file(
+    "${CMAKE_CURRENT_SOURCE_DIR}/ydotoold.service.in"
+    "${PROJECT_BINARY_DIR}/ydotoold.service"
+    @ONLY)
+install(FILES "${PROJECT_BINARY_DIR}/ydotoold.service" DESTINATION ${SD_UNITDIR})

Daemon/ydotoold.service.in → Daemon/systemd/ydotoold.service.in

@@ -1,5 +1,5 @@
 [Unit]
-Description=Starts ydotoold user service
+Description=Starts ydotoold Daemon
 
 [Service]
 Type=simple
@@ -13,4 +13,3 @@ TimeoutSec=180
 
 [Install]
 WantedBy=basic.target
-