BVK Chaitanya wrote:
> Attached!
>
>
>   

+size=128M
+disk1=`mktemp`
+disk2=`mktemp`
Plain mktemp will fail on FreeBSD. Idk for well freebsd supports linux lvm though
+lvm=/sbin/lvmn
+parted=/sbin/parted
+losetup=/sbin/losetup
You don't check for real existence of these files. And letting use $PATH is better IMO than hardcoding path to executables
+if test `id -u` != "0"; then
+    exit 0
+fi
+
There should be a warning here

>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> http://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko