Activity log for bug #1666829

Date Who What changed Old value New value Message
2017-02-22 09:05:12 RJVB bug added bug
2017-02-22 09:05:12 RJVB attachment added patch-qarte3-unicode_issues.diff https://bugs.launchpad.net/bugs/1666829/+attachment/4824212/+files/patch-qarte3-unicode_issues.diff
2017-02-22 09:07:41 RJVB description A priori there is nothing platform-specific in the qarte code, and indeed it works on Mac and MS Windows without significant patching. There is a difference in the way Python/Mac handles Unicode strings, though, that lead to runtime failure. Attached is a patch that handles the few remaining expressions where this is a blocking problem in basic usage. From the looks of it these are all cases where a filename is retrieved from the server that is UTF8-encoded. If there is no way to make the `open()` function treat its filename argument as a unicode string then there are probably other locations where equivalent operations are carried out (e.g. in the scheduled loader implementation). A priori there is nothing platform-specific in the qarte code, and indeed it works on Mac and MS Windows without significant patching. There is a difference in the way Python/Mac handles Unicode strings, though, that leads to runtime failure. Attached is a patch that handles the few remaining expressions where this is a blocking problem in basic usage. From the looks of it these are all cases where a filename is retrieved from the server that is UTF8-encoded. If there is no way to make the `open()` function treat its filename argument as a unicode string then there are probably other locations where equivalent operations are carried out (e.g. in the scheduled loader implementation). Alternatively it should be possible to override/replace the open function with one that calls codecs.open . Edit: the patch is safe to use on Linux.
2017-02-24 08:29:38 VinsS qarte: status New Incomplete
2017-02-27 12:52:23 RJVB attachment added patch-qarte3-unicode_issues-simplified.diff https://bugs.launchpad.net/qarte/+bug/1666829/+attachment/4827651/+files/patch-qarte3-unicode_issues-simplified.diff