Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 299656 - lvm2-2.02.56-r2 now includes lvm2create_initrd, but that script is pretty out of date
Summary: lvm2-2.02.56-r2 now includes lvm2create_initrd, but that script is pretty out...
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-04 18:52 UTC by Karl Hakimian
Modified: 2010-01-05 00:07 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated lvm2create_initrd script. Uses cpio instread of ext2 and deals with suspend etc. (lvm2create_initrd,14.31 KB, text/plain)
2010-01-04 18:53 UTC, Karl Hakimian
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Hakimian 2010-01-04 18:52:23 UTC
While the addition of the lvm2create_initrd script is welcome, the included script uses some old mechanisms. I have been updating my own copy of that script for some time and have made it available on the forums. I think including a more modern version would be a good idea. I'm attaching my script to this bug in hopes that it will be added to the official ebuild.

Reproducible: Always

Steps to Reproduce:
1. NA
2.
3.

Actual Results:  
NA

Expected Results:  
NA

Replacement script proposed and provided.
Comment 1 Karl Hakimian 2010-01-04 18:53:17 UTC
Created attachment 215193 [details]
Updated lvm2create_initrd script. Uses cpio instread of ext2 and deals with suspend etc.
Comment 2 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2010-01-04 23:38:35 UTC
Please send your updates to upstream. I included it ONLY because some users were using it already, and they complained it broke with the static changes.
I patched it explicitly so that it would continue to work with the lvm.static naming issue (which your version seems to omit, but is critically required).
Comment 3 Karl Hakimian 2010-01-04 23:41:57 UTC
I can certainly submit it upstream.

The lvm.static is not needed with mine since it will copy the libs needed for the included binaries. I could switch it to static, but I don't see the benefit and the current version is working.
Comment 4 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2010-01-05 00:07:01 UTC
Also please talk to the users from bug #296198 that wanted the script changed to make sure your version works for them.