grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] Bug fix for parser


From: Bean
Subject: [PATCH] Bug fix for parser
Date: Mon, 31 Dec 2007 23:51:39 +0800

Hi,

I write a patch for parser related bug, it fix the following problems:

1, major modification for parse.y, now it should work better. For example,

if echo aa; echo bb; then echo cc; fi

it works properly now.

2, Check for undefined variable. for example, if AA is not defined,

echo $AA

caused problem in the old version. Now it shows blank line.

3, This following commands:

function aa { echo bb; }
aa

Shows

bb
error: unknown command `aa'

Now it is ok.

4, If an command return error in grub.cfg, the following content is
not parsed, for example

test A=B
menuentry aa {
set root=(hd0,1)
chainloader +1
}

The menu is not parsed as test A=B set errno to 1.

This bug can be quite tricky, for example, i remember someone report
that update-grub has problem as the menu is not showed at all, but in
fact, the problem is caused by the font command.

5, echo module not included in command.lst
yes, this is problem is very old, but nobody seems to be fixing it.

-- 
Bean

Attachment: grub2-parser-new2.diff
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]