reassign 768329 plymouth 0.9.0-8
retitle 768329 plymouth: Please enable splash for jessie
tag 768329 + patch
thanks

Hello,

I'm reassigning this bug to plymouth.

I already looked at this and I have a proposed fix (see attached patch).

With this patch, we have now 3 situations when the package is installed:

 - The package is installed and no extra boot parameter is passed to
   the kernel: plymouth will display the details which displays the
   usual boot messages.

 - The package is installed and "splash" is passed to the kernel: A
   fancy boot screen will be displayed to the user.

 - The package is installed and "nosplash" (or "plymouth.enable=0") is
   passed to the kernel: Everything is disabled like if the package was
   not installed.

What do you think about this solution?

Cheers,

Laurent Bigonville
commit d54e54ceb8278bb0fb70328f027dedfe89374d0a
Author: Laurent Bigonville <bi...@bigon.be>
Date:   Mon Nov 10 11:41:07 2014 +0100

    Enable I/O multiplexing functionality of plymouth by default as soon as the main package is installed. (Closes: #768329)

diff --git a/debian/changelog b/debian/changelog
index 23a13bd..aad5e51 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -10,8 +10,10 @@ plymouth (0.9.0-9) UNRELEASED; urgency=medium
   * debian/local/plymouth.hook: Properly copy .plymouth file into the
     initramfs for themes that are not shipping images, this should fix the
     tribar theme.
+  * Enable I/O multiplexing functionality of plymouth by default as soon as
+    the main package is installed. (Closes: #768329)
 
- -- Laurent Bigonville <bi...@debian.org>  Mon, 10 Nov 2014 12:48:27 +0100
+ -- Laurent Bigonville <bi...@debian.org>  Mon, 10 Nov 2014 11:38:11 +0100
 
 plymouth (0.9.0-8) unstable; urgency=medium
 
diff --git a/debian/local/plymouth.init-premount b/debian/local/plymouth.init-premount
index 6e1dfd3..e5a7db6 100644
--- a/debian/local/plymouth.init-premount
+++ b/debian/local/plymouth.init-premount
@@ -14,7 +14,7 @@ case ${1} in
 		;;
 esac
 
-SPLASH="false"
+SPLASH="true"
 
 for ARGUMENT in $(cat /proc/cmdline)
 do
@@ -23,7 +23,7 @@ do
 			SPLASH="true"
 			;;
 
-		nosplash*)
+		nosplash*|plymouth.enable=0)
 			SPLASH="false"
 			;;
 	esac
diff --git a/debian/patches/0005-cmdline.patch b/debian/patches/0005-cmdline.patch
index 4f19726..398ee3c 100644
--- a/debian/patches/0005-cmdline.patch
+++ b/debian/patches/0005-cmdline.patch
@@ -1,89 +1,74 @@
-Author: Daniel Baumann <m...@daniel-baumann.ch>
+Author: Laurent Bigonville <bi...@debian.org
 Description:
- Using 'splash' as boot parameter instead of 'plymouth.enable' in system unit
- files (Closes: #680517).
+ Handle 'nosplash' boot parameter as 'plymouth.enable=0' in system unit
 
-diff -Naurp plymouth.orig/systemd-units/plymouth-halt.service.in plymouth/systemd-units/plymouth-halt.service.in
---- plymouth.orig/systemd-units/plymouth-halt.service.in
-+++ plymouth/systemd-units/plymouth-halt.service.in
-@@ -3,7 +3,7 @@ Description=Show Plymouth Halt Screen
- After=getty@tty1.service display-manager.service plymouth-start.service
+--- a/systemd-units/plymouth-halt.service.in
++++ b/systemd-units/plymouth-halt.service.in
+@@ -4,6 +4,7 @@ After=getty@tty1.service display-manager
  Before=systemd-halt.service
  DefaultDependencies=no
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  
  [Service]
  ExecStart=@PLYMOUTH_DAEMON_DIR@/plymouthd --mode=shutdown --attach-to-session
-diff -Naurp plymouth.orig/systemd-units/plymouth-kexec.service.in plymouth/systemd-units/plymouth-kexec.service.in
---- plymouth.orig/systemd-units/plymouth-kexec.service.in
-+++ plymouth/systemd-units/plymouth-kexec.service.in
-@@ -3,7 +3,7 @@ Description=Show Plymouth Reboot with ke
- After=getty@tty1.service display-manager.service plymouth-start.service
+--- a/systemd-units/plymouth-kexec.service.in
++++ b/systemd-units/plymouth-kexec.service.in
+@@ -4,6 +4,7 @@ After=getty@tty1.service display-manager
  Before=systemd-kexec.service
  DefaultDependencies=no
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  
  [Service]
  ExecStart=@PLYMOUTH_DAEMON_DIR@/plymouthd --mode=shutdown --attach-to-session
-diff -Naurp plymouth.orig/systemd-units/plymouth-poweroff.service.in plymouth/systemd-units/plymouth-poweroff.service.in
---- plymouth.orig/systemd-units/plymouth-poweroff.service.in
-+++ plymouth/systemd-units/plymouth-poweroff.service.in
-@@ -3,7 +3,7 @@ Description=Show Plymouth Power Off Scre
- After=getty@tty1.service display-manager.service plymouth-start.service
+--- a/systemd-units/plymouth-poweroff.service.in
++++ b/systemd-units/plymouth-poweroff.service.in
+@@ -4,6 +4,7 @@ After=getty@tty1.service display-manager
  Before=systemd-poweroff.service
  DefaultDependencies=no
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  
  [Service]
  ExecStart=@PLYMOUTH_DAEMON_DIR@/plymouthd --mode=shutdown --attach-to-session
-diff -Naurp plymouth.orig/systemd-units/plymouth-reboot.service.in plymouth/systemd-units/plymouth-reboot.service.in
---- plymouth.orig/systemd-units/plymouth-reboot.service.in
-+++ plymouth/systemd-units/plymouth-reboot.service.in
-@@ -3,7 +3,7 @@ Description=Show Plymouth Reboot Screen
- After=getty@tty1.service display-manager.service plymouth-start.service
+--- a/systemd-units/plymouth-reboot.service.in
++++ b/systemd-units/plymouth-reboot.service.in
+@@ -4,6 +4,7 @@ After=getty@tty1.service display-manager
  Before=systemd-reboot.service
  DefaultDependencies=no
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  
  [Service]
  ExecStart=@PLYMOUTH_DAEMON_DIR@/plymouthd --mode=shutdown --attach-to-session
-diff -Naurp plymouth.orig/systemd-units/plymouth-start.service.in plymouth/systemd-units/plymouth-start.service.in
---- plymouth.orig/systemd-units/plymouth-start.service.in
-+++ plymouth/systemd-units/plymouth-start.service.in
-@@ -4,7 +4,7 @@ DefaultDependencies=no
- Wants=systemd-ask-password-plymouth.path systemd-vconsole-setup.service
+--- a/systemd-units/plymouth-start.service.in
++++ b/systemd-units/plymouth-start.service.in
+@@ -5,6 +5,7 @@ Wants=systemd-ask-password-plymouth.path
  After=systemd-vconsole-setup.service systemd-udev-trigger.service systemd-udevd.service
  Before=systemd-ask-password-plymouth.service
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  
  [Service]
  ExecStart=@PLYMOUTH_DAEMON_DIR@/plymouthd --mode=boot --pid-file=@plymouthruntimedir@/pid --attach-to-session
-diff -Naurp plymouth.orig/systemd-units/systemd-ask-password-plymouth.path.in plymouth/systemd-units/systemd-ask-password-plymouth.path.in
---- plymouth.orig/systemd-units/systemd-ask-password-plymouth.path.in	2012-09-26 18:50:53.000000000 +0200
-+++ plymouth/systemd-units/systemd-ask-password-plymouth.path.in
-@@ -5,7 +5,7 @@ DefaultDependencies=no
- Conflicts=shutdown.target
+--- a/systemd-units/systemd-ask-password-plymouth.path.in
++++ b/systemd-units/systemd-ask-password-plymouth.path.in
+@@ -6,6 +6,7 @@ Conflicts=shutdown.target
  After=plymouth-start.service
  Before=basic.target shutdown.target
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  ConditionPathExists=/run/plymouth/pid
  
  [Path]
-diff -Naurp plymouth.orig/systemd-units/systemd-ask-password-plymouth.service.in plymouth/systemd-units/systemd-ask-password-plymouth.service.in
---- plymouth.orig/systemd-units/systemd-ask-password-plymouth.service.in
-+++ plymouth/systemd-units/systemd-ask-password-plymouth.service.in
-@@ -5,7 +5,7 @@ DefaultDependencies=no
- Conflicts=shutdown.target
+--- a/systemd-units/systemd-ask-password-plymouth.service.in
++++ b/systemd-units/systemd-ask-password-plymouth.service.in
+@@ -6,6 +6,7 @@ Conflicts=shutdown.target
  After=plymouth-start.service
  Before=shutdown.target
--ConditionKernelCommandLine=!plymouth.enable=0
-+ConditionKernelCommandLine=splash
+ ConditionKernelCommandLine=!plymouth.enable=0
++ConditionKernelCommandLine=!nosplash
  ConditionPathExists=/run/plymouth/pid
  
  [Service]
diff --git a/debian/patches/drop-systemd-vconsole-setup-service.patch b/debian/patches/drop-systemd-vconsole-setup-service.patch
index 9c6c3a5..33739e2 100644
--- a/debian/patches/drop-systemd-vconsole-setup-service.patch
+++ b/debian/patches/drop-systemd-vconsole-setup-service.patch
@@ -15,5 +15,5 @@ Forwarded: not-needed
 +Wants=systemd-ask-password-plymouth.path
 +After=systemd-udev-trigger.service systemd-udevd.service keyboard-setup.service
  Before=systemd-ask-password-plymouth.service
- ConditionKernelCommandLine=splash
- 
+ ConditionKernelCommandLine=!plymouth.enable=0
+ ConditionKernelCommandLine=!nosplash
diff --git a/debian/plymouth.NEWS b/debian/plymouth.NEWS
new file mode 100644
index 0000000..bf1d515
--- /dev/null
+++ b/debian/plymouth.NEWS
@@ -0,0 +1,13 @@
+plymouth (0.9.0-9) UNRELEASED; urgency=medium
+
+  Starting with this version, installing the plymouth package will
+  automatically enable plymouth I/O multiplexing functionality. This is needed
+  if you are using an event-based init system. The bootup messages will still be
+  displayed as usual.
+
+  If you don't want this functionality, you could either uninstall the
+  plymouth package or pass "plymouth.enable=0" or "nosplash" to the kernel
+  cmdline. On the other hand if you want a fancy boot screen, you explicitly
+  need to pass "splash" to the kernel cmdline.
+
+ -- Laurent Bigonville <bi...@debian.org>  Mon, 10 Nov 2014 11:23:37 +0100
diff --git a/debian/plymouth.init b/debian/plymouth.init
index a2eb17d..2658bcb 100644
--- a/debian/plymouth.init
+++ b/debian/plymouth.init
@@ -26,6 +26,7 @@ fi
 
 set -e
 
+SPLASH="true"
 for ARGUMENT in $(cat /proc/cmdline)
 do
 	case "${ARGUMENT}" in
@@ -33,7 +34,7 @@ do
 			SPLASH="true"
 			;;
 
-		nosplash*)
+		nosplash*|plymouth.enable=0)
 			SPLASH="false"
 			;;
 	esac

Reply via email to