Summary: | app-laptop/ibm-acpi: allow it to install despite 2.6.10+ | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | TGL <tom.gl> |
Component: | Current packages | Assignee: | Mobile Herd (OBSOLETE) <mobile+disabled> |
Status: | RESOLVED WONTFIX | ||
Severity: | enhancement | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | ibm-acpi-0.10.patch |
Description
TGL
2005-01-18 13:39:00 UTC
Created attachment 48883 [details, diff]
ibm-acpi-0.10.patch
Since the ibm-acpi ebuilds contain no user-space utilities I see no reason to allow it to be installed when CONFIG_IBM_ACPI is enabled in the kernel. The documentation is included in the kernel as Documentation/ibm-acpi.txt. The example acpi scripts wont help much as the in-kernel version might differ from the version available in portage. The problem with using it on some but not all kernels is easily solved: 'emerge ibm-acpi && emerge -C ibm-acpi'. That will install the kernel module and, since /lib/modules/ is config protected, de-install everything but the kernel module again. > I see no reason to allow it to be installed when CONFIG_IBM_ACPI is enabled Any reason to forbid it? I don't see what this change would break. > The example acpi scripts wont help much as the in-kernel version might differ > from the version available in portage. The worst that can happen is that the version in portage provides handlers for events that don't yet exist with the in-kernel version. But the other handlers are still perfectly useful. > emerge ibm-acpi && emerge -C ibm-acpi But then i loose the docs and don't know about updates. I would compile it out of portage if i didn't want the package to be in installed, or i would use --oneshot if i didn't want it registered in my world. It would break bug reports: "I have problem foo with the ibm_acpi module" "Which version of app-laptop/ibm-acpi do you have installed?" Ok, that makes perfect sense. Thanks for the explanation. |