[issue1616979] cp720 encoding map

[Abdulmonem]

Abdulmonem  added the comment:

As a user I experienced this bug. With  python 3.1, the interpreter 
terminate with fatal error:
"Py_Initialize: can't initialize sys standard streams
LookupError: unknown encoding: cp720"

I think, this can be replicated by changing the active code page in the 
cmd session, before invoking the interpreter. The following command will 
do so:
> chcp 720

without the patch python would crash after this command.
I think testing the patch after this command is sufficient.

--
nosy: +abu_mohammed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc  added the comment:

Instead of using another source of third-party files, I suggest to use the 
Windows 
functions to generate the mapping.
The attached patch contains a script, genwincodec.py, which uses 
MultiByteToWideChar 
and generates a codec file.

I use it like this:
.\PCBuild\python Tools\unicode\genwincodec.py 720 > Lib\encodings\cp720.py

The generated file is also in the patch.

--
keywords: +needs review -patch
nosy: +amaury.forgeotdarc
stage: test needed -> patch review
Added file: http://bugs.python.org/file14489/genwincodec-trunk.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Amaury Forgeot d'Arc]

Changes by Amaury Forgeot d'Arc :


--
keywords: +patch
Added file: http://bugs.python.org/file14490/genwincodec-py3k.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Martin v . Löwis]

Martin v. Löwis  added the comment:

Amaury: your approach sounds fine to me, please apply.

--
resolution:  -> accepted

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Martin v . Löwis]

Martin v. Löwis  added the comment:

Reconsidering, I'd like to ask for two changes:
- please record the command(s) used to generate tables on Windows
somewhere, in either Tools/unicode/Makefile, or a separate batch file.
- please arrange for the doc string of the generated file to identify
the source of the data base; in particular, make sure that the Windows
version on which this was run is identified. It might be that future
Windows versions change the mappings.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc  added the comment:

The codec file now starts with the comment:
"""Python Character Mapping Codec cp720 generated on Windows:
Vista 6.0.6002 SP2 Multiprocessor Free with the command:
  python Tools/unicode/genwincodec.py 720
"""

I also added a file Tools\unicode\genwincodecs.bat that currently only 
generates cp720.py.

Applied in r74000 (trunk) and r74003 & r74004 (py3k)

--
resolution: accepted -> fixed
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Alexander Belchenko]

Alexander Belchenko  added the comment:

As the author of original patch I want to note that it seems your merged
patch does not update the documentation (list of standard encodings).

Please, update the docs as well.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc  added the comment:

I think it is, see r74006 and
http://docs.python.org/dev/library/codecs.html#standard-encodings
(this is the doc for the future 2.7 version)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Alexander Belchenko]

Alexander Belchenko  added the comment:

OK, thanks.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1616979] cp720 encoding map

[Daniel Diniz]

Changes by Daniel Diniz :


--
components: +Unicode -None
stage:  -> test needed
type:  -> feature request
versions: +Python 2.7, Python 3.1

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com