Summary: | При инсталяции удаляется уже существующий загрузчик из MBR | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | Boris Popov <popov.b> |
Component: | alterator-lilo | Assignee: | Anton V. Boyarshinov <boyarsh> |
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
Severity: | major | ||
Priority: | P2 | CC: | boyarsh, mike, rom_as, slazav, torokhov_s_a |
Version: | unstable | ||
Hardware: | all | ||
OS: | Linux | ||
Bug Depends on: | |||
Bug Blocks: | 19564 |
Description
Boris Popov
2009-01-28 15:13:34 MSK
в процессе установки столкнулся с аналогичной проблемой: после выбора экпертного режима отказался от установки загрузчика, но перезапись mbr предложенным системой установки загрузчиком всё равно произошла. (до этого в mbr так же стоял grub от другого дистрибутива на этом же физическом диске) 2 ldv: и впрямь, я о такой возможности не подумал. Есть мысли? Так понимаю, дежурным blocker target сейчас Office Server -- в нём тоже реально. (In reply to comment #2) > 2 ldv: и впрямь, я о такой возможности не подумал. Есть мысли? Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких скриптах как *add-memtest. Не считаю эту проблему фатальной. > Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких
> скриптах как *add-memtest.
Какое-то ужасное решение, мне кажется. Искать по логам инсталятора, да? Или передавать через спец.комментарий в lilo.conf?
Может, лучше вообще комментировать boot=, если поставили галочку "не ставить загрузчик"? И пусть кто хочет пытается запускать! :)
(In reply to comment #6) > > Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких > > скриптах как *add-memtest. > > Какое-то ужасное решение, мне кажется. Искать по логам инсталятора, да? Нет, конечно. > Или передавать через спец.комментарий в lilo.conf? > > Может, лучше вообще комментировать boot=, если поставили галочку "не ставить > загрузчик"? И пусть кто хочет пытается запускать! :) Это, кстати, вариант. А проверять с помощью detectliloboot.sh > Это, кстати, вариант. А проверять с помощью detectliloboot.sh
Кстати, похоже, что если нет boot, то lilo тихо ставит на "свой" раздел и не ругается :(
Так что не комментировать, а портить придется.
s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :)
Но как-то это еще более некрасиво...
Ну или писать правильный lilo.conf, но класть его рядом, в lilo.conf.tmp (In reply to comment #7) > Это, кстати, вариант. А проверять с помощью detectliloboot.sh http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g4551ae7 (In reply to comment #8) > Так что не комментировать, а портить придется. > s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :) > Но как-то это еще более некрасиво... boot=/dev/null # ALT Linux bootloader prohibited during installation ? (In reply to comment #9) > Ну или писать правильный lilo.conf, но класть его рядом, в lilo.conf.tmp lilo.conf.sample тогда уж, но тоже да. Причём при повторном запуске из уже установленной системы придётся смотреть, а нет ли такого, и соответственно править его и предупреждать, что пока не решим устанавливать бутлоадер -- эффекта не будет. Если решишь так делать, давай повесим отдельный FR на alterator-lilo и обдумаем там. (In reply to comment #10) > http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g4551ae7 Спасибо! >90% смешанных инсталяций (Ubuntu/Fedora с GRUB) это спасёт, хотя остаётся возможность "чужого" lilo и его мы перетрём. Перестраховка всё равно не помешает, но явно надо сделать и что-то по направлению мыслей Славы. (In reply to comment #8) > > Это, кстати, вариант. А проверять с помощью detectliloboot.sh > > Кстати, похоже, что если нет boot, то lilo тихо ставит на "свой" раздел и не > ругается :( > Так что не комментировать, а портить придется. > s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :) > > Но как-то это еще более некрасиво... Лучше, чтобы lilo.conf был установлен (тогда installkernel и тот же add-memtest смогут его дополнять), но с деактивированным boot=, например, boot="/#disabled#/path/to/device" (In reply to comment #11) > Спасибо! >90% смешанных инсталяций (Ubuntu/Fedora с GRUB) это спасёт, хотя > остаётся возможность "чужого" lilo и его мы перетрём. Перестраховка всё равно > не помешает, но явно надо сделать и что-то по направлению мыслей Славы. detectliloboot.sh ищет boot device согласно записи "boot=" в /etc/lilo.conf (In reply to comment #10) > (In reply to comment #7) > > Это, кстати, вариант. А проверять с помощью detectliloboot.sh > > http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g4551ae7 Нет, дублировать installkernel нехорошо. Может быть, лучше задействовать несуществующую на данный момент опцию installkernel --launch-only: http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g52d3639 > boot=/dev/null # ALT Linux bootloader prohibited during installation Плохо. Информацию не восстановишь простым способом. > lilo.conf.sample Действительно, ни люди, ни installkernel не найдут. > boot="/#disabled#/path/to/device" Видимо, так лучше всего. > Нет, дублировать installkernel нехорошо. Может быть, лучше задействовать > несуществующую на данный момент опцию installkernel --launch-only: Еще лучше -- другую несуществующую опцию: installkernel --label memtest -m "$MEMTEST" > Еще лучше -- другую несуществующую опцию: > installkernel --label memtest -m "$MEMTEST" вот такую примерно: http://git.altlinux.org/people/slazav/packages/?p=bootloader-utils.git;a=commit;h=73daf2b3a8ab6b4cc011caf6b2726391504754de (In reply to comment #15) > > Еще лучше -- другую несуществующую опцию: > > installkernel --label memtest -m "$MEMTEST" > > вот такую примерно: > http://git.altlinux.org/people/slazav/packages/?p=bootloader-utils.git;a=commit;h=73daf2b3a8ab6b4cc011caf6b2726391504754de Где бы ещё пример использования посмотреть? Я имел в виду, что вместо двух вызовов installkernel с исправлением label между ними, можно исправление label унести в installkernel. В том что я написал, параметр --label, надеюсь, означает "использовать указанный label, если такого еще нет". Ну а 50-add-memtest будет, кажется, таким: http://git.altlinux.org/people/slazav/packages/?p=installer-feature-add-memtest.git;a=commit;h=ba00e361e0ca2d87e3449397945066dba90e5713 Перевешиваю на alterator-lilo и закрываю, добавив защиту параметра boot= при выборе пункта "не устанавливать загрузчик". (In reply to comment #18) > Перевешиваю на alterator-lilo и закрываю, добавив защиту параметра boot= при > выборе пункта "не устанавливать загрузчик". Одного этого изменения недостаточно: без изменений в bootloader-utils и add-memtest последний будет завершаться с ошибкой, если boot= будет защищён. > Одного этого изменения недостаточно
Да, с alterator-lilo я чуть было не поспешил :) Хотя, кажется, это не должно бы привести к чему-то плохому... Или add-memtest с ошибкой смущает инсталятор?
Cейчас еще потестирую изменения в bootloader-utils, и, видимо, надо уже делать task со всеми тремя?
(In reply to comment #21) > > Одного этого изменения недостаточно > > Да, с alterator-lilo я чуть было не поспешил :) Хотя, кажется, это не должно бы > привести к чему-то плохому... Или add-memtest с ошибкой смущает инсталятор? Сообщение об ошибке увидит пользователь, в то время как ошибки как таковой нет -- это нехорошо. > Cейчас еще потестирую изменения в bootloader-utils, и, видимо, надо уже делать > task со всеми тремя? Да. alterator-lilo-1.8-alt11 -> sisyphus: * Mon Sep 21 2009 Vladislav Zavjalov <slazav@altlinux> 1.8-alt11 - backend: protect boot= parameter if user don't want to install bootloader (closes: #18679) installer-feature-add-memtest-0.4-alt1 -> sisyphus: * Mon Sep 21 2009 Dmitry V. Levin <ldv@altlinux> 0.4-alt1 - Use installkernel --label (Vladislav Zavjalov; closes: #18679). |