Commit f1ca0b11 authored by Amaury Forgeot d'Arc's avatar Amaury Forgeot d'Arc

Issue 1342: Python could not start if installed in a directory

with non-ascii characters.

This is the simple fix, which uses the FileSystemEncoding.
Replacing all the char* with unicode strings is a major rewrite,
and needs more thinking.
parent c354c2e6
......@@ -12,6 +12,9 @@ What's new in Python 3.0b1?
Core and Builtins
-----------------
- Issue #1342: On windows, Python could not start when installed in a
directory with non-ascii characters.
- Implement PEP 3121: new module initialization and finalization API.
- Removed the already-defunct ``-t`` option.
......
......@@ -1364,19 +1364,26 @@ find_module(char *fullname, char *subname, PyObject *path, char *buf,
if (!v)
return NULL;
if (PyUnicode_Check(v)) {
v = _PyUnicode_AsDefaultEncodedString(v, NULL);
v = PyUnicode_AsEncodedString(v,
Py_FileSystemDefaultEncoding, NULL);
if (v == NULL)
return NULL;
}
if (!PyBytes_Check(v))
else if (!PyBytes_Check(v))
continue;
else
Py_INCREF(v);
base = PyBytes_AS_STRING(v);
size = PyBytes_GET_SIZE(v);
len = size;
if (len + 2 + namelen + MAXSUFFIXSIZE >= buflen) {
Py_DECREF(v);
continue; /* Too long */
}
strcpy(buf, base);
Py_DECREF(v);
if (strlen(buf) != len) {
continue; /* v contains '\0' */
}
......@@ -3155,8 +3162,8 @@ NullImporter_init(NullImporter *self, PyObject *args, PyObject *kwds)
if (!_PyArg_NoKeywords("NullImporter()", kwds))
return -1;
if (!PyArg_ParseTuple(args, "s:NullImporter",
&path))
if (!PyArg_ParseTuple(args, "es:NullImporter",
Py_FileSystemDefaultEncoding, &path))
return -1;
pathlen = strlen(path);
......
......@@ -695,7 +695,7 @@ initstdio(void)
PyObject *std = NULL;
int status = 0, fd;
PyObject * encoding_attr;
char *encoding, *errors;
char *encoding = NULL, *errors;
/* Hack to avoid a nasty recursion issue when Python is invoked
in verbose mode: pre-import the Latin-1 and UTF-8 codecs */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment