[1.0.1] the path to sikuli scripts can't contain non-roman characters

Bug #512476 reported by Tsung-Hsiang Chang
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
SikuliX
In Progress
Medium
RaiMan

Bug Description

confirmed on Windows, Mac.
It seems Jython doesn't treat unicode characters correctly.

Tags: filenames todo
Changed in sikuli:
importance: Undecided → High
status: New → Triaged
Revision history for this message
niknah (hankin0) wrote :

Here's a patch to pass the path via java instead of jython.

Revision history for this message
niknah (hankin0) wrote :

previous patch was bad, use this one.

Revision history for this message
Tsung-Hsiang Chang (vgod) wrote :

Passing the path via java is a good idea. I will incorporate your patch into the source tree.
Thanks!

Revision history for this message
Tsung-Hsiang Chang (vgod) wrote :

I just committed a fix for Mac (and Linux I think).
But this problem is more complicated than I thought.
Sikuli uses UTF-8 for its file path's encoding, but Windows doesn't. That is, we may need another encoding conversion in the C++ layer to fix this problem.

Changed in sikuli:
status: Triaged → In Progress
Revision history for this message
niknah (hankin0) wrote :

Saving scripts with the ide crashes because it uses glob.glob when saving a file, glob.glob uses os.listdir which didn't support unicode until the recent version...

http://bugs.jython.org/issue1520

Can't seem to do anything with unicode in windows xp, maybe I need a non-english version.

RaiMan (raimund-hocke)
Changed in sikuli:
importance: High → Medium
assignee: nobody → RaiMan (raimund-hocke)
milestone: none → x1.0
tags: added: filenames
RaiMan (raimund-hocke)
Changed in sikuli:
importance: Medium → High
RaiMan (raimund-hocke)
Changed in sikuli:
milestone: x1.0 → x1.1
Revision history for this message
WG (foxfox) wrote :

Also "ü" "ö" "ä" "ß"

RaiMan (raimund-hocke)
tags: added: todo
summary: - the path to sikuli scripts can't contain non-roman characters
+ [1.0.1] the path to sikuli scripts can't contain non-roman characters
RaiMan (raimund-hocke)
Changed in sikuli:
importance: High → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.