Re: backwards compatibility in grub.cfg

[Christian Franke]

Robert Millan wrote:

> diff -x configure -x config.h.in -x CVS -x '*~' -x '*.mk' -urp 
> ../grub2/util/update-grub_lib.in ./util/update-grub_lib.in
> --- ../grub2/util/update-grub_lib.in    2008-05-30 22:55:35.000000000 +0200
> +++ ./util/update-grub_lib.in   2008-05-30 23:16:53.000000000 +0200
> @@ -116,10 +116,9 @@ prepare_grub_to_access_device ()
>  
>    # If there's a filesystem UUID that GRUB is capable of identifiing, use it;
>    # otherwise set root as per value in device.map.
> +  echo "set root=`${grub_probe} --device ${device} --target=drive`"
>    if fs_uuid="`grub-probe --device ${device} --target=fs_uuid 2> /dev/null`" ; 
> then
>   

Outside of the scope of the patch, but shouldn't '${grub_probe}' be used 
instead of 'grub-probe' in the rest of the script?

Same issue is in update-grub.in: grub_probe is set but never used, so 
update-grub fails if /usr/sbin is not in PATH.

--
Christian Franke