On Tue, 02 May 2017 at 08:00:44 +0100, Simon McVittie wrote:
> On Tue, 02 May 2017 at 01:38:15 +0200, Axel Beckert wrote:
> > JFTR: This needs an "|| true" if your init script uses "set -e",
> > otherwise starting or restarting the DM besides a fresh system bootup
> > might fail.
> 
> Oh, good point. I should change that for nodm.

I've done a 0-day NMU of nodm with the attached diff to fix this
mistake in my earlier NMU; I hope the nodm maintainers don't mind.
Thanks to Axel Beckert for pointing this out.

    S
diffstat for nodm-0.13 nodm-0.13

 changelog |    8 ++++++++
 nodm.init |    2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff -Nru nodm-0.13/debian/changelog nodm-0.13/debian/changelog
--- nodm-0.13/debian/changelog	2017-04-27 18:37:16.000000000 +0100
+++ nodm-0.13/debian/changelog	2017-05-02 21:57:24.000000000 +0100
@@ -1,3 +1,11 @@
+nodm (0.13-1.2) unstable; urgency=medium
+
+  * Non-maintainer upload
+  * nodm.init: Do not fail to start if plymouth has already been
+    stopped
+
+ -- Simon McVittie <s...@debian.org>  Tue, 02 May 2017 21:57:24 +0100
+
 nodm (0.13-1.1) unstable; urgency=medium
 
   * d/p/nodm.service-Ask-Plymouth-to-stop-before-starting-no.patch:
diff -Nru nodm-0.13/debian/nodm.init nodm-0.13/debian/nodm.init
--- nodm-0.13/debian/nodm.init	2017-04-27 18:37:16.000000000 +0100
+++ nodm-0.13/debian/nodm.init	2017-05-02 21:57:24.000000000 +0100
@@ -52,7 +52,7 @@
 case "$1" in
 	start)
 		if [ -x /bin/plymouth ]; then
-			/bin/plymouth quit
+			/bin/plymouth quit || true
 		fi
 		[ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
 		if [ "$NODM_ENABLED" = "no" ] || [ "$NODM_ENABLED" = "false" ]

Reply via email to