Comment 31 for bug 1273764

Revision history for this message
Tony Beta Lambda (tony-xty) wrote :

After reading quick_boot.patch, I think the environment variable $quick_boot (set by default) was added in order to give the user an option to force timeout to be 0. However, the test seems reversed to me:

  if [ "$quick_boot" = 1 ] && [ "x${found_other_os}" != "x" ]; then
    cat << EOF
set timeout_style=menu
if [ "\${timeout}" = 0 ]; then
  set timeout=10
fi
EOF
  fi

meaning timeout is increased to 10 when $quick_boot is set, which is not what I'll call "quick_boot". I suggest changing it to "$quick_boot" = 0.

And there should be a way to modify this variable somewhere else, since any change to the file itself will get overridden in an upgrade.