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

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

    <bug>
          <bug_id>27525</bug_id>
          
          <creation_ts>2012-07-06 18:55:52 +0400</creation_ts>
          <short_desc>/usr/bin/mtools uses iconv functions but mtools package does not require glibc-gconv-modules</short_desc>
          <delta_ts>2012-07-10 15:21:48 +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>mtools</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ivan Ovcherenko">asdus</reporter>
          <assigned_to name="Michael Shigorin">mike</assigned_to>
          <cc>antohami</cc>
    
    <cc>mike</cc>
    
    <cc>zerg</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>132178</commentid>
    <comment_count>0</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-06 18:55:52 +0400</bug_when>
    <thetext>Собирается и так, но при работе, в частности при использовании из syslinux-а ругается на невозможность перекодировки имени файла, лечится установкой iconv.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132179</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-07-06 21:35:39 +0400</bug_when>
    <thetext>Странно, сколько пользуюсь -- не натыкался.  Можно пример?

(добавить-то зависимость можно, но это ещё и ~5.5M glibc-gconv-modules сразу)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132190</commentid>
    <comment_count>2</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-09 10:29:14 +0400</bug_when>
    <thetext>&gt; Странно, сколько пользуюсь -- не натыкался.  Можно пример?
в хешерном chroot-е, без установленного iconv

# rpm -q syslinux mtools iconv
syslinux-4.04-alt3
mtools-4.0.17-alt2
warning: package iconv is not installed
# dd if=/dev/zero of=/.image/test.img bs=512 count=262144
262144+0 records in
262144+0 records out
134217728 bytes (134 MB) copied, 1.4348 s, 93.5 MB/s
# parted /.image/test.img mklabel msdos
# parted /.image/test.img unit b mkpartfs primary fat16 1048576 134217727
WARNING: you are attempting to use parted to operate on (mkpartfs) a file system.
parted&apos;s file system manipulation code is not as robust as what you&apos;ll find in
dedicated, file-system-specific packages like e2fsprogs.  We recommend
you use parted only to manipulate partition tables, whenever possible.
Support for performing most operations on most types of file systems
will be removed in an upcoming release.
# parted /.image/test.img toggle 1 boot
# syslinux --install --stupid --offset 16384 /.image/test.img
Error converting to codepage 850 Invalid argument
Cannot initialize &apos;S:&apos;
Bad target s:/ldlinux.sys
syslinux: failed to create ldlinux.sys
# 

# rpm -q syslinux mtools iconv
syslinux-4.04-alt3
mtools-4.0.17-alt2
iconv-2.11.3-alt8
# dd if=/dev/zero of=/.image/test.img bs=512 count=262144
262144+0 records in
262144+0 records out
134217728 bytes (134 MB) copied, 1.14089 s, 118 MB/s
# parted /.image/test.img mklabel msdos
# parted /.image/test.img unit b mkpartfs primary fat16 1048576 134217727
WARNING: you are attempting to use parted to operate on (mkpartfs) a file system.
parted&apos;s file system manipulation code is not as robust as what you&apos;ll find in
dedicated, file-system-specific packages like e2fsprogs.  We recommend
you use parted only to manipulate partition tables, whenever possible.
Support for performing most operations on most types of file systems
will be removed in an upcoming release.
# parted /.image/test.img toggle 1 boot
# syslinux --install --stupid --offset 16384 /.image/test.img
#</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132198</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-07-09 20:04:22 +0400</bug_when>
    <thetext>А это точно про mtools и не syslinux?  Возьмите образ, подготовленный в чруте с iconv, скопируйте его в чрут без iconv и mtools, но с syslinux, и проверьте.

Исходники mtools содержат #include &lt;iconv.h&gt;, но не exec*().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132206</commentid>
    <comment_count>4</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-10 10:40:14 +0400</bug_when>
    <thetext>Прежде чем написать - отследил, где вылетает...

Это вызовы из syslinux-а (syslinux/mtools/syslinux.c):
status = system(&quot;mattrib -h -r -s s:/ldlinux.sys 2&gt;/dev/null&quot;);
// ... skipped ...
mtp = popen(&quot;mcopy -D o -D O -o - s:/ldlinux.sys&quot;, &quot;w&quot;);

А это кусок, где вылетает mtools (mtools/charsetConv.c):
from = iconv_open(wcharCp, dosCp); // В первом вхождении</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132208</commentid>
    <comment_count>5</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-07-10 11:34:35 +0400</bug_when>
    <thetext>(In reply to comment #4)
&gt; from = iconv_open(wcharCp, dosCp);
Спасибо; если удобно -- добавьте в чрут только glibc-gconv-modules, сам бинарник iconv там явно не нужен.  И зафиксим уже :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132211</commentid>
    <comment_count>6</comment_count>
    <who name="Sergey V Turchin">zerg</who>
    <bug_when>2012-07-10 11:44:06 +0400</bug_when>
    <thetext>(В ответ на комментарий №4)
&gt; Это вызовы из syslinux-а (syslinux/mtools/syslinux.c):
&gt; system(&quot;mattrib
Т.к. там нет проверки на наличие, поставлю зависимость в syslinux</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132212</commentid>
    <comment_count>7</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-10 11:45:42 +0400</bug_when>
    <thetext>(В ответ на комментарий №5)
&gt; (In reply to comment #4)
&gt; &gt; from = iconv_open(wcharCp, dosCp);
&gt; Спасибо; если удобно -- добавьте в чрут только glibc-gconv-modules, сам
&gt; бинарник iconv там явно не нужен.  И зафиксим уже :)

Все хорошо собралось с glibc-gconv-modules вместо iconv.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132213</commentid>
    <comment_count>8</comment_count>
    <who name="Sergey V Turchin">zerg</who>
    <bug_when>2012-07-10 11:47:08 +0400</bug_when>
    <thetext>(В ответ на комментарий №6)
&gt; Т.к. там нет проверки на наличие, поставлю зависимость в syslinux
Хотя, она стоит уже.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132214</commentid>
    <comment_count>9</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-10 11:50:37 +0400</bug_when>
    <thetext>(В ответ на комментарий №8)
&gt; (В ответ на комментарий №6)
&gt; &gt; Т.к. там нет проверки на наличие, поставлю зависимость в syslinux
&gt; Хотя, она стоит уже.

Разве? Не вижу...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132216</commentid>
    <comment_count>10</comment_count>
    <who name="Sergey V Turchin">zerg</who>
    <bug_when>2012-07-10 12:12:02 +0400</bug_when>
    <thetext>(В ответ на комментарий №9)
&gt; &gt; Хотя, она стоит уже.
&gt; Разве? Не вижу...
rpm -q --requires syslinux</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132217</commentid>
    <comment_count>11</comment_count>
    <who name="Sergey V Turchin">zerg</who>
    <bug_when>2012-07-10 12:14:33 +0400</bug_when>
    <thetext>(В ответ на комментарий №10)
&gt; (В ответ на комментарий №9)
&gt; &gt; &gt; Хотя, она стоит уже.
&gt; &gt; Разве? Не вижу...
&gt; rpm -q --requires syslinux
Я про mtools. А вызовов выполения iconv нет в syslinux.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132223</commentid>
    <comment_count>12</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2012-07-10 14:19:57 +0400</bug_when>
    <thetext>$ rpmelfsym Sisyphus/files/x86_64/RPMS/mtools-4.0.17-alt2.x86_64.rpm |grep iconv
/usr/bin/mtools	U	iconv
/usr/bin/mtools	U	iconv_close
/usr/bin/mtools	U	iconv_open</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132225</commentid>
    <comment_count>13</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-07-10 14:23:38 +0400</bug_when>
    <thetext>(In reply to comment #8)
&gt; &gt; Т.к. там нет проверки на наличие, поставлю зависимость в syslinux
&gt; Хотя, она стоит уже.
$ rpm -qR syslinux-4.04-alt3 mtools-4.0.17-alt2 | grep conv
$ _

Отправил 4.0.17-alt3.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132226</commentid>
    <comment_count>14</comment_count>
    <who name="Ivan Ovcherenko">asdus</who>
    <bug_when>2012-07-10 14:25:36 +0400</bug_when>
    <thetext>По поводу названия бага и его сути, скорее &quot;mtools require glibc-gconv-modules&quot; получается...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132227</commentid>
    <comment_count>15</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2012-07-10 14:32:31 +0400</bug_when>
    <thetext>mtools-4.0.17-alt3 -&gt; sisyphus:

* Tue Jul 10 2012 Michael Shigorin &lt;mike@altlinux&gt; 4.0.17-alt3
- added Requires: glibc-gconv-modules (closes: #27525)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132231</commentid>
    <comment_count>16</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-07-10 15:21:23 +0400</bug_when>
    <thetext>Флэшмобим? :]</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>