Re: [PATCH] Grub2 cannot find LVM volume groups with a dash (-) in the name

[Sander Marechal <s.marechal@jejik.com>]

Felix Zielcke wrote:
> Well attached is now my cute little asprintf patch with a very ugly LVM
> part.
> 
> It works for
> 
> volumegroup: linuxvg
> logicalvolume: lvol0
> /dev/mapper/linuxvg-lvol0
> 
> volume group: m-y--vg
> logical volume: l-vol--0
> /dev/mapper/m--y----vg-l--vol----0
> 
> 
> Please comment this, please give me hints how to make it better.
> I don't like the code myself but it seems like we should handle this
> double dash problem :(

I commented on this on #grub but will post here as well, to preserve it
for posterity :-)

I tested the patch against grub 20080724 in Debian Lenny and it works
great :-) I can't comment on the rest of the code. I'm not a grub developer.

-- 
Sander Marechal