From 17f4dfd34244faacb8d73d722e29e8199a94265a Mon Sep 17 00:00:00 2001
From: patrick96 <p.ziegler96@gmail.com>
Date: Thu, 24 Dec 2020 01:40:26 +0100
Subject: [PATCH] cmake: Use lists

---
 CMakeLists.txt        |  3 +--
 config/CMakeLists.txt | 21 ++++++++++++---------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 3823d5ea..fdaf177c 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -22,8 +22,7 @@ else()
   set(APP_VERSION "${version_txt}")
 endif()
 
-set(CMAKE_MODULE_PATH
-  ${CMAKE_MODULE_PATH}
+list(APPEND CMAKE_MODULE_PATH
   ${PROJECT_SOURCE_DIR}/cmake
   ${PROJECT_SOURCE_DIR}/cmake/common
   ${PROJECT_SOURCE_DIR}/cmake/modules)
diff --git a/config/CMakeLists.txt b/config/CMakeLists.txt
index b75bbebb..7b14f98e 100644
--- a/config/CMakeLists.txt
+++ b/config/CMakeLists.txt
@@ -2,9 +2,9 @@
 # Generate configuration file
 #
 
-set(MODULES_LEFT "bspwm i3")
+set(MODULES_LEFT "bspwm;i3")
 set(MODULES_CENTER "mpd")
-set(MODULES_RIGHT "filesystem backlight-acpi alsa pulseaudio xkeyboard memory cpu wlan eth battery temperature date powermenu")
+set(MODULES_RIGHT "filesystem;backlight-acpi;alsa;pulseaudio;xkeyboard;memory;cpu;wlan;eth;battery;temperature;date;powermenu")
 
 set(FONT_FIXED "fixed:pixelsize=10")
 set(FONT_UNIFONT "unifont:fontformat=truetype")
@@ -17,24 +17,28 @@ queryfont(FONT_SIJI ${FONT_SIJI} FIELDS family pixelsize)
 # Strip disabled modules {{{
 
 if(NOT ENABLE_PULSEAUDIO)
-  string(REPLACE " pulseaudio" "" MODULES_RIGHT ${MODULES_RIGHT})
+  list(REMOVE_ITEM MODULES_RIGHT "pulseaudio")
 endif()
 if(NOT ENABLE_ALSA)
-  string(REPLACE " alsa" "" MODULES_RIGHT ${MODULES_RIGHT})
+  list(REMOVE_ITEM MODULES_RIGHT "alsa")
 endif()
 if(NOT ENABLE_I3)
-  string(REPLACE " i3" "" MODULES_LEFT ${MODULES_LEFT})
+  list(REMOVE_ITEM MODULES_LEFT "i3")
 endif()
 if(NOT ENABLE_MPD)
-  string(REPLACE "mpd" "" MODULES_CENTER ${MODULES_CENTER})
+  list(REMOVE_ITEM MODULES_CENTER "mpd")
 endif()
 if(NOT ENABLE_NETWORK)
-  string(REPLACE " wlan eth" "" MODULES_RIGHT ${MODULES_RIGHT})
+  list(REMOVE_ITEM MODULES_RIGHT "wlan" "eth")
 endif()
 if(NOT WITH_XKB)
-  string(REPLACE "xkeyboard " "" MODULES_RIGHT ${MODULES_RIGHT})
+  list(REMOVE_ITEM MODULES_RIGHT "xkeyboard")
 endif()
 
+string(REPLACE ";" " " MODULES_LEFT "${MODULES_LEFT}")
+string(REPLACE ";" " " MODULES_CENTER "${MODULES_CENTER}")
+string(REPLACE ";" " " MODULES_RIGHT "${MODULES_RIGHT}")
+
 # }}}
 # Get battery/adapter name {{{
 
@@ -89,7 +93,6 @@ install(FILES ${CMAKE_BINARY_DIR}/generated-sources/config
   COMPONENT config)
 
 # }}}
-
 # Target: userconfig {{{
 
 configure_file(