Re: [PATCH] Updated fix for Debian Bug#574863: boot failure with lvm2 and snapshot volumes

2010-07-02 Thread sean finney
hi there, On Tue, Jun 29, 2010 at 07:30:57AM +0200, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > I've cleaned up your patch using my more generic string parsing > functions. Could you test attached patch? i read over the patch and it looks good. i've just tested the patch and grub-install, gr

Re: [PATCH] Updated fix for Debian Bug#574863: boot failure with lvm2 and snapshot volumes

2010-06-28 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 06/03/2010 08:41 PM, sean finney wrote: > and here is a slightly updated patch. there is no functional change in the > code, i have only reformatted the whitespace etc so that the code matches > with the style of the surrounding code. > > i have also tested this now on my primary testing system

[PATCH] Updated fix for Debian Bug#574863: boot failure with lvm2 and snapshot volumes

2010-06-03 Thread sean finney
and here is a slightly updated patch. there is no functional change in the code, i have only reformatted the whitespace etc so that the code matches with the style of the surrounding code. i have also tested this now on my primary testing system and haven't noticed any problems. sean O

Fix for Debian Bug#574863: boot failure with lvm2 and snapshot volumes

2010-06-02 Thread sean finney
okay, I think the attached patch should fix the problem. I haven't tested it thoroughly, though my system does boot. It seems there may be a seperate issue with os-prober that results in some junk entries being added to grub.conf if the snapshot volumes happen to be root filesystems, but that pro