Bug 441111

Summary:	RFE: own /etc/kernel/ and /etc/kernel/postinst.d/
Product:	[Fedora] Fedora	Reporter:	Thorsten Leemhuis <fedora>
Component:	mkinitrd	Assignee:	Peter Jones <pjones>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	low
Version:	rawhide	CC:	dcantrell, matt_domsch, wtogami
Target Milestone:	---	Keywords:	FutureFeature
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	6.0.49-1.fc9	Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-04-24 04:52:11 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:

Description Thorsten Leemhuis 2008-04-06 11:19:37 UTC

Description of problem:
Mkinitrd since applying the RFE from Bug 433121 supports calling stuff from
/etc/kernel/postinst.d/ via /sbin/new-kernel-pkg in rpmposttrans; different
packages might use this directory, but the directory does not exists. That's
bad, as each and every package that might want to use it would need to own it,
which is not acceptable according to the fedora packaging guidelines. Thus I'd
say the mkinitrd package should create and own those packages

Version-Release number of selected component (if applicable):
mkinitrd-6.0.41-1.fc9.x86_64

Additional info:
Mattdm, do you agree with that? If yes I can prepare and apply a patch for the
spec file.

While at it: Matt, in your patch posted in 
https://www.redhat.com/archives/fedora-kernel-list/2008-February/msg00083.html
there were different hooks for install (/etc/kernel/postinst.d/) and uninstall
(/etc/kernel/prerm.d/). When looking at /sbin/new-kernel-pkg from rawhide
mkinitrd it seems during both install and uninstall scripts from
/etc/kernel/postinst.d/ get called. Is that the intended way or is there a
mis-merge?

Comment 1 Thorsten Leemhuis 2008-04-06 18:19:16 UTC

While at it (2): The code in /sbin/new-kernel-pkg is broken in any case. See
line 192:

[...]
files="/etc/kernel/postinst.d/*[^~] /etc/kernel/postinst.d/$version/*[^~]"
for f in files ; do
[...]

s/in files/in $files/ might help a lot ;-)

Comment 2 Peter Jones 2008-04-07 14:42:26 UTC

Will be fixed in 6.0.42, thanks.

Comment 3 Jon Stanley 2008-04-23 20:29:51 UTC

Adding FutureFeature keyword to RFE's.

Comment 4 Thorsten Leemhuis 2008-04-24 04:52:11 UTC

this actually was fixed; closing