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

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

    <bug>
          <bug_id>28271</bug_id>
          
          <creation_ts>2012-12-25 15:11:36 +0400</creation_ts>
          <short_desc>Гибридные образы, создаваемые новым mkimage через xorisso, имеют неправильную метку тома.</short_desc>
          <delta_ts>2016-08-15 12:58:08 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>mkimage</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>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>27685</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Anton V. Boyarshinov">boyarsh</reporter>
          <assigned_to name="Michael Shigorin">mike</assigned_to>
          <cc>aen</cc>
    
    <cc>glebfm</cc>
    
    <cc>legion</cc>
    
    <cc>mike</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>136373</commentid>
    <comment_count>0</comment_count>
    <who name="Anton V. Boyarshinov">boyarsh</who>
    <bug_when>2012-12-25 15:11:36 +0400</bug_when>
    <thetext>Гибридные образы, создаваемые новым mkimage через xorisso, имеют неправильную метку тома. Метка должна браться из соответствуещей переменной mkimage (ну и начинаться со слова ALT -- иначе наш пропагатор не определит её как образ, с которого надо ставиться)

/dev/sdc1: UUID=&quot;2012-12-24-10-11-39-00&quot; LABEL=&quot;ISOIMAGE&quot; TYPE=&quot;iso9660&quot; 
/dev/sdc2: SEC_TYPE=&quot;msdos&quot; UUID=&quot;A575-1063&quot; TYPE=&quot;vfat&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136375</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-12-25 16:50:32 +0400</bug_when>
    <thetext>(In reply to comment #0)
&gt; Гибридные образы, создаваемые новым mkimage через xorisso, имеют неправильную
&gt; метку тома. Метка должна браться из соответствуещей переменной mkimage
Это уже исправлено, но ещё не долетело до сизифа:
http://git.altlinux.org/people/mike/packages/?p=mkimage.git;a=commitdiff;h=6054b1400ff7704c5446def6bab92cef36146c81

2 legion: ping :)

&gt; (ну и начинаться со слова ALT -- иначе наш пропагатор не определит её как
&gt; образ, с которого надо ставиться)
А вот это я &quot;чуть не&quot; сломал в не смерженном ещё левобранче m-p, но для буквально пары опубликованных образов и regular-* это изменение точно не затронуло:

 distro/.base: distro/.init use/kernel
 	@$(call set,META_SYSTEM_ID,LINUX)
-	@$(call set,META_VOL_ID,ALT Linux $(IMAGE_NAME)/$(ARCH))
+	@$(call set,META_VOL_ID,$(IMAGE_NAME))
 	@$(call set,META_VOL_SET,ALT Linux)

Дело в том, что &quot;ALT Linux &quot;&quot;/x86_64&quot; добавляет 17 символов, а жёсткий предел размера строки -- 32 символа (дальше xorriso вываливается в конце сборки, что особенно приятно).  Т.е. минимум половину лимита мы съедаем и в случае i586.

Это отдельная проблема, но с ней тоже надо что-то думать, а пока хоть упомяну.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136376</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-12-25 16:51:44 +0400</bug_when>
    <thetext>2 legion: т.е. прошу собрать в сизиф aaaf8e4ad81738e2ec03024ced98b8aa628f6444 из http://git.altlinux.org/people/mike/packages/?p=mkimage.git или посмотреть и подтвердить, что это приличный 0.2.6-alt1.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136378</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Gladkov">legion</who>
    <bug_when>2012-12-25 17:29:01 +0400</bug_when>
    <thetext>(В ответ на комментарий №2)
&gt; 2 legion: т.е. прошу собрать в сизиф aaaf8e4ad81738e2ec03024ced98b8aa628f6444
&gt; из http://git.altlinux.org/people/mike/packages/?p=mkimage.git или посмотреть и
&gt; подтвердить, что это приличный 0.2.6-alt1.

Не нужно ли в

http://git.altlinux.org/people/mike/packages/?p=mkimage.git;a=blob;f=tools/mki-pack-isoboot;h=7f75a5a529ef112586d66972f72678ee06efb5a9;hb=6a1a4b6d7b5678bcf624dce4b2b50a9f11787c50#l34

также использовать новую переменную ISOHYBRID ?

Иначе получается, что если нет EFI_BOOTLOADER и ISOHYBRID, то syslinux будет поставлен, но не нужен.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136381</commentid>
    <comment_count>4</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-12-25 18:52:57 +0400</bug_when>
    <thetext>(In reply to comment #3)
&gt; Не нужно ли в [tools/mki-pack-isoboot] также использовать новую переменную
&gt; ISOHYBRID ?
Нужно; возможно, он вытянулся ещё чем-то в профиле, т.к. работающие гибридные образы я предложенным mkimage собирал.

&gt; Иначе получается, что если нет EFI_BOOTLOADER и ISOHYBRID,
&gt; то syslinux будет поставлен, но не нужен.
Он тогда как раз может быть _не_ поставлен, но нужен для ISOHYBRID.

Сделал тривиальный патч, тестирую...

-[ -z &quot;${EFI_BOOTLOADER:-}&quot; ] ||
+[ -z &quot;${EFI_BOOTLOADER:-}&quot; -a -z &quot;${ISOHYBRID:-}&quot; ] ||</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136390</commentid>
    <comment_count>5</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-12-25 20:28:28 +0400</bug_when>
    <thetext>Проверил с use/isohybrid, use/efi и без них, УМВР:
http://git.altlinux.org/people/mike/packages/?p=mkimage.git;a=commitdiff;h=4bc5bb16590b8ac8d5ea9a6d0c6a56792b597dcc</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136800</commentid>
    <comment_count>6</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2013-01-14 01:52:33 +0400</bug_when>
    <thetext>Так как, принимаем?  А то у меня тут следующая кучка патчей зреет -- вычистка и обобщение mki-copy-efiboot :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136834</commentid>
    <comment_count>7</comment_count>
    <who name="Alexey Gladkov">legion</who>
    <bug_when>2013-01-15 00:43:24 +0400</bug_when>
    <thetext>0.2.6-alt1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>137034</commentid>
    <comment_count>8</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2013-01-20 18:18:24 +0400</bug_when>
    <thetext>Теперь снова порядок, извините за незамеченный вовремя слом.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158074</commentid>
    <comment_count>9</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2016-08-15 12:58:08 +0300</bug_when>
    <thetext>(В ответ на комментарий №1)
&gt; Дело в том, что &quot;ALT Linux &quot;&quot;/x86_64&quot; добавляет 17 символов [...]
Теперь в моде просто &quot;ALT&quot; (-6, т.е. +11).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>