Re: [PATCH] File access library for lua

[Marco Gerards]

Bean <address@hidden> writes:

> Hi,
>
> Update for this patch:
>
> 1, enum_device now pass fs and uuid as well
> 2, enum_file change parameter order, now the callback function is the
> first, path is the second
> 3, add parameter checking for library function
> 4, add three function
>
> file_eof - test if eof is encounter for a file
> file_exist - test if a file exists
> add_menu - add menu item
>
> I also add a script osdetect.lua which can detect Windows
> 2K/NT/XP/Vista and linux at runtime, to use it, add this in grub.cfg:
>
> source osdetect.lua

Can you please comment on what Robert said?

Personally, I do not mind supporting lua.  But I prefer that scripts
included with GRUB are written using the default scripting language.
Some good reasons for this:

- I do not know lua, I assume Okuji doesn't either.
- Only using one script already requires the user to use lua: slowdown
- Personally, I hope people improve on the default scripting language

--
Marco