* Christoph Anton Mitterer <calestyo@scientia.net> [Thu Jun 10, 2010 at 02:51:04PM +0200]: > May I suggest to drop the code from get_fstype in /usr/share/initramfs-tools/scripts/functions > that uses vol_id. > AFAIK vol_id is no longer available in udev or any other Debian package, > so this is dead code. > Attached is a patch for this. > --- /usr/share/initramfs-tools/scripts/functions 2010-06-08 12:10:38.000000000 +0200 > +++ /usr/share/initramfs-tools/scripts/functions 2010-06-10 14:50:25.822262987 +0200 > @@ -285,13 +285,10 @@ > local FS FSTYPE FSSIZE RET > FS="${1}" > - # vol_id has a more complete list of file systems, > - # but fstype is more robust > + # blk_id has a more complete list of file systems, but fstype is more robust > eval $(fstype "${FS}" 2> /dev/null) > if [ "$FSTYPE" = "unknown" ] && command -v blkid >/dev/null 2>&1 ; then > FSTYPE=$(blkid -o value -s TYPE "${FS}") > - elif [ "$FSTYPE" = "unknown" ] && [ -x /lib/udev/vol_id ]; then > - FSTYPE=$(/lib/udev/vol_id -t "${FS}" 2> /dev/null) > fi > RET=$? Yeah, it's just fallback code. udev in lenny provides /lib/udev/vol_id, though lenny's e2fsprogs (which is an essential package) also ships blkid - so this should be save. maks, any objections against applying this patch? regards, -mika-
Attachment:
signature.asc
Description: Digital signature