Terry J. Reedy added the comment:
What I think:
1. Perhaps I should have noticed that
self.tk.call(_flatten((self._w, cmd)))):
has 3 '('s and 4 ')'s and looked at the previous line for the complete
expression.
2. Perhaps Python should switch os.sep ('\\') and os.altsep ('/') on Windows
and otherwise 'sanitize', as needed, all file names it gets from Windows, so it
always uses '/' internally as the path separator on Windows as well as *nix.
The current situation has been a constant headache. (Example: until patched
this year, patchcheck.py did not work completely on Windows.) Beyond the scope
of this issue.
3. Without waiting for 2. to happen, perhaps Idle should do so. Another example
of the \ problem: if one recursively searches c:/programs/python34/lib/idlelib,
the output window will put out entries with mixed usage:
c:/programs/python34/lib/idlelib\idle_test\test_rstrip.py:
This is confusing to read and not much useful when copied for pasting.
Also beyond the scope of this issue.
4. Without waiting for 3, and given that tk is (just sometimes?) cooking
strings as if they were literals, Idle should at least sanitize (\ to /)
filenames it sends to tk to avoid cooking altogether. Is tk also replacing the
2 char sequence \t with the tab char?
4a. I suspect the tk cooking behavior should be documented better than it is. I
was not aware of it.
5. Making the tkinter tests pass (when written correctly) is enough to justify
a patch. Better soon than just before release.
You did not directly say whether your patch fixes the Idle 0.py problem, but I
presume the change to _configure() is intended to. In any case, I will try to
test this on my system tomorrow.
----------
_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue19020>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
