Control: tag -1 patch

[email protected] [2015-11-13  7:53 +0100]:
> your package setserial declares a dependency on the initscripts
> package.
> 
> Please check, if this dependency is actually needed and remove it
> otherwise.

This was introduced in https://bugs.debian.org/734496 because
/etc/init.d/etc-setserial has "Required-Start: checkfs". IMHO this
should just say "$local_fs" instead, and drop the initscripts
dependency again. Corresponding patch attached.

Note that /etc/init.d/setserial has a corresponding systemd unit, but
/etc/init.d/etc-setserial does not. Could this be systemd-ified too?
Or is it not necessary under systemd, and etc-setserial.service should
simply be masked?

Thanks,

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
diff -Nru setserial-2.17/debian/changelog setserial-2.17/debian/changelog
--- setserial-2.17/debian/changelog     2015-09-22 23:32:05.000000000 +0200
+++ setserial-2.17/debian/changelog     2016-06-08 11:33:30.000000000 +0200
@@ -1,3 +1,10 @@
+setserial (2.17-49ubuntu1) yakkety; urgency=medium
+
+  * 10_rc_etc_setserial.patch: Change etc-setserial's "checkfs" dependency to
+    $local_fs, and drop initscripts dependency. (Closes: #734496)
+
+ -- Martin Pitt <[email protected]>  Wed, 08 Jun 2016 11:32:31 +0200
+
 setserial (2.17-49) unstable; urgency=medium
 
   * as upstream archive vanished, watch file is useless
diff -Nru setserial-2.17/debian/control setserial-2.17/debian/control
--- setserial-2.17/debian/control       2015-09-22 23:20:55.000000000 +0200
+++ setserial-2.17/debian/control       2016-06-08 11:32:28.000000000 +0200
@@ -7,7 +7,7 @@
 
 Package: setserial
 Architecture: any
-Depends: ${shlibs:Depends}, ${misc:Depends}, initscripts
+Depends: ${shlibs:Depends}, ${misc:Depends}
 Description: controls configuration of serial ports
  Set and/or report the configuration information associated with
  a serial port. This information includes what I/O port and which IRQ
diff -Nru setserial-2.17/debian/patches/10_rc_etc_setserial.patch 
setserial-2.17/debian/patches/10_rc_etc_setserial.patch
--- setserial-2.17/debian/patches/10_rc_etc_setserial.patch     2015-09-22 
23:30:40.000000000 +0200
+++ setserial-2.17/debian/patches/10_rc_etc_setserial.patch     2016-06-08 
11:32:14.000000000 +0200
@@ -11,7 +11,7 @@
 +#
 +### BEGIN INIT INFO
 +# Provides:             etc-setserial
-+# Required-Start:       checkfs
++# Required-Start:       $local_fs
 +# Required-Stop:        $remote_fs
 +# Default-Start:        S
 +# Default-Stop:         0 1 6

Reply via email to