Bug 25676

Summary:	'failsafe vga=normal' parameters used for non-failsafe menu items
Product:	Sisyphus	Reporter:	Sergey Vlasov <vsu>
Component:	grub2	Assignee:	Vitaly Kuznetsov <vitty>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	boyarsh, mcpain, mike, nickel
Version:	unstable
Hardware:	all
OS:	Linux

Description Sergey Vlasov 2011-05-27 14:53:14 MSK

Начиная с grub2-1.99-alt1, в /etc/grub.d/10_linux при генерации grub.cfg параметры GRUB_CMDLINE_LINUX='failsafe vga=normal' добавляются ко всем вариантам загрузки, а не только к failsafe, как было в предыдущих версиях.

Вообще, если делать как следует, там должно быть не два параметра конфигурации, а три - помимо существующих GRUB_CMDLINE_LINUX и GRUB_CMDLINE_LINUX_DEFAULT, ещё должен быть GRUB_CMDLINE_LINUX_RECOVERY (куда следует унести прибитую гвоздями в скрипте опцию single, которая, кстати, в предыдущей сборке пакета тоже убиралась патчем, а теперь не убирается).

Comment 1 Vitaly Kuznetsov 2011-05-27 15:30:42 MSK

Логика в самом grub2 следующая:
GRUB_CMDLINE_LINUX - и для failsafe, и для normal
GRUB_CMDLINE_LINUX_DEFAULT - для normal

для 1.98 я её немного менял в следующем ключе:
GRUB_CMDLINE_LINUX - для failsafe
GRUB_CMDLINE_LINUX_DEFAULT - для normal и failsafe

какова должна быть логика, если параметров будет 3?

Comment 2 Vitaly Kuznetsov 2011-05-27 17:45:55 MSK

(In reply to comment #1)
> для 1.98 я её немного менял в следующем ключе:
> GRUB_CMDLINE_LINUX - для failsafe
> GRUB_CMDLINE_LINUX_DEFAULT - для normal и failsafe

не совсем так, 
GRUB_CMDLINE_LINUX - для failsafe
GRUB_CMDLINE_LINUX_DEFAULT - для normal

я сейчас верну эту логику, а потом можно подумать о том, как сделать правильно

Comment 3 Repository Robot 2011-05-27 18:20:27 MSK

grub2-1.99-alt3 -> sisyphus:

* Fri May 27 2011 Vitaly Kuznetsov <vitty@altlinux> 1.99-alt3
- fix adding failsafe options to non-failsave cmdline (ALT #25676)
- change 'splash=silent' to 'splash' in default sysconfig

Comment 4 Sergey Vlasov 2011-05-27 19:27:42 MSK

(В ответ на комментарий №1)
> какова должна быть логика, если параметров будет 3?

GRUB_CMDLINE_LINUX - и для failsafe, и для normal
GRUB_CMDLINE_LINUX_DEFAULT - для normal
GRUB_CMDLINE_LINUX_RECOVERY - для failsafe
(первые два варианта совпадают с апстримным поведением, добавляется только одна дополнительная переменная)

Хотя это надо бы предлагать апстриму.