fails to purge if module utilities are not installed

Bug #22708 reported by Debian Bug Importer
4
Affects Status Importance Assigned to Milestone
initrd-tools (Debian)
Fix Released
Unknown
initrd-tools (Ubuntu)
Invalid
High
Jeff Bailey

Bug Description

Automatically imported from Debian bug report #330446 http://bugs.debian.org/330446

Revision history for this message
In , dann frazier (dannf) wrote : pending

tags 330446 + pending
thanks

I've comitted the following patch to svn.

--- debian/initrd-tools.postrm (revision 4292)
+++ debian/initrd-tools.postrm (working copy)
@@ -4,7 +4,9 @@

 if [ "$1" = purge ]; then
        rm -rf /etc/mkinitrd
- update-modules
+ if [ -x /sbin/update-modules ]; then
+ /sbin/update-modules
+ fi
        update-rc.d initrd-tools.sh remove > /dev/null
 fi

--
dann frazier <email address hidden>

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Automatically imported from Debian bug report #330446 http://bugs.debian.org/330446

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Tue, 27 Sep 2005 20:34:45 -0700
From: Ryan Murray <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: fails to purge if module utilities are not installed

Package: initrd-tools
Version: 0.1.82
Severity: serious

sh-3.00# dpkg -P --pending
(Reading database ... 8535 files and directories currently installed.)
Removing initrd-tools ...
Purging configuration files for initrd-tools ...
/var/lib/dpkg/info/initrd-tools.postrm: line 7: update-modules: command not found
dpkg: error processing initrd-tools (--purge):
 subprocess post-removal script returned error exit status 127
Errors were encountered while processing:
 initrd-tools

You aren't guaranteed that anything is installed in your postrm script -- so
you need to not fail if there are no module tools to update modules with.

-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: powerpc
Shell: /bin/sh linked to /bin/bash

Revision history for this message
Jeff Bailey (jbailey) wrote :

Update-modules call wasn't added until after the version we have in Breezy.

Closing as NOTWARTY

Tks,
Jeff Bailey

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <1127881682.8546.26.camel@localhost>
Date: Tue, 27 Sep 2005 22:28:02 -0600
From: dann frazier <email address hidden>
To: <email address hidden>
Cc: <email address hidden>
Subject: pending

tags 330446 + pending
thanks

I've comitted the following patch to svn.

--- debian/initrd-tools.postrm (revision 4292)
+++ debian/initrd-tools.postrm (working copy)
@@ -4,7 +4,9 @@

 if [ "$1" = purge ]; then
        rm -rf /etc/mkinitrd
- update-modules
+ if [ -x /sbin/update-modules ]; then
+ /sbin/update-modules
+ fi
        update-rc.d initrd-tools.sh remove > /dev/null
 fi

--
dann frazier <email address hidden>

Revision history for this message
In , Sven Luther (luther) wrote : Fixed in NMU of initrd-tools 0.1.83

tag 324351 + fixed
tag 329614 + fixed
tag 330446 + fixed
tag 333857 + fixed

quit

This message was generated automatically in response to a
non-maintainer upload. The .changes file follows.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Fri, 14 Oct 2005 07:50:57 +0000
Source: initrd-tools
Binary: initrd-tools
Architecture: source all
Version: 0.1.83
Distribution: unstable
Urgency: low
Maintainer: Debian kernel team <email address hidden>
Changed-By: Sven Luther <email address hidden>
Description:
 initrd-tools - tools to create initrd image for prepackaged Linux kernel
Closes: 324351 329614 330446 333857
Changes:
 initrd-tools (0.1.83) unstable; urgency=low
 .
   [ Simon Horman ]
   * Clarify naming of scripts that will be run.
     Thanks to Simon Schoar. (Closes: #324351)
   * Make sure that path to output image is always fully qualified,
     else an image with a relative will not end up where requested
     (closes: #329614)
 .
   [ dann frazier ]
   * Don't assume update-modules is available during postrm (closes: #330446)
 .
   [ Sven Luther ]
   * Added --supported-(host|target)-version support for the new post-2.6.13
     ramdisk-tool policy. Added linux-ramdisk-tool virtual package too.
     (Closes: #333857)
Files:
 bb2f5ea3f208f972649dfb5d1cac123e 644 utils optional initrd-tools_0.1.83.dsc
 eff2ad61dc7e18f607d541bd71a53dac 28815 utils optional initrd-tools_0.1.83.tar.gz
 dc7b11f6eb3cc3cf934c6639f1a1db3d 31728 utils optional initrd-tools_0.1.83_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)

iD8DBQFDUs2o2WTeT3CRQaQRAhifAJ9jQVG+GwyAs7lGMD/dwllcAN/GmACfSEAA
igkOcTRK7n9s9dQdA/xC2ew=
=7/gM
-----END PGP SIGNATURE-----

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Sun, 16 Oct 2005 15:17:06 -0700
From: Sven Luther <email address hidden>
To: <email address hidden>
Cc: Sven Luther <email address hidden>, Debian kernel team <email address hidden>
Subject: Fixed in NMU of initrd-tools 0.1.83

tag 324351 + fixed
tag 329614 + fixed
tag 330446 + fixed
tag 333857 + fixed

quit

This message was generated automatically in response to a
non-maintainer upload. The .changes file follows.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Fri, 14 Oct 2005 07:50:57 +0000
Source: initrd-tools
Binary: initrd-tools
Architecture: source all
Version: 0.1.83
Distribution: unstable
Urgency: low
Maintainer: Debian kernel team <email address hidden>
Changed-By: Sven Luther <email address hidden>
Description:
 initrd-tools - tools to create initrd image for prepackaged Linux kernel
Closes: 324351 329614 330446 333857
Changes:
 initrd-tools (0.1.83) unstable; urgency=low
 .
   [ Simon Horman ]
   * Clarify naming of scripts that will be run.
     Thanks to Simon Schoar. (Closes: #324351)
   * Make sure that path to output image is always fully qualified,
     else an image with a relative will not end up where requested
     (closes: #329614)
 .
   [ dann frazier ]
   * Don't assume update-modules is available during postrm (closes: #330446)
 .
   [ Sven Luther ]
   * Added --supported-(host|target)-version support for the new post-2.6.13
     ramdisk-tool policy. Added linux-ramdisk-tool virtual package too.
     (Closes: #333857)
Files:
 bb2f5ea3f208f972649dfb5d1cac123e 644 utils optional initrd-tools_0.1.83.dsc
 eff2ad61dc7e18f607d541bd71a53dac 28815 utils optional initrd-tools_0.1.83.tar.gz
 dc7b11f6eb3cc3cf934c6639f1a1db3d 31728 utils optional initrd-tools_0.1.83_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)

iD8DBQFDUs2o2WTeT3CRQaQRAhifAJ9jQVG+GwyAs7lGMD/dwllcAN/GmACfSEAA
igkOcTRK7n9s9dQdA/xC2ew=
=7/gM
-----END PGP SIGNATURE-----

Revision history for this message
In , Simon Horman (horms) wrote :

On Sun, Oct 16, 2005 at 03:17:06PM -0700, Sven Luther wrote:
> tag 324351 + fixed
> tag 329614 + fixed
> tag 330446 + fixed
> tag 333857 + fixed
>
> quit
>
> This message was generated automatically in response to a
> non-maintainer upload. The .changes file follows.

For anyone who cares, this was not really an NMU, other
than in the technical sense. It was made by Sven, who
is a member of the Kernel Team, who maintains initrd-tools.

I have added him and myself as uploaders to avoid any future
confusion regarding uploads of initrd-tools made by Sven
(or myself).

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Format: 1.7
> Date: Fri, 14 Oct 2005 07:50:57 +0000
> Source: initrd-tools
> Binary: initrd-tools
> Architecture: source all
> Version: 0.1.83
> Distribution: unstable
> Urgency: low
> Maintainer: Debian kernel team <email address hidden>
> Changed-By: Sven Luther <email address hidden>
> Description:
> initrd-tools - tools to create initrd image for prepackaged Linux kernel
> Closes: 324351 329614 330446 333857
> Changes:
> initrd-tools (0.1.83) unstable; urgency=low
> .
> [ Simon Horman ]
> * Clarify naming of scripts that will be run.
> Thanks to Simon Schoar. (Closes: #324351)
> * Make sure that path to output image is always fully qualified,
> else an image with a relative will not end up where requested
> (closes: #329614)
> .
> [ dann frazier ]
> * Don't assume update-modules is available during postrm (closes: #330446)
> .
> [ Sven Luther ]
> * Added --supported-(host|target)-version support for the new post-2.6.13
> ramdisk-tool policy. Added linux-ramdisk-tool virtual package too.
> (Closes: #333857)
> Files:
> bb2f5ea3f208f972649dfb5d1cac123e 644 utils optional initrd-tools_0.1.83.dsc
> eff2ad61dc7e18f607d541bd71a53dac 28815 utils optional initrd-tools_0.1.83.tar.gz
> dc7b11f6eb3cc3cf934c6639f1a1db3d 31728 utils optional initrd-tools_0.1.83_all.deb
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.1 (GNU/Linux)
>
> iD8DBQFDUs2o2WTeT3CRQaQRAhifAJ9jQVG+GwyAs7lGMD/dwllcAN/GmACfSEAA
> igkOcTRK7n9s9dQdA/xC2ew=
> =7/gM
> -----END PGP SIGNATURE-----

--
Horms

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Mon, 17 Oct 2005 12:23:08 +0900
From: Horms <email address hidden>
To: Debian kernel team <email address hidden>
Cc: <email address hidden>, <email address hidden>,
 <email address hidden>, <email address hidden>
Subject: Re: Fixed in NMU of initrd-tools 0.1.83

On Sun, Oct 16, 2005 at 03:17:06PM -0700, Sven Luther wrote:
> tag 324351 + fixed
> tag 329614 + fixed
> tag 330446 + fixed
> tag 333857 + fixed
>
> quit
>
> This message was generated automatically in response to a
> non-maintainer upload. The .changes file follows.

For anyone who cares, this was not really an NMU, other
than in the technical sense. It was made by Sven, who
is a member of the Kernel Team, who maintains initrd-tools.

I have added him and myself as uploaders to avoid any future
confusion regarding uploads of initrd-tools made by Sven
(or myself).

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Format: 1.7
> Date: Fri, 14 Oct 2005 07:50:57 +0000
> Source: initrd-tools
> Binary: initrd-tools
> Architecture: source all
> Version: 0.1.83
> Distribution: unstable
> Urgency: low
> Maintainer: Debian kernel team <email address hidden>
> Changed-By: Sven Luther <email address hidden>
> Description:
> initrd-tools - tools to create initrd image for prepackaged Linux kernel
> Closes: 324351 329614 330446 333857
> Changes:
> initrd-tools (0.1.83) unstable; urgency=low
> .
> [ Simon Horman ]
> * Clarify naming of scripts that will be run.
> Thanks to Simon Schoar. (Closes: #324351)
> * Make sure that path to output image is always fully qualified,
> else an image with a relative will not end up where requested
> (closes: #329614)
> .
> [ dann frazier ]
> * Don't assume update-modules is available during postrm (closes: #330446)
> .
> [ Sven Luther ]
> * Added --supported-(host|target)-version support for the new post-2.6.13
> ramdisk-tool policy. Added linux-ramdisk-tool virtual package too.
> (Closes: #333857)
> Files:
> bb2f5ea3f208f972649dfb5d1cac123e 644 utils optional initrd-tools_0.1.83.dsc
> eff2ad61dc7e18f607d541bd71a53dac 28815 utils optional initrd-tools_0.1.83.tar.gz
> dc7b11f6eb3cc3cf934c6639f1a1db3d 31728 utils optional initrd-tools_0.1.83_all.deb
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.1 (GNU/Linux)
>
> iD8DBQFDUs2o2WTeT3CRQaQRAhifAJ9jQVG+GwyAs7lGMD/dwllcAN/GmACfSEAA
> igkOcTRK7n9s9dQdA/xC2ew=
> =7/gM
> -----END PGP SIGNATURE-----

--
Horms

Revision history for this message
In , Adam D. Barratt (debian-bts-adam-barratt) wrote : Bugs fixed in NMU, documenting versions

# Hi,
#
# These bugs were fixed in an NMU, but have not been acknowledged by the
# maintainers. With version tracking in the Debian BTS, it is important
# to know which version of a package fixes each bug so that they can be
# tracked for release status, so I'm closing these bugs with the
#relevant version information now

close 271427 8.14+v8.11+urw-0.1
close 314698 0.35-2.1
close 325635 0.35-2.1
close 328017 0.35-2.1
close 320115 2.0-4.2
close 320284 1.11
close 320899 11.4.1870-7.1
close 327078 11.4.1870-7.1
close 327349 11.4.1870-7.1
close 320903 1:0.71-1.2
close 327946 1:0.71-1.2
close 320941 2.0.3-1.1
close 321126 2.6.3.2
close 321545 0.1.3b-1.1
close 341341 0.1.3b-1.1
close 321553 0.1.12-2.2
close 321644 2:1.7.12-1.1
close 346013 2:1.7.12-1.1
close 321816 2.61-2.1
close 321967 4.0.0-2.1
close 330024 4.0.0-2.1
close 321998 0.9.21-0.1
close 322583 0.3.8.1-4
close 322853 0.7.1-3.1
close 356739 0.7.1-3.1
close 322961 0.4.3.1.dfsg-0.1
close 322972 9.4.2-2.4
close 323084 0.4.5+cvs20030824-1.4
close 323160 0.1.10-0.1
close 323355 1.2.11-0.2
close 323725 0.18.2-10.1
close 323942 0.4.0-4.1
close 324371 4.3-18.1
close 324553 2.9.5.0.37.5.2
close 324558 1.2-release-2.1
close 324579 1.11-6.2
close 324606 1.2-release-2.2
close 324908 0.12.4-4.1
close 325210 2.6.0-1.1
close 325490 0.7.1-1.1
close 325514 0.8.6-1.1
close 326468 0.8.6-1.1
close 325532 2:1.7.12-1
close 327366 2:1.7.12-1
close 329778 2:1.7.12-1
close 332480 2:1.7.12-1
close 325635 0.35-2.1
close 328017 0.35-2.1
close 325835 0.1.12-7.1
close 325851 2:1.7.8-1sarge2
close 325938 0.9.8beta2-4.1
close 327930 0.9.8beta2-4.1
close 326285 0.99.3-5.1
close 326295 0.8.2-5.1
close 373110 0.8.2-5.1
close 379331 0.8.2-5.1
close 379334 0.8.2-5.1
close 326298 0.2.12-2.1
close 326311 0.3.5-1pre1.1
close 326355 2.1.8-2.1
close 326362 0.6-7.2
close 326371 0.90beta1-10.1
close 326372 1.0-0.1
close 326378 0.1.17-4.3
close 326466 6.3.2-2.1
close 347129 6.3.2-2.1
close 347205 6.3.2-2.1
close 326489 0.3.7-2.1
close 326756 1.0.9-1.1
close 365518 1.0.9-1.1
close 327429 1.2-1.1
close 350429 1.2-1.1
close 327911 2.3.5-1.1
close 327718 0.6.0-8.2
close 327933 0.9.2-1.1
close 327936 0.8.5-1.1
close 327970 0.5.1-2.1
close 327984 1.3-2.1
close 327986 0.2.36-4.1
close 291328 0.2.36-4.1
close 327996 1.0-1.1
close 328002 1.0.0-9.1
close 328018 2.1.3-2.1
close 328039 1.18A-2.1
close 328172 1.002-0.2
close 328333 4.1.2-1.1
close 328334 1.34-7.1
close 328335 0.8.2-2.1
close 328352 0.13-3.1
close 328364 0.4.0-test5-2.1
close 329467 1.3.1
close 330446 0.1.83
close 333857 0.1.83
close 330666 6:6.2.4.5-0.2
close 330938 0.5.1-2.2

Changed in initrd-tools:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.