[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#361137: marked as done ([AMENDMENT 06/04/2006] Make use of invoke-rc.d, if available, mandatory)

Your message dated Tue, 25 Apr 2006 22:47:06 -0700
with message-id <E1FYcri-0000Sa-Kl@spohr.debian.org>
and subject line Bug#361137: fixed in debian-policy
has caused the attached Bug report 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 I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: debian-policy
Severity: wishlist

Given that the initial reactions to this were uniformly positive, I'll
need to actually start following procedure and use the BTS.

The formal proposal is to change ("Running initscripts"), second
paragraph, from this:

        The use of invoke-rc.d to invoke the /etc/init.d/* initscripts
        is strongly recommended[51], instead of calling them directly.

to this:

        The use of invoke-rc.d to invoke the /etc/init.d/* initscripts
        is mandatory if it is installed.

Because invoke-rc.d is in a package (sysv-rc or file-rc) that an
essential package depends on, it is guaranteed that invoke-rc.d is
always on the system. In principle, we could then strike the four last
words of the proposed paragraph, which would simplify things for
packages. I am unsure of whether we should make invoke-rc.d's
essentialness more explicit in that case.

ma, 2006-04-03 kello 00:38 +0300, Lars Wirzenius kirjoitti:
> Current policy states in section ("Running initscripts") the
> following: "The use of invoke-rc.d to invoke the /etc/init.d/*
> initscripts is strongly recommended[51], instead of calling them
> directly." 
> Footnote 51 further says: "In the future, the use of invoke-rc.d to
> invoke initscripts shall be made mandatory. Maintainers are advised to
> switch to invoke-rc.d as soon as possible."
> I propose that the future has arrived.
> I ran the attached script on all binary packages in sid/main/i386, and
> it reported 134 packages that I then analyzed manually. There were only
> two false positives (both mentioned how to run their init.d script in a
> message to the user, but didn't run it themselves). All other reported
> packages were true positives. I have attached the list as well.
> Almost all of the packages in the list are either dict-* packages
> (seemingly from the same template), have code from an old debhelper
> version (so a rebuild might suffice), or are otherwise pretty easy to
> fix. Even though the number of packages are fairly large, it is my
> opinion that it should be possible to keep the transition period quite
> short, weeks at most, and I'm willing to put some time into it.
> See also bug #353659 against lintian to add a check for this.
> I would like to see this policy change happen in time for all packages
> to be updated in etch. This would mean that sysadmins can, finally, rely
> on policy-rc.d working reliably. Also it means that it would be easier
> to build chroots, and not have to worry about services and daemons being
> started inside them unnecessarily.
> I realize that my script doesn't find all problematic packages. It is
> meant as a quick estimate. For proper testing, I have recently
> implemented changes into piuparts that should make it possible to find
> all problematic packages: in my development version /proc now gets
> mounted (ergo, start-stop-daemon works), and after packages have been
> installed, lsof checks that no processes run inside the chroot. This
> will, I hope, catch packages that don't use invoke-rc.d when they
> should.
> Are there any objections to this proposed change? (If there are none
> within a few days, I'll file the appropriate bug against debian-policy.)

Just GNU it!

--- End Message ---
--- Begin Message ---
Source: debian-policy

We believe that the bug you reported is fixed in the latest version of
debian-policy, which is due to be installed in the Debian FTP archive:

  to pool/main/d/debian-policy/debian-policy_3.7.0.0.dsc
  to pool/main/d/debian-policy/debian-policy_3.7.0.0.tar.gz
  to pool/main/d/debian-policy/debian-policy_3.7.0.0_all.deb

A summary of the changes between this version and the previous one is

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 361137@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
Manoj Srivastava <srivasta@debian.org> (supplier of updated debian-policy package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)

Hash: SHA1

Format: 1.7
Date: Tue, 25 Apr 2006 23:56:16 -0500
Source: debian-policy
Binary: debian-policy
Architecture: source all
Distribution: unstable
Urgency: low
Maintainer: Debian Policy List <debian-policy@lists.debian.org>
Changed-By: Manoj Srivastava <srivasta@debian.org>
 debian-policy - Debian Policy Manual and related documents
Closes: 32263 117916 148194 190753 199849 212434 230217 341232 342611 344158 346598 349010 349775 355263 357613 359817 361137 362247 362975
 debian-policy ( unstable; urgency=low
   * Bug fix: "[PENDING AMENDMENT 20/01/2000] Splitting cgi-bin", thanks to
     Brian White.                                             (Closes: #32263).
   * Bug fix: "debian-policy: [PROPOSAL] Should update to Filesystem
     Hierarchy Standard FHS 2.3", thanks to Tobias Burnus
                                           (Closes: #230217, #212434, #344158).
   * Bug fix: "[AMENDMENT 11/04/2006] Permit multi-line fields in
     debian/control", thanks to John R. Daily. Mention that all fields,
     except the Uploaders, are supposed to be a single logical line, which
     may be spread over multiple physical lines (newline followed by space
     is elided). Also mention that anything parsing the control file must
     allow for a multi-line uploaders field.                (Closes: #148194).
   * Bug fix: "[AMENDMENT 12/04/2004] frown on programs in PATH with
     language extentions", thanks to Joey Hess.             (Closes: #190753).
   * Bug fix: "init script stop example should use --oknodo", thanks to
     Matt Kraai. Removed the example entirely.              (Closes: #346598).
   * Bug fix: "policy 12.5: Please recommend a sane practice WRT different
     gpl versions (was: Re: RFC/RFS: beef - a flexible BrainFuck
     interpreter)", thanks to Justin Pryzby. The subject leaves something
     to be desired, but polic should not attempt to enumerate all common
     licenses.                                               (Closes: #355263).
   * Bug fix: "debian-policy: Conflicting Architecture definitions", thanks
     to Hans Ulrich Niedermann. Punt to dpkg-architecture to providing
     legal architecture strings, since that's what is used by everyone
     anyway.  The version in policy was wrong, but that s=does not seem to
     have hindered anyone, which indicates that this policy directive was
     uneeded. Now the dpkg-architecture list is deemed authoritative, which
     it is, but the format for the string is defined by policy, and the
     current list of architecture strings is in an informative foot note.
                                                            (Closes: #357613).
   * Bug fix: "[AMENDMENT 06/04/2006] Make use of invoke-rc.d, if
     available, mandatory", thanks to Lars Wirzenius.        (Closes: #361137).
   * Bug fix: "no longer current regarding X font paths", thanks to Joey
     Hess                                                    (Closes: #362247).
   * Bug fix: "debian-policy: please prohibit circular dependencies, or
     mention that dependencies won't be respected during prerm remove",
     thanks to Justin Pryzby. Well, we did not prohibit circular
     dependencies. But we do now have a warning that In case of circular
     dependencies, since installation or removal order honoring the
     dependency order can't be established, dependency loops are broken at
     some random point, and some packages may not be able to rely on their
     dependencies being present when being installed or removed, depending
     on which side of the break of the circular dependcy loop they happen
     to be on.                                                (Closes: #362975).
   * Bug fix: "8.6.4. Providing a `shlibs' file: s/should create/must
     provide/", thanks to Christoph Berg. Clarified the wording.
                                                              (Closes: #341232).
   * Bug fix: "debian-policy: Chapter 6 - Package maintainer scripts:
     redundant info about exit status", thanks to Daniel Bonniot
                                                              (Closes: #349010).
   * Bug fix: "debian-policy: Refers to upgrading-checklist.txt instead of
     upgrading-checklist.txt.gz", thanks to Matt Kraai         (Closes: #349775).
   * Bug fix: "debian-policy: dpkg-gencontrol now uses -isp by default",
     thanks to Guillem Jover                                   (Closes: #359817).
   * Bug fix: "[PROPOSAL] unclear recommendation for debconf w/
     dpkg-statoverride", thanks to Eduard Bloch                (Closes: #199849).
   * debian-policy: please support Watch file as recommendation, thanks to
      Bluefuture                                               (Closes: #342611).
   * Bug fix: "[PROPOSED] Mandate http servers to provide httpd-cgi as
     relevenat", thanks to Uwe Hermann. This is already supported by the
     http servers out there.                                   (Closes: #117916).
 a99c53850f2296afd22844be148c71c9 820 doc optional debian-policy_3.7.0.0.dsc
 e43f135a8c5ad3f7090435137bf14bb1 771599 doc optional debian-policy_3.7.0.0.tar.gz
 eede58ceff0218bca767236ca6f214b2 1510902 doc optional debian-policy_3.7.0.0_all.deb

Version: GnuPG v1.4.3 (GNU/Linux)


--- End Message ---

Reply to: