Re: [PATCH] Bug fix for LVM

[Bean]

On Thu, Jul 23, 2009 at 1:33 AM, Robert Millan<address@hidden> wrote:
> On Sun, Jul 19, 2009 at 05:41:04PM +0800, Bean wrote:
>> +      if (is_raid)
>> +     printf ("raid mdraid");
>> +
>> +      if (is_lvm)
>> +     printf ((is_raid) ? " lvm" : "lvm");
>
> Is there a better way to handle this?  Perhaps we could make the list
> newline separated instead of space separated and avoid the problem
> altogether.

Hi,

Actually, if we allows an extra space at the end of line, it can be
written like this:

if (is_raid)
     printf ("raid mdraid ");

if (is_lvm)
     printf ("lvm ");

The space is not visible, and ignored by grub-install anyway.

-- 
Bean