Re: status update for grub 2 developments?

[Bean <bean123@126.com>]

[-- Attachment #1: Type: text/plain, Size: 850 bytes --]

On Sun, Jul 01, 2007 at 07:49:20PM +0200, adrian15 wrote:
> Bean escribió:
> >>Well, if people want to work on scripting or networking they should
> >>talk to me.  I have lots of code on my hd that is incomplete or just
> >>not cleaned up.  It would be a waste of time if it were rewritten.
> >>And please poke me a lot to get me to work... ;)
> >
> >I'm interested in scripting. What could I do to help you ?
> 
> 	As long as you may re-read Marco_g source code you can try to fix a 
> 	bug
> that I have found I suppose in the parser.
> 
> 	If you type any command with a \ character not being inside " "
> characters it gives an grub internal error.
> 
> 	echo "\n"		--> OK
> 	echo \n fff 		--> GRUB INTERNAL ERROR

It's fixed, please test.

echo "\n"
\n

echo \n fff
n fff

Note: you need to apply the patch grub2-parser-u2.diff first.

-- 
Bean

[-- Attachment #2: grub2-parser-u2-1.diff --]
[-- Type: text/plain, Size: 604 bytes --]

	* normal/lexer.c (check_textstate): Add GRUB_PARSER_STATE_ESC.


Index: normal/lexer.c
===================================================================
RCS file: /sources/grub/grub2/normal/lexer.c,v
retrieving revision 1.6
diff -u -r1.6 lexer.c
--- normal/lexer.c	4 Jun 2006 15:56:55 -0000	1.6
+++ normal/lexer.c	25 Jun 2007 18:27:59 -0000
@@ -41,7 +41,8 @@
 {
   return (state == GRUB_PARSER_STATE_TEXT
 	  || state == GRUB_PARSER_STATE_QUOTE
-	  || state == GRUB_PARSER_STATE_DQUOTE);
+	  || state == GRUB_PARSER_STATE_DQUOTE
+	  || state == GRUB_PARSER_STATE_ESC);
 }
 
 struct grub_lexer_param *