From 2fd98c913ad1a7c10e0c6923bc92d81c1a0b3fbd Mon Sep 17 00:00:00 2001 From: Peter Jung Date: Fri, 29 Mar 2024 20:05:41 +0100 Subject: [PATCH] plymouthcfg: Use plymouth-set-default-theme instead of sed Currently we are running into an issue, when setting the default theme via "sed", since the "[DAEMON]" entry in /etc/plymouth/plymouthd.conf is commented, which results into that the theme is not correctly applied. Signed-off-by: Peter Jung --- CHANGES-3.3 | 1 + src/modules/plymouthcfg/main.py | 4 +--- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/CHANGES-3.3 b/CHANGES-3.3 index 5312c1e82..1aff89918 100644 --- a/CHANGES-3.3 +++ b/CHANGES-3.3 @@ -24,6 +24,7 @@ This release contains contributions from (alphabetically by first name): - *displaymanager* module can configure an alternate SDDM configuration file. - *networkcfg* a bug affecting NetPlan + NetworkManager was fixed. - *initcpiocfg* Add microcode hook to initcpiocfg + - *plymouthcfg* Use plymouth-set-default-theme to avoid issues with configuration # 3.3.4 (2024-02-27) diff --git a/src/modules/plymouthcfg/main.py b/src/modules/plymouthcfg/main.py index 5e66fce67..529d3426e 100644 --- a/src/modules/plymouthcfg/main.py +++ b/src/modules/plymouthcfg/main.py @@ -48,9 +48,7 @@ class PlymouthController: def setTheme(self): plymouth_theme = libcalamares.job.configuration["plymouth_theme"] - target_env_call(["sed", "-e", 's|^.*Theme=.*|Theme=' + - plymouth_theme + '|', "-i", - "/etc/plymouth/plymouthd.conf"]) + target_env_call(["plymouth-set-default-theme", plymouth_theme]) def run(self): if detect_plymouth():