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

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

    <bug>
          <bug_id>1015</bug_id>
          
          <creation_ts>2002-06-18 02:46:22 +0400</creation_ts>
          <short_desc>detectloader misdetects the loader in some circumstances</short_desc>
          <delta_ts>2009-09-16 17:31:02 +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>bootloader-utils</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>P4</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>7371</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Mikhail Zabaluev">mhz</reporter>
          <assigned_to name="placeholder@altlinux.org">placeholder</assigned_to>
          <cc>at</cc>
    
    <cc>bga</cc>
    
    <cc>boyarsh</cc>
    
    <cc>crux</cc>
    
    <cc>glebfm</cc>
    
    <cc>inger</cc>
    
    <cc>ldv</cc>
    
    <cc>mike</cc>
    
    <cc>placeholder</cc>
    
    <cc>rider</cc>
    
    <cc>sem</cc>
    
    <cc>slazav</cc>
    
    <cc>sr</cc>
    
    <cc>vitty</cc>
    
    <cc>vsu</cc>
    
    <cc>vt</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>968</commentid>
    <comment_count>0</comment_count>
    <who name="Mikhail Zabaluev">mhz</who>
    <bug_when>2002-06-18 02:46:22 +0400</bug_when>
    <thetext>In this loop:

    foreach (@disks) {
        $loader = typeOfMBR($_) and return $loader;
    }

The @disks list is not sorted by device name. This can lead to, e.g., /dev/hdd checked before /dev/hda. My /dev/hdd (my old drive) hosts LILO, and this is what detectloader finds first. This resulted in a hosed system when I have updated grub on my system which resides in /dev/hda.
---

---

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3775</commentid>
    <comment_count>1</comment_count>
    <who name="Mikhail Zabaluev">mhz</who>
    <bug_when>2002-06-18 03:14:07 +0400</bug_when>
    <thetext>I hereby propose a solution.
Make detectloader accept the optional device parameter, as a pathname or in
Grub format. Having this parameter specified, detectloader restricts probes to this device. The install scripts for Grub and Lilo should then determine target device from their respective configuration files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>7140</commentid>
    <comment_count>2</comment_count>
    <who name="Mikhail Zabaluev">mhz</who>
    <bug_when>2002-06-18 03:14:07 +0400</bug_when>
    <thetext>I hereby propose a solution.
Make detectloader accept the optional device parameter, as a pathname or in
Grub format. Having this parameter specified, detectloader restricts probes to this device. The install scripts for Grub and Lilo should then determine target device from their respective configuration files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3776</commentid>
    <comment_count>3</comment_count>
    <who name="inger@altlinux.org">inger</who>
    <bug_when>2003-01-15 18:31:01 +0300</bug_when>
    <thetext>добавлен skiplist</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>7141</commentid>
    <comment_count>4</comment_count>
    <who name="inger@altlinux.org">inger</who>
    <bug_when>2003-01-15 18:31:01 +0300</bug_when>
    <thetext>добавлен skiplist</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>10291</commentid>
    <comment_count>5</comment_count>
    <who name="Sergey Vlasov">vsu</who>
    <bug_when>2003-10-18 21:57:33 +0400</bug_when>
    <thetext>1) skiplist - это не решение. Невозможно перечислить все места, где не надо
искать загрузчик - после подключения USB flash и прочих подобных устройств тоже
появляются записи в /proc/partitions. Должно быть ещё средство для явного
указания списка устройств.

2) Всё это должно быть документировано - запись &quot;skiplist for mhz&quot; в %changelog
пакета никак не может считаться документацией.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>14679</commentid>
    <comment_count>6</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2004-06-03 20:16:21 +0400</bug_when>
    <thetext>Reassigned to maintainer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35560</commentid>
    <comment_count>7</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2006-02-09 16:49:03 +0300</bug_when>
    <thetext>Неужто нельзя прочитать существующие конфиги, посмотреть где там у них должен
быть bootloader и уже там искать ?
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>96454</commentid>
    <comment_count>8</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2009-08-14 18:11:09 +0400</bug_when>
    <thetext>Мужики, подсобите?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99200</commentid>
    <comment_count>9</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2009-09-16 17:31:02 +0400</bug_when>
    <thetext>bootloader-utils-0.4.1-alt1 -&gt; sisyphus:

* Wed Sep 16 2009 Vladislav Zavjalov &lt;slazav@altlinux&gt; 0.4.1-alt1

- installkernel:
  - change lilo restarting logic (closes #1014, #1015, #20695):
    old behaviour: restart lilo if first bootloader found in
      devices from /proc/partitions is lilo
    new one: restart lilo when lilo bootloader is found in
      device from boot= setting in lilo.conf
  - when detectliloboot.sh fails to find device, there are no
    need to run detectloader.sh (thx Dmitry V. Levin)
      (closes: #21106)
  - suppress unwanted output from cd -
- detectloader: replace by two shell scripts:
    detectliloboot.sh -- to get boot device from lilo.conf
    detectloader.sh -- to get bootloader type on the given device or file
- bootloader_utils.pm:
  - (getroot): recognize UUID and LABEL
      (thx Dmitry V. Levin) (closes: #2194, #18127)
  - remove unused functions (partitions, typeOfMBR) used in
      removed detectloader program
- rebootin
  - remove GRUB support
  - fix working with quoted labels in lilo.conf (closes: #11446).
  - cleanup code, improve error and help messages
  - add long options, add -l option for listing available labels.
  - update manpage
- remove URL and Packager tags (thx Dmitry V. Levin).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>