Re: os-prober invalidates part of the menu management

[shawn wilson]

On Nov 22, 2015 1:58 PM, "Arbiel (gmx)" <address@hidden> wrote:
>

> Unfortunately, 30_os-prober generates the following code
> ---------------------------
> set timeout_style=menu
>  if [ "${timeout}" = 0 ]; then
>   set timeout=10
>  fi
> ---------------------------
>
> What is the purpose of this code, and shouldn't it be suppressed ?
>

I assume it's to set a sane default if nothing is set. However this might be better done with:
[ -z "${timeout}" ]
Or maybe:
[ "${timeout}" = "" ]
(all untested)

Though, whatever change should use similar logic that are used for other counters variables.