Bug 74167
Summary: | mkinitrd breaks on legal content of /etc/modules.conf | ||||||
---|---|---|---|---|---|---|---|
Product: | [Retired] Red Hat Linux | Reporter: | Michal Jaegermann <michal> | ||||
Component: | mkinitrd | Assignee: | Erik Troan <ewt> | ||||
Status: | CLOSED WONTFIX | QA Contact: | David Lawrence <dkl> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | 7.3 | CC: | herrold | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2003-01-14 22:33:42 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Michal Jaegermann
2002-09-17 07:21:39 UTC
Created attachment 76345 [details]
Simple patch to handle all legal /etc/modules.conf files in mkinitrd
Oops! A space is missing. The patch should read options=$( /sbin/modprobe -c | grep "^options $module " | \ instead of options=$( /sbin/modprobe -c | grep "^options $module" | \ or, alternatively, "$module" can be tested in a subseqent while-loop. Sigh! This still will not work in general without adding to modprobe an option to request a configuration for a specified kernel instead of the current one. This is not likely be very difficult and/or invasive. Trying to reparse modules.conf correctly in a shell script likely will be always a very dicey proposition. as you've noted, this is not going to be straightforward to fix. if the upstream modutils folks decide to help out I'll add the support to mkinitrd |