Your message dated Sun, 14 Apr 2013 12:17:22 +0100
with message-id <[email protected]>
and subject line Re: Bug#683801: Lintian: E: 
init.d-script-missing-dependency-on-remote_fs [patch]
has caused the Debian Bug report #683801,
regarding sysvinit: Lintian: E: init.d-script-missing-dependency-on-remote_fs 
[patch]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
683801: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683801
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: sysvinit
Version: 2.88dsf-29
Severity: normal

This patch fixes Lintian: E: init.d-script-missing-dependency-on-remote_fs
for src:sysvinit. One would however ponder why such a critical package as
'init' would dare depend upon binaries from /usr in the first place.

-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 3.2.0-3-686-pae (SMP w/1 CPU core)
Locale: LANG=fi_FI.UTF-8, LC_CTYPE=fi_FI.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages sysvinit depends on:
ii  debianutils     4.3.3
ii  initscripts     2.88dsf-29
ii  libc6           2.13-35
ii  libselinux1     2.1.9-5
ii  libsepol1       2.1.4-3
ii  sysv-rc         2.88dsf-29
ii  sysvinit-utils  2.88dsf-29

sysvinit recommends no packages.

sysvinit suggests no packages.

-- no debconf information
>From dc41abb9a4b123c2b860e9f1141f71708113324a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin-=C3=89ric=20Racine?= <[email protected]>
Date: Sat, 4 Aug 2012 09:04:25 +0300
Subject: sysvinit: Lintian: E: init.d-script-missing-dependency-on-remote_fs

---
 debian/src/bootlogd/etc/init.d/bootlogd        |    2 +-
 debian/src/initscripts/etc/init.d/bootlogs     |    2 +-
 debian/src/initscripts/etc/init.d/checkfs.sh   |    2 +-
 debian/src/initscripts/etc/init.d/checkroot.sh |    4 ++--
 debian/src/initscripts/etc/init.d/halt         |    2 +-
 debian/src/initscripts/etc/init.d/killprocs    |    4 ++--
 debian/src/initscripts/etc/init.d/motd         |    2 +-
 debian/src/initscripts/etc/init.d/mountall.sh  |    4 ++--
 debian/src/initscripts/etc/init.d/mountnfs.sh  |    2 +-
 debian/src/initscripts/etc/init.d/reboot       |    2 +-
 debian/src/initscripts/etc/init.d/umountfs     |    2 +-
 debian/src/initscripts/etc/init.d/umountnfs.sh |    2 +-
 12 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/debian/src/bootlogd/etc/init.d/bootlogd b/debian/src/bootlogd/etc/init.d/bootlogd
index f6de3a6..4f380de 100644
--- a/debian/src/bootlogd/etc/init.d/bootlogd
+++ b/debian/src/bootlogd/etc/init.d/bootlogd
@@ -1,7 +1,7 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          bootlogd
-# Required-Start:    mountdevsubfs
+# Required-Start:    mountdevsubfs $remote_fs
 # X-Start-Before:    hostname keymap keyboard-setup procps pcmcia hwclock hwclockfirst hdparm hibernate-cleanup lvm2
 # Required-Stop:
 # Default-Start:     S
diff --git a/debian/src/initscripts/etc/init.d/bootlogs b/debian/src/initscripts/etc/init.d/bootlogs
index 3ad7d8b..2a44719 100644
--- a/debian/src/initscripts/etc/init.d/bootlogs
+++ b/debian/src/initscripts/etc/init.d/bootlogs
@@ -1,7 +1,7 @@
 #!/bin/sh
 ### BEGIN INIT INFO
 # Provides:          bootlogs
-# Required-Start:    hostname $local_fs
+# Required-Start:    hostname $local_fs $remote_fs
 # Required-Stop:
 # Should-Start:      $x-display-manager gdm kdm xdm ldm sdm wdm nodm
 # Default-Start:     1 2 3 4 5
diff --git a/debian/src/initscripts/etc/init.d/checkfs.sh b/debian/src/initscripts/etc/init.d/checkfs.sh
index 7c619b3..1ae4687 100755
--- a/debian/src/initscripts/etc/init.d/checkfs.sh
+++ b/debian/src/initscripts/etc/init.d/checkfs.sh
@@ -1,7 +1,7 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          checkfs
-# Required-Start:    checkroot
+# Required-Start:    checkroot $remote_fs
 # Required-Stop:
 # Should-Start:      mtab
 # Default-Start:     S
diff --git a/debian/src/initscripts/etc/init.d/checkroot.sh b/debian/src/initscripts/etc/init.d/checkroot.sh
index fc261f3..6789226 100755
--- a/debian/src/initscripts/etc/init.d/checkroot.sh
+++ b/debian/src/initscripts/etc/init.d/checkroot.sh
@@ -1,8 +1,8 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          checkroot
-# Required-Start:    mountdevsubfs hostname
-# Required-Stop:     
+# Required-Start:    mountdevsubfs hostname $remote_fs
+# Required-Stop:
 # Should-Start:      keymap hwclockfirst hdparm bootlogd
 # Should-stop:
 # Default-Start:     S
diff --git a/debian/src/initscripts/etc/init.d/halt b/debian/src/initscripts/etc/init.d/halt
index 3b359fe..aa717ff 100755
--- a/debian/src/initscripts/etc/init.d/halt
+++ b/debian/src/initscripts/etc/init.d/halt
@@ -2,7 +2,7 @@
 ### BEGIN INIT INFO
 # Provides:          halt
 # Required-Start:
-# Required-Stop:
+# Required-Stop:     $remote_fs
 # Default-Start:
 # Default-Stop:      0
 # Short-Description: Execute the halt command.
diff --git a/debian/src/initscripts/etc/init.d/killprocs b/debian/src/initscripts/etc/init.d/killprocs
index 6cb5b91..0b2f8c2 100755
--- a/debian/src/initscripts/etc/init.d/killprocs
+++ b/debian/src/initscripts/etc/init.d/killprocs
@@ -1,7 +1,7 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          killprocs
-# Required-Start:    $local_fs
+# Required-Start:    $local_fs $remote_fs
 # Required-Stop:
 # Default-Start:     1
 # Default-Stop:
@@ -24,7 +24,7 @@ do_start () {
 		# value, because either killall5 work and it make
 		# sense to wait for processes to die, or it fail and
 		# there is nothing to wait for.
-		
+
 		if killall5 -18 ; then
 		    :
 		else
diff --git a/debian/src/initscripts/etc/init.d/motd b/debian/src/initscripts/etc/init.d/motd
index e0d7706..cfc2c7f 100644
--- a/debian/src/initscripts/etc/init.d/motd
+++ b/debian/src/initscripts/etc/init.d/motd
@@ -1,7 +1,7 @@
 #!/bin/sh
 ### BEGIN INIT INFO
 # Provides:          motd
-# Required-Start:    hostname $local_fs
+# Required-Start:    hostname $local_fs $remote_fs
 # Required-Stop:
 # Should-Start:
 # Default-Start:     1 2 3 4 5
diff --git a/debian/src/initscripts/etc/init.d/mountall.sh b/debian/src/initscripts/etc/init.d/mountall.sh
index 3ad1ea7..8caffba 100755
--- a/debian/src/initscripts/etc/init.d/mountall.sh
+++ b/debian/src/initscripts/etc/init.d/mountall.sh
@@ -1,8 +1,8 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          mountall
-# Required-Start:    checkfs
-# Required-Stop: 
+# Required-Start:    checkfs $remote_fs
+# Required-Stop:
 # Default-Start:     S
 # Default-Stop:
 # Short-Description: Mount all filesystems.
diff --git a/debian/src/initscripts/etc/init.d/mountnfs.sh b/debian/src/initscripts/etc/init.d/mountnfs.sh
index ce193c0..9a86bff 100755
--- a/debian/src/initscripts/etc/init.d/mountnfs.sh
+++ b/debian/src/initscripts/etc/init.d/mountnfs.sh
@@ -1,7 +1,7 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:          mountnfs
-# Required-Start:    $local_fs
+# Required-Start:    $local_fs $remote_fs
 # Required-Stop:
 # Should-Start:      $network $portmap nfs-common  udev-mtab
 # Default-Start:     S
diff --git a/debian/src/initscripts/etc/init.d/reboot b/debian/src/initscripts/etc/init.d/reboot
index af59a80..8190756 100755
--- a/debian/src/initscripts/etc/init.d/reboot
+++ b/debian/src/initscripts/etc/init.d/reboot
@@ -2,7 +2,7 @@
 ### BEGIN INIT INFO
 # Provides:          reboot
 # Required-Start:
-# Required-Stop:
+# Required-Stop:     $remote_fs
 # Default-Start:
 # Default-Stop:      6
 # Short-Description: Execute the reboot command.
diff --git a/debian/src/initscripts/etc/init.d/umountfs b/debian/src/initscripts/etc/init.d/umountfs
index ae27d0e..cdf7a8d 100755
--- a/debian/src/initscripts/etc/init.d/umountfs
+++ b/debian/src/initscripts/etc/init.d/umountfs
@@ -2,7 +2,7 @@
 ### BEGIN INIT INFO
 # Provides:          umountfs
 # Required-Start:
-# Required-Stop:     umountroot
+# Required-Stop:     umountroot $remote_fs
 # Default-Start:
 # Default-Stop:      0 6
 # Short-Description: Turn off swap and unmount all local file systems.
diff --git a/debian/src/initscripts/etc/init.d/umountnfs.sh b/debian/src/initscripts/etc/init.d/umountnfs.sh
index 807be19..e1b4017 100755
--- a/debian/src/initscripts/etc/init.d/umountnfs.sh
+++ b/debian/src/initscripts/etc/init.d/umountnfs.sh
@@ -2,7 +2,7 @@
 ### BEGIN INIT INFO
 # Provides:          umountnfs
 # Required-Start:
-# Required-Stop:     umountfs
+# Required-Stop:     umountfs $remote_fs
 # Should-Stop:       $network $portmap nfs-common
 # Default-Start:
 # Default-Stop:      0 6
-- 
1.7.10.4


--- End Message ---
--- Begin Message ---
On Sat, Aug 04, 2012 at 09:17:55AM +0300, Martin-Éric Racine wrote:
> This patch fixes Lintian: E: init.d-script-missing-dependency-on-remote_fs
> for src:sysvinit. One would however ponder why such a critical package as
> 'init' would dare depend upon binaries from /usr in the first place.

I'm closing this bug, since the premise is invalid.  The lack of
a dependency on remote_fs is completely intentional, because these
scripts must be started before remote_fs (and some /provide/
remote_fs), so applying this patch would break booting due to
introducing circular dependencies.

We could certainly add some lintian overrides for the warnings,
if appropriate, but that's a separate issue.


Regards,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux    http://people.debian.org/~rleigh/
 `. `'   schroot and sbuild  http://alioth.debian.org/projects/buildd-tools
   `-    GPG Public Key      F33D 281D 470A B443 6756 147C 07B3 C8BC 4083 E800

--- End Message ---
_______________________________________________
Pkg-sysvinit-devel mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-sysvinit-devel

Reply via email to