[issue10552] Tools/unicode/gencodec.py error

Change by STINNER Victor : -- nosy: -vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue10552] Tools/unicode/gencodec.py error

Irit Katriel added the comment: I don't think Martin's patch has been applied. Is it needed? -- nosy: +iritkatriel ___ Python tracker ___ _

[issue10552] Tools/unicode/gencodec.py error

Martin Panter added the comment: Here is a new version of Kuchling’s patch. I restored some mapping files which do not give any errors (including the mac_turkish codec, which is actually documented), and removed both readme files. -- components: +Unicode nosy: +haypo, vadmium versions:

[issue10552] Tools/unicode/gencodec.py error

Changes by A.M. Kuchling : -- nosy: -akuchling ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.py

[issue10552] Tools/unicode/gencodec.py error

Changes by Alexander Belopolsky : -- nosy: +hynek, ned.deily, ronaldoussoren ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue10552] Tools/unicode/gencodec.py error

Changes by Alexander Belopolsky : -- assignee: belopolsky -> ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue10552] Tools/unicode/gencodec.py error

A.M. Kuchling added the comment: For the Mac issue, we could just delete the mapping files before processing them. I've attached a patch that modifies the Makefile. -- nosy: +akuchling Added file: http://bugs.python.org/file32565/10552-remove-apple-files.txt __

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: Committed in revision 86891. Keeping open to address Mac issue. -- assignee: -> belopolsky components: +Macintosh priority: normal -> low stage: commit review -> needs patch ___ Python tracker

[issue10552] Tools/unicode/gencodec.py error

Martin v. Löwis added the comment: > These changes don't affect gencodec.py, but it may be inconvenient to > run makeunicodedata.py and gencodec.py using different versions of > Python. As MAL explains: these are completely unrelated, independent tools, and gencodec isn't run more than once per

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: On Mon, Nov 29, 2010 at 1:38 PM, Marc-Andre Lemburg wrote: .. > Sure, we won't need that script anytime soon and if we do, we > can just as well use the Python2 version. That may not be true. I compared 2.7 and py3k versions and the later has some new f

[issue10552] Tools/unicode/gencodec.py error

Marc-Andre Lemburg added the comment: Alexander Belopolsky wrote: > > Alexander Belopolsky added the comment: > > On Mon, Nov 29, 2010 at 1:21 PM, Marc-Andre Lemburg > wrote: > .. >> BTW: You appear to have a comma appended to the constant, that doesn't >> belong there: >> >> +# Placeholder

[issue10552] Tools/unicode/gencodec.py error

Changes by Alexander Belopolsky : Removed file: http://bugs.python.org/file19843/issue10552a.diff ___ Python tracker ___ ___ Python-bugs-list

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: On Mon, Nov 29, 2010 at 1:21 PM, Marc-Andre Lemburg wrote: .. > BTW: You appear to have a comma appended to the constant, that doesn't > belong there: > > +# Placeholder for a missing codepoint > +MISSING_CODE = -1, > + > > Perhaps that's causing the seco

[issue10552] Tools/unicode/gencodec.py error

Marc-Andre Lemburg added the comment: gencodec.py is only rarely used, namely when adding new codecs based on Unicode mapping files. It is not run regularly on the files from ftp.unicode.org and only updated on demand. AFAIK, it was last used on Python2 and never on Python3, hence the errors y

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: Martin, I believe you were the last to update the unicode database. (See r85371.) Did you use python2.x to generate it or you have your own private copy of these tools? I noticed that genwincodecs.bat refers to c:\python26\python in 2.7 branch and c:

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: Please ignore Makefile changes in the patch. -- ___ Python tracker ___ ___ Python-bugs-list m

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: Attached patch uses MISSING_CODE as Mark suggested. There are still errors apparently because parsecodes() may return either an int or a tuple. I think only mac encodings are affected, so I would like to commit the current patch before tackling this i

[issue10552] Tools/unicode/gencodec.py error

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue10552] Tools/unicode/gencodec.py error

Marc-Andre Lemburg added the comment: Alexander Belopolsky wrote: > > Alexander Belopolsky added the comment: > > Attached patch addresses the issue by using -1 instead of None for missing > codes. Comparison of generated encoding files to those in Lib/encodings > shows only whitespace cha

[issue10552] Tools/unicode/gencodec.py error

Alexander Belopolsky added the comment: Attached patch addresses the issue by using -1 instead of None for missing codes. Comparison of generated encoding files to those in Lib/encodings shows only whitespace changes except one which appears to be a change on the unicode.org side: diff -b

[issue10552] Tools/unicode/gencodec.py error

New submission from Alexander Belopolsky : $ ../../python.exe gencodec.py MAPPINGS/VENDORS/MISC/ build/ converting APL-ISO-IR-68.TXT to build/apl_iso_ir_68.py and build/apl_iso_ir_68.mapping converting ATARIST.TXT to build/atarist.py and build/atarist.mapping converting CP1006.TXT to build/cp100