<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>6693</bug_id>
          
          <creation_ts>2014-09-08 11:28:32 +0000</creation_ts>
          <short_desc>package_deb doesn&apos;t handle non-ASCII control files</short_desc>
          <delta_ts>2015-09-15 15:05:03 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>11</classification_id>
          <classification>Runtime</classification>
          <product>Package Management Issues</product>
          <component>Debian Packaging - DEB</component>
          <version>1.7</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.9 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ross Burton">ross.burton</reporter>
          <assigned_to name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</assigned_to>
          <cc>anibal.limon</cc>
    
    <cc>jose.a.lamego</cc>
    
    <cc>richard.purdie</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>45439</commentid>
    <comment_count>0</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2014-09-08 11:28:32 +0000</bug_when>
    <thetext>If eg a DESCRIPTION field contains non-ASCII, package_deb will throw an exception:

*** 0137:                         ctrlfile.write(u&apos;%s\n&apos; % unicode(textwrap.fill(description.strip(), width=74, initial_indent=&apos; &apos;, subsequent_indent=&apos; &apos;)))
     0138:
     0139:                else:
     0140:                     ctrlfile.write(unicode(c % tuple(pullData(fs, localdata))))
     0141:        except KeyError:

Exception: UnicodeDecodeError: &apos;ascii&apos; codec can&apos;t decode byte 0xc2 in position 45: ordinal not in range(128)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45536</commentid>
    <comment_count>1</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2014-09-11 14:38:00 +0000</bug_when>
    <thetext>Rather than package specific backends falling over this kind of issue we should sanity check the various fields in insane.bbclass for these kinds of issues. I know upper case and certain symbols can cause issues in some fields for example.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53100</commentid>
    <comment_count>2</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2015-08-13 17:08:33 +0000</bug_when>
    <thetext>A couple of patches sent to the community, so under reviewed now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53103</commentid>
    <comment_count>3</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2015-08-13 19:07:09 +0000</bug_when>
    <thetext>Considering I filed this last year I should have been more explicit.

Debian control files are defined to be in UTF-8, so we should just use a UTF-8 encoder when writing the files instead of ASCII.  RPM also appears to be UTF-8.  Not sure about opkg right now, I suspect it just assumes ASCII-compatible and hopes for the best...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53149</commentid>
    <comment_count>4</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2015-08-17 15:17:07 +0000</bug_when>
    <thetext>V2 patch sent to the ML. This time, invalid characters are considered those non-UTF8.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>54095</commentid>
    <comment_count>5</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2015-09-15 15:05:03 +0000</bug_when>
    <thetext>Patch merged on master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=1019d8c802d426e0692f3b5939fa34fb46ff002d</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>