<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>18679</bug_id>
          
          <creation_ts>2009-01-28 15:13:34 +0300</creation_ts>
          <short_desc>При инсталяции удаляется уже существующий загрузчик из MBR</short_desc>
          <delta_ts>2009-09-22 12:03:49 +0400</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>alterator-lilo</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>19564</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Boris Popov">popov.b</reporter>
          <assigned_to name="Anton V. Boyarshinov">boyarsh</assigned_to>
          <cc>boyarsh</cc>
    
    <cc>mike</cc>
    
    <cc>rom_as</cc>
    
    <cc>slazav</cc>
    
    <cc>torokhov_s_a</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>85128</commentid>
    <comment_count>0</comment_count>
    <who name="Boris Popov">popov.b</who>
    <bug_when>2009-01-28 15:13:34 +0300</bug_when>
    <thetext>Во время инсталяции и выборе экспертного режима установки lilo, при отказе от установки lilo, lilo, тем не менее, устаналивается в mbr и затирет предыдущий загрузчик (в данном случае у меня стоял grub).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98922</commentid>
    <comment_count>1</comment_count>
    <who name="sorath">torokhov_s_a</who>
    <bug_when>2009-09-11 22:16:35 +0400</bug_when>
    <thetext>в процессе установки столкнулся с аналогичной проблемой:
после выбора экпертного режима отказался от установки загрузчика,
но перезапись mbr предложенным системой установки загрузчиком всё равно произошла.
(до этого в mbr так же стоял grub от другого дистрибутива на этом же физическом диске)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99442</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-09-19 00:49:25 +0400</bug_when>
    <thetext>2 ldv: и впрямь, я о такой возможности не подумал.  Есть мысли?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99444</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-09-19 00:52:18 +0400</bug_when>
    <thetext>Так понимаю, дежурным blocker target сейчас Office Server -- в нём тоже реально.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99537</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 02:46:43 +0400</bug_when>
    <thetext>(In reply to comment #2)
&gt; 2 ldv: и впрямь, я о такой возможности не подумал.  Есть мысли?

Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких скриптах как *add-memtest.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99538</commentid>
    <comment_count>5</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 02:48:05 +0400</bug_when>
    <thetext>Не считаю эту проблему фатальной.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99546</commentid>
    <comment_count>6</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-20 03:51:45 +0400</bug_when>
    <thetext>&gt; Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких
&gt; скриптах как *add-memtest.

Какое-то ужасное решение, мне кажется. Искать по логам инсталятора, да? Или передавать через спец.комментарий в lilo.conf?

Может, лучше вообще комментировать boot=, если поставили галочку &quot;не ставить загрузчик&quot;? И пусть кто хочет пытается запускать! :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99547</commentid>
    <comment_count>7</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 03:59:23 +0400</bug_when>
    <thetext>
(In reply to comment #6)
&gt; &gt; Помечать факт отказа от установки загрузчика, и проверять эту пометку в таких
&gt; &gt; скриптах как *add-memtest.
&gt; 
&gt; Какое-то ужасное решение, мне кажется. Искать по логам инсталятора, да?

Нет, конечно.

&gt; Или передавать через спец.комментарий в lilo.conf?
&gt; 
&gt; Может, лучше вообще комментировать boot=, если поставили галочку &quot;не ставить
&gt; загрузчик&quot;? И пусть кто хочет пытается запускать! :)

Это, кстати, вариант.  А проверять с помощью detectliloboot.sh</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99548</commentid>
    <comment_count>8</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-20 04:07:52 +0400</bug_when>
    <thetext>&gt; Это, кстати, вариант.  А проверять с помощью detectliloboot.sh

Кстати, похоже, что если нет boot, то lilo тихо ставит на &quot;свой&quot; раздел и не ругается :(
Так что не комментировать, а портить придется.
s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :)

Но как-то это еще более некрасиво...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99550</commentid>
    <comment_count>9</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-20 04:14:49 +0400</bug_when>
    <thetext>Ну или писать правильный lilo.conf, но класть его рядом, в lilo.conf.tmp</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99552</commentid>
    <comment_count>10</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 04:27:54 +0400</bug_when>
    <thetext>(In reply to comment #7)
&gt; Это, кстати, вариант.  А проверять с помощью detectliloboot.sh

http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g4551ae7</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99575</commentid>
    <comment_count>11</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-09-20 15:54:37 +0400</bug_when>
    <thetext>(In reply to comment #8)
&gt; Так что не комментировать, а портить придется.
&gt; s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :)
&gt; Но как-то это еще более некрасиво...
boot=/dev/null # ALT Linux bootloader prohibited during installation
?

(In reply to comment #9)
&gt; Ну или писать правильный lilo.conf, но класть его рядом, в lilo.conf.tmp
lilo.conf.sample тогда уж, но тоже да.  Причём при повторном запуске из уже установленной системы придётся смотреть, а нет ли такого, и соответственно править его и предупреждать, что пока не решим устанавливать бутлоадер -- эффекта не будет.  Если решишь так делать, давай повесим отдельный FR на alterator-lilo и обдумаем там.

(In reply to comment #10)
&gt; http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-g4551ae7
Спасибо!  &gt;90% смешанных инсталяций (Ubuntu/Fedora с GRUB) это спасёт, хотя остаётся возможность &quot;чужого&quot; lilo и его мы перетрём.  Перестраховка всё равно не помешает, но явно надо сделать и что-то по направлению мыслей Славы.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99580</commentid>
    <comment_count>12</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 16:23:35 +0400</bug_when>
    <thetext>(In reply to comment #8)
&gt; &gt; Это, кстати, вариант.  А проверять с помощью detectliloboot.sh
&gt; 
&gt; Кстати, похоже, что если нет boot, то lilo тихо ставит на &quot;свой&quot; раздел и не
&gt; ругается :(
&gt; Так что не комментировать, а портить придется.
&gt; s/^[[:space:]]*boot=/__remove_this_to_use_lilo!__ boot=/ :)
&gt; 
&gt; Но как-то это еще более некрасиво...

Лучше, чтобы lilo.conf был установлен (тогда installkernel и тот же add-memtest смогут его дополнять), но с деактивированным boot=, например,
boot=&quot;/#disabled#/path/to/device&quot;

(In reply to comment #11)
&gt; Спасибо!  &gt;90% смешанных инсталяций (Ubuntu/Fedora с GRUB) это спасёт, хотя
&gt; остаётся возможность &quot;чужого&quot; lilo и его мы перетрём.  Перестраховка всё равно
&gt; не помешает, но явно надо сделать и что-то по направлению мыслей Славы.

detectliloboot.sh ищет boot device согласно записи &quot;boot=&quot; в /etc/lilo.conf</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99597</commentid>
    <comment_count>13</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-20 20:11:26 +0400</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #7)
&gt; &gt; Это, кстати, вариант.  А проверять с помощью detectliloboot.sh
&gt; 
&gt; 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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99612</commentid>
    <comment_count>14</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-20 22:55:35 +0400</bug_when>
    <thetext>&gt; boot=/dev/null # ALT Linux bootloader prohibited during installation

Плохо. Информацию не восстановишь простым способом.

&gt; lilo.conf.sample

Действительно, ни люди, ни installkernel не найдут.

&gt; boot=&quot;/#disabled#/path/to/device&quot;

Видимо, так лучше всего.

&gt; Нет, дублировать installkernel нехорошо.  Может быть, лучше задействовать
&gt; несуществующую на данный момент опцию installkernel --launch-only:

Еще лучше -- другую несуществующую опцию:
installkernel --label memtest -m &quot;$MEMTEST&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99642</commentid>
    <comment_count>15</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-21 11:20:05 +0400</bug_when>
    <thetext>&gt; Еще лучше -- другую несуществующую опцию:
&gt; installkernel --label memtest -m &quot;$MEMTEST&quot;

вот такую примерно:
http://git.altlinux.org/people/slazav/packages/?p=bootloader-utils.git;a=commit;h=73daf2b3a8ab6b4cc011caf6b2726391504754de</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99648</commentid>
    <comment_count>16</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-21 12:55:18 +0400</bug_when>
    <thetext>(In reply to comment #15)
&gt; &gt; Еще лучше -- другую несуществующую опцию:
&gt; &gt; installkernel --label memtest -m &quot;$MEMTEST&quot;
&gt; 
&gt; вот такую примерно:
&gt; http://git.altlinux.org/people/slazav/packages/?p=bootloader-utils.git;a=commit;h=73daf2b3a8ab6b4cc011caf6b2726391504754de

Где бы ещё пример использования посмотреть?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99650</commentid>
    <comment_count>17</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-21 13:08:32 +0400</bug_when>
    <thetext>Я имел в виду, что вместо двух вызовов installkernel с исправлением label между ними, можно исправление label унести в installkernel.

В том что я написал, параметр --label, надеюсь, означает &quot;использовать указанный label, если такого еще нет&quot;.

Ну а 50-add-memtest будет, кажется, таким:
http://git.altlinux.org/people/slazav/packages/?p=installer-feature-add-memtest.git;a=commit;h=ba00e361e0ca2d87e3449397945066dba90e5713</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99701</commentid>
    <comment_count>18</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-21 15:44:33 +0400</bug_when>
    <thetext>Перевешиваю на alterator-lilo и закрываю, добавив защиту параметра boot= при выборе пункта &quot;не устанавливать загрузчик&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99712</commentid>
    <comment_count>19</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-21 16:22:52 +0400</bug_when>
    <thetext>(In reply to comment #18)
&gt; Перевешиваю на alterator-lilo и закрываю, добавив защиту параметра boot= при
&gt; выборе пункта &quot;не устанавливать загрузчик&quot;.

Одного этого изменения недостаточно: без изменений в bootloader-utils и add-memtest последний будет завершаться с ошибкой, если boot= будет защищён.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99715</commentid>
    <comment_count>20</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-21 16:35:03 +0400</bug_when>
    <thetext>http://git.altlinux.org/people/ldv/packages/?p=installer-feature-add-memtest.git;a=commitdiff;h=0.3-alt1-1-gebb3e16</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99717</commentid>
    <comment_count>21</comment_count>
    <who name="Vladislav Zavjalov">slazav</who>
    <bug_when>2009-09-21 16:48:46 +0400</bug_when>
    <thetext>&gt; Одного этого изменения недостаточно

Да, с alterator-lilo я чуть было не поспешил :) Хотя, кажется, это не должно бы привести к чему-то плохому... Или add-memtest с ошибкой смущает инсталятор?

Cейчас еще потестирую изменения в bootloader-utils, и, видимо, надо уже делать task со всеми тремя?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99719</commentid>
    <comment_count>22</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2009-09-21 17:02:23 +0400</bug_when>
    <thetext>(In reply to comment #21)
&gt; &gt; Одного этого изменения недостаточно
&gt; 
&gt; Да, с alterator-lilo я чуть было не поспешил :) Хотя, кажется, это не должно бы
&gt; привести к чему-то плохому... Или add-memtest с ошибкой смущает инсталятор?

Сообщение об ошибке увидит пользователь, в то время как ошибки как таковой нет -- это нехорошо.

&gt; Cейчас еще потестирую изменения в bootloader-utils, и, видимо, надо уже делать
&gt; task со всеми тремя?

Да.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99803</commentid>
    <comment_count>23</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2009-09-22 12:03:48 +0400</bug_when>
    <thetext>alterator-lilo-1.8-alt11 -&gt; sisyphus:

* Mon Sep 21 2009 Vladislav Zavjalov &lt;slazav@altlinux&gt; 1.8-alt11

- backend: protect boot= parameter if user don&apos;t want to install
  bootloader (closes: #18679)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99804</commentid>
    <comment_count>24</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2009-09-22 12:03:49 +0400</bug_when>
    <thetext>installer-feature-add-memtest-0.4-alt1 -&gt; sisyphus:

* Mon Sep 21 2009 Dmitry V. Levin &lt;ldv@altlinux&gt; 0.4-alt1

- Use installkernel --label (Vladislav Zavjalov; closes: #18679).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>