Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-17 Thread Leonardo Sandoval
Ross/Raj, thanks for your comments. I will send a v2 patches, this time with the correct checks (invalid characters are those non-UTF8) and patch titles. On 08/13/2015 02:34 PM, Burton, Ross wrote: On 13 August 2015 at 20:21, Khem Raj raj.k...@gmail.com wrote: but check it still fine isnt

[OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread leonardo . sandoval . gonzalez
From: Leonardo Sandoval leonardo.sandoval.gonza...@linux.intel.com On package creation, handle exception when encoding non-ASCII characteres. [YOCTO #6693] Signed-off-by: Leonardo Sandoval leonardo.sandoval.gonza...@linux.intel.com --- meta/classes/package_deb.bbclass | 4 1 file changed,

Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread Khem Raj
On Thu, Aug 13, 2015 at 12:05 PM, Burton, Ross ross.bur...@intel.com wrote: Debian control files are defined to be UTF-8, so the use of an ASCII encoding method is wrong (https://www.debian.org/doc/debian-policy/ch-controlfields.html). (RPM appears to assume UTF-8 too) but check it still

Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread Burton, Ross
On 13 August 2015 at 20:21, Khem Raj raj.k...@gmail.com wrote: but check it still fine isnt it. Of course, gracefully handling encoding failures is still sensible. Ross -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org

Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread Burton, Ross
On 13 August 2015 at 20:31, Mark Hatle mark.ha...@windriver.com wrote: RPM isn't really utf-8.. it's more single 8-bit characters... UTF-8 (1 byte characters) work fine.. multibyte are not promised to work. If you need special encoding (more then 8-bit characters) then you should be using

Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread Burton, Ross
On 13 August 2015 at 09:58, leonardo.sandoval.gonza...@linux.intel.com wrote: On package creation, handle exception when encoding non-ASCII characteres. Debian control files are defined to be UTF-8, so the use of an ASCII encoding method is wrong (

Re: [OE-core] [PATCH 2/2] package_deb.bbclass: Handle exception when encoding non-ASCII characters

2015-08-13 Thread Mark Hatle
On 8/13/15 2:21 PM, Khem Raj wrote: On Thu, Aug 13, 2015 at 12:05 PM, Burton, Ross ross.bur...@intel.com wrote: Debian control files are defined to be UTF-8, so the use of an ASCII encoding method is wrong (https://www.debian.org/doc/debian-policy/ch-controlfields.html). (RPM appears to