header_postinst_hook should use /etc/kernel/header_postinst.d instead

Bug #120049 reported by Matt Domsch on 2007-06-12
4
Affects Status Importance Assigned to Milestone
kernel-package (Ubuntu)
Medium
Unassigned
linux-source-2.6.22 (Ubuntu)
Medium
Ben Collins

Bug Description

The linux-image package has preinst and postinst scripts that let other packages hook in, by placing a script in /etc/kernel/preinst.d/ and /etc/kernel/postinst.d/.
The linux-headers package also has a way for other packages to hook in, but it requires editing /etc/kernel-img.conf and adding a header_postinst_hook=... line, which only allows for a single application to hook there.

Dell DKMS 2.0.17 will make use of the /etc/kernel/preinst.d/ and postinst.d/ directories to be aware of linux-image package updates, but it also needs to hook linux-headers postinst. It would be nice for linux-headers to have a similar directory mechanism, ala /etc/kernel/header_preinst.d/ and header_postinst.d/, rather than needing to edit /etc/kernel-img.conf.

Matt Domsch (matt-domsch) wrote :
Ben Collins (ben-collins) wrote :

This actually needs to be handled in kernel-package for feisty.

In gutsy, it's controlled by linux-source itself.

Changed in linux-source-2.6.20:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Changed in linux-source-2.6.22:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Amit Bhutani (amit.bhutani) wrote :

Has this been resolved? Status says "Triaged". What exactly does this mean?

Tim Gardner (timg-tpi) wrote :
Changed in kernel-package:
status: Triaged → Invalid
Changed in linux-source-2.6.22:
status: Triaged → Invalid
Changed in redfish:
status: New → Invalid
Tim Gardner (timg-tpi) wrote :

Need to rethink this.

Changed in linux-source-2.6.22:
assignee: ubuntu-kernel-team → timg-tpi
status: Invalid → Confirmed
Matt Domsch (matt-domsch) wrote :

Patch to kernel-package to make linux-headers search in /etc/kernel/header_postinst.d/ and /etc/kernel/src_postinst.d/ and related .d/$version/ directories for files to execute rather than relying solely upon the header_postinst_hook variable.

Changed in dell:
importance: Undecided → Medium
status: Invalid → Triaged
Changed in kernel-package:
status: Invalid → Confirmed
Matt Domsch (matt-domsch) wrote :

same patch applies to gutsy and feisty both.

Ben Collins (ben-collins) wrote :

Patch applied to headers-postinst hook in ubuntu-gutsy.git

Changed in linux-source-2.6.22:
assignee: timg-tpi → ben-collins
status: Confirmed → Fix Committed
Ben Collins (ben-collins) wrote :

linux-source-2.6.22 (2.6.22-11.34) gutsy; urgency=low

  [Alan Stern]

  * USB: disable autosuspend by default for non-hubs
    - LP: #85488

  [Ben Collins]

  * ubuntu: Enable LEDS_TRIGGERS and related options
    - Needed for iwlwifi
  * ubuntu: Add real ABI files for virtual flavour
  * ubuntu: Re-enable missing CONFIG_SERPENT for hppa64
    - Noticed by Lamont
  * ubuntu: Add linux-headers postinst to handle hooks
    - LP: #125816
  * ubuntu: Add support for /etc/kernel/headers_postinst.d/ to
    headers-postinst
    - LP: #120049
  * cell: Add binary-custom flavour "cell" to support ps3

  [Mattia Dongili]

  * sony-laptop: restore the last user requested brightness level on
    resume.
    - LP: #117331

  [Tejun Heo]

  * ata_piix: fix suspend/resume for some TOSHIBA laptops
    - LP: #139045
  * PCI: export __pci_reenable_device()
    - needed for ata_piix change

  [Tim Gardner]

  * Enable Sierra Wireless MC8775 0x6813
    - LP: #131167

  [Zhang Rui]

  * ACPI: work around duplicate name "VID" problem on T61
    - Noted by mjg59

 -- Ben Collins <email address hidden> Sun, 16 Sep 2007 22:31:47 -0400

Changed in linux-source-2.6.22:
status: Fix Committed → Fix Released

Closing, per Ben Collins' comments on 9/19/07

Changed in dell:
status: Triaged → Fix Released

Per a decision made by the Ubuntu Kernel Team, bugs will longer be assigned to the ubuntu-kernel-team in Launchpad as part of the bug triage process. The ubuntu-kernel-team is being unassigned from this bug report. Refer to https://wiki.ubuntu.com/KernelTeamBugPolicies for more information. Thanks.

Changed in somerville:
importance: Undecided → Medium
status: New → Fix Released
no longer affects: dell
Timothy R. Chavez (timrchavez) wrote :

The bug task for the somerville project has been removed by an automated script. This bug has been cloned on that project and is available here: https://bugs.launchpad.net/bugs/1306152

no longer affects: somerville
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers