[issue10617] Collections ABCs can’t be linked to

[Raymond Hettinger]

Raymond Hettinger  added the comment:

This looks fine, but it may be better to put the tables first and details 
second.

--

___
Python tracker 

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

[issue11060] distutils2 sdist does not complain about version that is not PEP 386 compliant

[yeswanth]

yeswanth  added the comment:

Added some test cases for it .

--
keywords: +patch
Added file: http://bugs.python.org/file21319/patch1.diff

___
Python tracker 

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

[issue11060] distutils2 sdist does not complain about version that is not PEP 386 compliant

[yeswanth]

Changes by yeswanth :


Removed file: http://bugs.python.org/file21008/help.txt

___
Python tracker 

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

[issue10617] Collections ABCs can’t be linked to

[Ezio Melotti]

Ezio Melotti  added the comment:

Here's a patch that adds the list of classes with a minimal description before 
the table.
I grouped a few classes to save some space. Container, Hashable, Sized, 
Callable could be grouped too.
I also added links to the methods and to glossary terms (the `set` term is 
missing from the glossary though).
The methods listed in the table can be turned to links too.
(The :meth:`__len__ <__len__>` markup is to hide the () after the __method__.)

--
assignee: georg.brandl -> ezio.melotti
keywords: +patch
stage: needs patch -> patch review
Added file: http://bugs.python.org/file21318/issue10617.diff

___
Python tracker 

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

[issue9584] Allow curly brace expansion

[Mathieu Bridon]

Mathieu Bridon  added the comment:

> "I removed the unused import (mostly as a simple test of mercurial, it's my 
> first commit there)."

Does it mean that Python development is not being done in SVN, as the 
documentations state it?

My patches have all been based on the SVN py3k branch, please tell me if I must 
base them on something else instead.

--

___
Python tracker 

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

[issue3080] Full unicode import system

[Andrew Svetlov]

Andrew Svetlov  added the comment:

As I see Victor has dropped OS/2 support from Python/import.c
Perhaps file Python/dynload_os2.c should be removed also.
Not sure about other dynload_* files.

--

___
Python tracker 

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

[issue8650] zlibmodule.c isn't 64-bit clean

[STINNER Victor]

STINNER Victor  added the comment:

> This only addresses the compress() and decompress() functions, 
> but e.g. crc32() and adler32() are also touched by this issue.

crc32() and adler32() were fixed by #10276.

PyZlib_objcompress() and PyZlib_objdecompress() should be fixed.

--

___
Python tracker 

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

[issue8420] Objects/setobject.c contains unsafe code

[Éric Araujo]

Éric Araujo  added the comment:

I was able to reproduce the crash in 2.7 and fix it with the patch.  I only had 
to remove a cast to long to make the patch apply.  I also changed tabs to 
spaces in two lines.

Please review and assign back to me for commit. :)

--
assignee: eric.araujo -> rhettinger
stage: needs patch -> commit review
Added file: http://bugs.python.org/file21317/fix-set-crashers-2.7.diff

___
Python tracker 

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

[issue8651] "s#" and friends can silently truncate buffer length

[STINNER Victor]

STINNER Victor  added the comment:

Ok, I backported the fix to 3.1.

Reopen the issue if you would like a port to 2.7 (I am too lazy to do it).

--
resolution:  -> 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

[issue11371] Localization of error messages in getopt

[Éric Araujo]

Éric Araujo  added the comment:

Done, thanks!

--
resolution: accepted -> fixed
stage: commit review -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue8651] "s#" and friends can silently truncate buffer length

[Roundup Robot]

Roundup Robot  added the comment:

New changeset d9633064458c by Victor Stinner in branch '3.1':
Issue #8651: PyArg_Parse*() functions raise an OverflowError if the file
http://hg.python.org/cpython/rev/d9633064458c

--
nosy: +python-dev

___
Python tracker 

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

[issue3080] Full unicode import system

[Roundup Robot]

Roundup Robot  added the comment:

New changeset ee4e780a6b7a by Éric Araujo in branch 'default':
Fix a typo (see #3080)
http://hg.python.org/cpython/rev/ee4e780a6b7a

--

___
Python tracker 

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

[issue11371] Localization of error messages in getopt

[Roundup Robot]

Roundup Robot  added the comment:

New changeset d3e46930ffe9 by Éric Araujo in branch 'default':
Mark getopt error messages as localizable (fixes #11371).
http://hg.python.org/cpython/rev/d3e46930ffe9

--
nosy: +python-dev

___
Python tracker 

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

[issue6501] Fatal error on startup with invalid PYTHONIOENCODING

[Éric Araujo]

Changes by Éric Araujo :


--
nosy: +haypo

___
Python tracker 

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

[issue5537] LWPCookieJar cannot handle cookies with expirations of 2038 or greater on 32-bit platforms

[STINNER Victor]

Changes by STINNER Victor :


--
resolution:  -> 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

[issue5537] LWPCookieJar cannot handle cookies with expirations of 2038 or greater on 32-bit platforms

[STINNER Victor]

STINNER Victor  added the comment:

> While it is unlikely that a purely numeric format such as "%Y-%m-%d
> %H:%M:%S" will be locale dependent, it is possible that some pre-C99
> systems would format dates using exotic digits is some locales.

Ok, I rewrote my patch to avoid strftime(). It should now be fixed.

FYI datetime.fromtimestamp() converts the timestamp to a double, which has a 
precision of 53 bits (no precision loss for year < 285,422,890 and so it's 
enough for year 2038).

--

___
Python tracker 

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

[issue5537] LWPCookieJar cannot handle cookies with expirations of 2038 or greater on 32-bit platforms

[Roundup Robot]

Roundup Robot  added the comment:

New changeset b15f60f9e256 by Victor Stinner in branch '3.1':
Issue #5537: Fix time2isoz() and time2netscape() functions of httplib.cookiejar
http://hg.python.org/cpython/rev/b15f60f9e256

--
nosy: +python-dev

___
Python tracker 

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

[issue11599] Useless error message when distutils fails compiling

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> It appears a choice has been made to print only the first argument in
> the error messages; attached patch changes that, please test.

Well, I don't have the failing setup at hand anymore, but the patch
looks reasonable to me. Thank you!

--

___
Python tracker 

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

[issue11599] Useless error message when distutils fails compiling

[Éric Araujo]

Éric Araujo  added the comment:

It appears a choice has been made to print only the first argument in the error 
messages; attached patch changes that, please test.

--
components: +Distutils2
keywords: +patch
nosy: +alexis
versions: +3rd party
Added file: http://bugs.python.org/file21316/fix-compile-error-messages.diff

___
Python tracker 

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

[issue5977] distutils build_ext.get_outputs returns wrong result

[Éric Araujo]

Changes by Éric Araujo :


--
components: +Distutils2
nosy: +alexis, eric.araujo
stage:  -> patch review
title: distutils build_ext.get_outputs returns wrong result (patch) -> 
distutils build_ext.get_outputs returns wrong result
type: crash -> behavior
versions: +3rd party, Python 3.2, Python 3.3 -Python 2.6, Python 3.0

___
Python tracker 

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

[issue10883] urllib: socket is not closed explicitly

[Nadeem Vawda]

Nadeem Vawda  added the comment:

> I saw the partial fix suggested by the patch, but for some reason I
> did not see ResourceWarning being shutup.

Do you mean that you aren't getting ResourceWarnings in the first place?
Or that you are getting warnings, and the partial fix isn't getting rid
of any of them?

Note: it doesn't get rid of *all* of the warnings for test_urllib2net;
only some of them.

About the remaining warnings, it seems that FTPHandler.ftp_open() creates
an ftpwrapper object to connect to the server, and then the ftpwrapper
doesn't get closed anywhere. It has to stay open until the caller has
finished reading from the connection, so finding the right place to close
it might be a bit tricky.

--

___
Python tracker 

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

[issue11230] "Full unicode import system" not in 3.2

[STINNER Victor]

STINNER Victor  added the comment:

>> Victor asked "Which web page should updated/fixed?"
> Answer: The Python 3.2 download page.

Sorry, but I don't see which page tells that Python 3.2 has a full Unicode 
support for import. In http://www.python.org/download/releases/3.2/, I can read 
"countless fixes regarding bytes/string issues; among them full support for a 
bytes environment (filenames, environment variables)".

"full support for a bytes environment" means that Python 3.2 has been fixed on 
UNIX to support undecodable filenames, but not that Python 3.2 supports 
unencodable filenames on Windows.

Can you propose a sentence which is more clear about bytes/Unicode?

Python 3.3 will have a full Unicode support for modules: issue #3080 is already 
fixed, and I think that #11619 can be fixed (maybe not easily).

--

___
Python tracker 

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

[issue8730] Spurious test failure in distutils

[Éric Araujo]

Éric Araujo  added the comment:

Ronald, can you tell what would be the check to put in skipIf?

--
versions: +Python 3.3

___
Python tracker 

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

[issue9181] Solaris extension building does not work with 64 bit python

[Éric Araujo]

Éric Araujo  added the comment:

Charles, can you still reproduce your bug now that #9437 has been fixed?

--

___
Python tracker 

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

[issue5863] bz2.BZ2File should accept other file-like objects. (issue4274045)

[Nadeem Vawda]

Nadeem Vawda  added the comment:

Corresponding patch for the module docs.

--
Added file: http://bugs.python.org/file21315/bz2-v4-doc.diff

___
Python tracker 

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

[issue9302] distutils.core.Extension: list parameters documented as strings

[Éric Araujo]

Éric Araujo  added the comment:

Thanks for the catch.  Are there no similar problems in the rest of the 
distutils doc, or did you search in one file only?

--
assignee: docs@python -> eric.araujo
components: +Distutils, Distutils2
nosy: +alexis, eric.araujo
stage:  -> patch review
title: Distutils document problem? -> distutils.core.Extension: list parameters 
documented as strings
versions: +3rd party, Python 2.7, Python 3.1, Python 3.3

___
Python tracker 

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

[issue11235] Source files with date modifed in 2106 cause OverflowError

[STINNER Victor]

STINNER Victor  added the comment:

By the way, a simpler fix would be just to not fail if the .pyc file cannot be 
created (but emit a warning / write an error to stderr).

--

___
Python tracker 

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

[issue5863] bz2.BZ2File should accept other file-like objects. (issue4274045)

[Nadeem Vawda]

Nadeem Vawda  added the comment:

Here is an updated patch, incorporating the feedback from your review.

The new patch no longer checks for errors in bz2CompressEnd()/bz2DecompressEnd()
in the dealloc functions for BZ2Compressor/BZ2Decompressor. I found that calling
PyErr_WriteUnraisable() results in spurious error messages if an exception is
raised by the init function, and in any case, the output would not be of much
use if a genuine error were to occur.

The patch adds implementations of most of the io.BufferedIOBase methods
(everything except detach(), read1() and truncate()), and includes unit tests
for fileno() and readinto().

--
Added file: http://bugs.python.org/file21314/bz2-v4.diff

___
Python tracker 

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

[issue11188] test_time error on AIX

[STINNER Victor]

STINNER Victor  added the comment:

strftime_aix.patch is a little bit too strict: it looks like strftime() 
supports large year values (year > ). We may only raise an error if the 
year is smaller than 1.

--

___
Python tracker 

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

[issue9306] distutils: raise informative error message when cmd_class is None

[Éric Araujo]

Éric Araujo  added the comment:

I’m inclined to reject the bug: 
http://docs.python.org/distutils/apiref#distutils.core.setup clearly says that 
the argument should be a dictionary.  The code in pyreadline should set it to 
an empty dict instead of None.  While it can be argued that providing clearly 
helpful messages is a service for our users, especially given that people 
writing setup scripts may be beginners, it seems to me that adding type 
checking would not be the Python way.  Raymond, what do you think?

--
components: +Distutils2 -Distutils
nosy: +alexis, rhettinger
versions: +3rd party -Python 3.2

___
Python tracker 

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

[issue10864] time.strftime("%Y"): limitation of 4 digits on OpenIndiana (Solaris)

[Roundup Robot]

Roundup Robot  added the comment:

New changeset d56eb817b181 by Victor Stinner in branch '3.2':
Issue #10864 has been fixed: remove the workaround
http://hg.python.org/cpython/rev/d56eb817b181

--
nosy: +python-dev

___
Python tracker 

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

[issue11619] On Windows, don't encode filenames in the import machinery

[Terry J. Reedy]

Changes by Terry J. Reedy :


--
nosy: +terry.reedy

___
Python tracker 

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

[issue11210] PyErr_SetFromWindowsErrWithFilenameObject() doesn't exist: remove it from pyerrors.h

[Roundup Robot]

Roundup Robot  added the comment:

New changeset c0829c4e19cf by Victor Stinner in branch 'default':
Closes #11210: Remove PyErr_SetFromWindowsErrWithFilenameObject() of pyerrors.h
http://hg.python.org/cpython/rev/c0829c4e19cf

--
nosy: +python-dev
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue10828] Python 3 doesn't support non-ASCII module names with a locale encoding different than UTF-8

[Terry J. Reedy]

Terry J. Reedy  added the comment:

As Victor noted, this issue is essentially a duplicate of #3080 (and others) 
and now #11619 and needs no independent action apart from the latter. Since the 
discussion with ingemar seems finished, I am now closing.

--
resolution:  -> duplicate
status: open -> closed
superseder:  -> On Windows, don't encode filenames in the import machinery

___
Python tracker 

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

[issue7311] Bug on regexp of HTMLParser

[Éric Araujo]

Changes by Éric Araujo :


--
nosy: +eric.araujo
versions: +Python 3.1, Python 3.2, Python 3.3 -Python 2.6

___
Python tracker 

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

[issue5474] distutils produces invalid RPM packages of prerelease python packages

[Éric Araujo]

Éric Araujo  added the comment:

With the new Mercurial repositories and adapted workflow, patches should apply 
to a clone of the 3.1 branch.

If you could also remove outdated or superseded patches from this issue (or 
give their names if you can’t remove them), it would help reviewing.  Thanks in 
advance.

--
nosy: +eric.araujo
versions: +Python 3.2, Python 3.3

___
Python tracker 

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

[issue11342] ResourceWarning: unclosed file <_io.TextIOWrapper

[Éric Araujo]

Éric Araujo  added the comment:

Any message will automatically change pending to open, don’t worry.

--
resolution:  -> works for me
status: open -> pending

___
Python tracker 

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

[issue9084] vimrc: use matchall() instead of ":match" to allow multiple matches

[Éric Araujo]

Changes by Éric Araujo :


--
assignee: benjamin.peterson -> brett.cannon
keywords: +patch -gsoc, needs review
nosy:  -benjamin.peterson

___
Python tracker 

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

[issue11342] ResourceWarning: unclosed file <_io.TextIOWrapper

[Nadeem Vawda]

Nadeem Vawda  added the comment:

Sorry about accidentally changing the issue status to "open". It seems that I 
don't have the requisite privileges to change it back to "pending"...

--

___
Python tracker 

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

[issue9084] vimrc: use matchall() instead of ":match" to allow multiple matches

[Éric Araujo]

Changes by Éric Araujo :


--
assignee: brett.cannon -> benjamin.peterson
dependencies: +Usefulness of the Misc/Vim/ files?
keywords: +gsoc -patch
nosy: +benjamin.peterson, eric.araujo

___
Python tracker 

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

[issue9309] Add environment variable complementing command line option -no-user-cfg

[Éric Araujo]

Éric Araujo  added the comment:

I’m not sure this is a distutils or virtualenv bug.  I will defer to Tarek’s 
decision.

--

___
Python tracker 

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

[issue11620] winsound.PlaySound() with SND_MEMORY should accept bytes instead of strings

[Tom Felker]

New submission from Tom Felker :

PlaySound supposedly lets you play a .WAV file whose contents are stored in a 
string, by passing the string and flags including winsound.SND_MEMORY.  I'm 
trying to use BytesIO object and the wave module to make a file in-memory, and 
pass this to winsound.  It's a TypeError if I pass the resultant bytes object 
directly, and if I pass a string object, I get

TypeError: must be str without null character or None, not str

Since wav files can contain zeros, for the feature to work at all, the C code 
would need to detect the SND_MEMORY flag and interpret the first argument as a 
bytes object, instead of a null-terminated string, in that case.

--
components: Library (Lib)
messages: 131583
nosy: Tom.Felker
priority: normal
severity: normal
status: open
title: winsound.PlaySound() with SND_MEMORY should accept bytes instead of 
strings
versions: Python 3.1

___
Python tracker 

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

[issue9750] sqlite3 iterdump fails on column with reserved name

[Éric Araujo]

Éric Araujo  added the comment:

> I have very limited time to use on this.
This is a perfectly reasonable stance, and we’re grateful for the work you’re 
doing.  When you say that you don’t have time, someone else will take up the 
patch and update it to address reviews, and you will get credited.

a) Thanks for the explanation.  This would be a useful comment in the source.

b) It’s not uncommon to make cosmetic changes in the same commit as a bugfix.  
Readability is valued.

--

___
Python tracker 

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

[issue11342] ResourceWarning: unclosed file <_io.TextIOWrapper

[Nadeem Vawda]

Nadeem Vawda  added the comment:

The first ResourceWarning looks like the test_cgi leak from issue10512, which 
was fixed in changeset ae186b3b2c55.

--
nosy: +nvawda
status: pending -> open

___
Python tracker 

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

[issue11591] "python -S" should be robust against e.g. "from site import addsitedir"

[Éric Araujo]

Éric Araujo  added the comment:

I think this requires a note in NEWS but also in Doc/library/site.rst.  Carl, 
would you like to make a new changeset with that edition in your repo?  (It 
will also provide a test for the Create Patch button after.)

--

___
Python tracker 

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

[issue9100] test_sysconfig fails (test_user_similar)

[Éric Araujo]

Éric Araujo  added the comment:

Would you like to turn your diagnostic into a patch?

--
versions: +Python 3.1, Python 3.2, Python 3.3

___
Python tracker 

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

[issue5800] make wsgiref.headers.Headers accept empty constructor

[Éric Araujo]

Éric Araujo  added the comment:

I think the doc should say something like “default value is an empty list”, for 
maximum clarity.

--
versions: +Python 3.3 -Python 3.2

___
Python tracker 

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

[issue10828] Python 3 doesn't support non-ASCII module names with a locale encoding different than UTF-8

[STINNER Victor]

STINNER Victor  added the comment:

I closed #3080: Python 3.3 is now able to handle non-ASCII characters in module 
names and paths. But it is only able to handle non-ASCII characters encodable 
to the ANSI code page. To support all characters, I opened the issue #11619 
(see also #10785).

--

___
Python tracker 

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

[issue11287] Add context manager support to dbm modules

[Éric Araujo]

Éric Araujo  added the comment:

Reviewed.

--

___
Python tracker 

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

[issue3080] Full unicode import system

[STINNER Victor]

STINNER Victor  added the comment:

Ok, I think that the most important part is now implemented in Python 3.3: use 
Unicode for module names and paths in the import machinery. Remaing parts are 
specific to Windows, and so I opened a new issue: #11619. Let's close this 3 
years old issue.

--
resolution:  -> 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

[issue11619] On Windows, don't encode filenames in the import machinery

[STINNER Victor]

New submission from STINNER Victor :

With #3080, Python 3.3 does now manipulate module paths and names as Unicode in 
the import machinery. But in 3 remaining places, it does encode filenames (to 
the ANSI code page):


a) _PyImport_LoadDynamicModule()

It should pass directly the PyObject* (instead of a char*) to 
_PyImport_GetDynLoadFunc(), but only on Windows (we may change the function 
name for Windows). _PyImport_GetDynLoadFunc() of dynload_win.c has to be 
patched to use the Unicode API (eg. LoadLibraryEx => LoadLibraryExW).


b) write_compiled_module()

The problem is to implement open_exclusive() for Windows using Unicode. 
open_exclusive() uses open() on Windows, but open() expects the filename as a 
byte string. We may use _Py_fopen() (_wfopen), but this function doesn't have 
an option to open the file in exclusive mode (O_EXCL flag). GNU has an 
extension: "x" flag in the file mode, but Windows doesn't support it.

The file is passed to marshal functions like PyMarshal_WriteLongToFile(), and 
so the file have to be a FILE*.


c) parse_source_module()

=> covered by the issue #10785.

--
components: Interpreter Core, Unicode, Windows
messages: 131574
nosy: amaury.forgeotdarc, haypo, pitrou
priority: normal
severity: normal
status: open
title: On Windows, don't encode filenames in the import machinery
versions: Python 3.3

___
Python tracker 

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

[issue11421] Subversion keywords missing on 2.5 checkout

[Martin v . Löwis]

Martin v. Löwis  added the comment:

If someone can propose a patch that has zero effect on a subversion checkout or 
export but fixes the issue with a hg checkout, I'd be fine adding it.

OTOH, I fail to see why this issue needs fixing.

--

___
Python tracker 

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

[issue3080] Full unicode import system

[STINNER Victor]

STINNER Victor  added the comment:

> c) _PyImport_LoadDynamicModule() encodes the filename for
> _PyImport_GetDynLoadFunc(). The prototype should be changed,
> but only on Windows, to accept a filename as a Unicode string.

Hum, the difficult part is to use Unicode in _PyImport_GetDynLoadFunc() for:

hDLL = LoadLibraryEx(pathname, NULL, LOAD_WITH_ALTERED_SEARCH_PATH);

There is a LoadLibraryW() function, but it doesn't have a flag argument. And I 
suppose that the LOAD_WITH_ALTERED_SEARCH_PATH option is important.

--

___
Python tracker 

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

[issue6087] distutils.sysconfig.get_python_lib gives surprising result when used with a Python build

[Éric Araujo]

Changes by Éric Araujo :


--
keywords: +easy, patch -buildbot, needs review
nosy: +alexis
stage:  -> commit review

___
Python tracker 

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

[issue3080] Full unicode import system

[STINNER Victor]

STINNER Victor  added the comment:

Ok. Python 3.3 does now support non-ASCII characters in module paths and names 
on Windows, but only characters encodable to the ANSI code page. To support the 
full Unicode range, we should remove all calls to PyUnicode_EncodeFSDefault() 
on Windows:

  a) parse_source_module() has to encode the filename because the parser has no 
function expecting a filename as a Python object. It uses currently 
PyParser_ASTFromFile().

  b) write_compiled_module() encodes the filename to call open_exclusive(). I 
don't know how to implement open_exclusive() for Windows using Unicode 
filename: open() expects the filename as a byte string. Can we use _Py_fopen() 
(_wfopen)? Or do you need the O_EXCL flag?

  c) _PyImport_LoadDynamicModule() encodes the filename for 
_PyImport_GetDynLoadFunc(). The prototype should be changed, but only on 
Windows, to accept a filename as a Unicode string.

Issue #10785 is the right fix to (a). When #10785 will be fixed, it will be 
easier to fix #9319 crash.

--

___
Python tracker 

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

[issue6087] distutils.sysconfig.get_python_lib gives surprising result when used with a Python build

[Éric Araujo]

Éric Araujo  added the comment:

The patches look good, thank you.  Do you have a command, script or test that 
would print the values, to make sure they’re right?

--
assignee: tarek -> eric.araujo
keywords: +buildbot, needs review -easy, patch
nosy:  -alexis
versions: +Python 3.3

___
Python tracker 

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

[issue9523] Improve dbm modules

[Éric Araujo]

Éric Araujo  added the comment:

I think the patch will not be suitable for 3.1 and 3.2, so there should be a 
doc patch to mention the limitations of the dbm API (keys() returning a list 
and all that).

--

___
Python tracker 

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

[issue11617] Sporadic failure in test_httpservers

[Roundup Robot]

Roundup Robot  added the comment:

New changeset fa20590d4694 by Antoine Pitrou in branch '3.2':
Try to strengthen test_httpservers (issue #11617)
http://hg.python.org/cpython/rev/fa20590d4694

New changeset 34db881edb4d by Antoine Pitrou in branch 'default':
Try to strengthen test_httpservers (issue #11617)
http://hg.python.org/cpython/rev/34db881edb4d

--
nosy: +python-dev

___
Python tracker 

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

[issue11421] Subversion keywords missing on 2.5 checkout

[Éric Araujo]

Éric Araujo  added the comment:

What’s the proper way to fix this?  Mention in the devguide that 2.5 and 2.6 
can’t be built from Mercurial, or commit some text in the place of the keywords 
in Mercurial?

--
nosy: +eric.araujo
versions: +Python 2.6 -Python 2.5

___
Python tracker 

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

[issue3722] print followed by exception eats print with doctest

[Terry J. Reedy]

Terry J. Reedy  added the comment:

You misunderstood my last response. The first paragraph *dismisses* the case of 
temporary print (raised by Charles-Axle) as out of scope for doctests and hence 
this issue. The next ones addresses *your* case of code *permanently* intended 
to print and raise and gives a workable solution to your example.

After failing to find a simpler example that works with 3.x*, I am now thinking 
of the following doc fix.

1. Add 'directly' before 'supported' in Tim's first sentence: "Examples 
containing both expected output and an exception are not supported." 

2. After the next sentence of explanation, add

"""Instead, try something like:

>>> def test():
...   print("hello")
...   raise IndexError()
...
>>> try:
...   test()
...   raise BaseException # Fail if no IndexError
... except IndexError:
...   pass
hello
"""

* I discovered a subtle consequence of the print change: "print 1, 1/0" will 
print '1' on a line by itself and then a traceback. "print(1,1/0)" just prints 
a traceback. Something like

class C():
def __str__(self):
raise IndexError
print(1, C(), sep='\n')

is required to print a value by itself on a line and then a traceback.
That is no simpler and probably less realistic than test() above.

--
keywords: +patch

___
Python tracker 

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

[issue11060] distutils2 sdist does not complain about version that is not PEP 386 compliant

[Éric Araujo]

Éric Araujo  added the comment:

data['version'] is not always present.  Can you find what part of the code 
fills it?

There is no default in the PEP, but a tool like mkcfg could suggest a useful 
value, say 0.1dev0.

--

___
Python tracker 

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

[issue11617] Sporadic failure in test_httpservers

[STINNER Victor]

STINNER Victor  added the comment:

Another example on x86 OpenIndiana 3.2:

test test_httpservers failed -- Traceback (most recent call last):
  File 
"/export/home/buildbot/32bits/3.2.cea-indiana-x86/build/Lib/test/test_httpservers.py",
 line 210, in test_latin1_header
self.assertEqual(res.getheader('X-Special'), 'D\xe4ngerous Mind')
AssertionError: None != 'D\xe4ngerous Mind'

--
nosy: +haypo

___
Python tracker 

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

[issue11127] sockets should not be pickleable

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Committed the patch with the test, thank you!

--
resolution:  -> fixed
stage: needs patch -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue11127] sockets should not be pickleable

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 5e13e5e6df30 by Antoine Pitrou in branch 'default':
Issue #11127: Raise a TypeError when trying to pickle a socket object.
http://hg.python.org/cpython/rev/5e13e5e6df30

--
nosy: +python-dev

___
Python tracker 

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

[issue11395] print(s) fails on Windows with long strings

[STINNER Victor]

STINNER Victor  added the comment:

I realized that it was a little more difficult to port the fix on 3.1 because 
3.1 doesn't have the fix for Windows 64 bits. So I only fixed Python 3.2 and 
3.3, also because nobody reported failure for Python 3.1 on Windows with -u 
flag.

I tested my fix: the test fails without the fix, and it pass correctly with the 
fix. So let's close the last regression that I introduced in Python 3.2!

--
resolution:  -> 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

[issue11616] reap_children should not use WNOHANG

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Probably a bad idea on my part :)

--
resolution:  -> rejected
status: open -> closed

___
Python tracker 

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

[issue8933] Invalid detection of metadata version

[Éric Araujo]

Éric Araujo  added the comment:

Diffs or full changesets are fine.  I’m putting this on standby until 
distutils2 is fully merged into the standard library and we have a clear 
workflow from packaging to d2 to d1.

--
status: open -> pending

___
Python tracker 

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

[issue11395] print(s) fails on Windows with long strings

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 8939a21bdb94 by Victor Stinner in branch '3.2':
Issue #11395: io.FileIO().write() clamps the data length to 32,767 bytes on
http://hg.python.org/cpython/rev/8939a21bdb94

New changeset 4b3472169493 by Victor Stinner in branch 'default':
(merge) Issue #11395: io.FileIO().write() clamps the data length to 32,767
http://hg.python.org/cpython/rev/4b3472169493

--
nosy: +python-dev

___
Python tracker 

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

[issue9364] some problems with the documentation of pydoc

[Éric Araujo]

Éric Araujo  added the comment:

yeswanth, would you like more guidance or should I finish the patches?

--

___
Python tracker 

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

[issue901727] extra_path kwarg to setup() undocumented

[Éric Araujo]

Éric Araujo  added the comment:

I added the missing words and reworded a few lines in the attached patch.

That said, I find that the new doc is barely helpful; it describes what the 
code does, but does not explain why one would want to use it.  Comments in the 
source suggest it’s a hack to support some corner case.  This is still obscure 
to me after reading about it and looking at the code :(

--
Added file: http://bugs.python.org/file21313/apiref-followup.diff

___
Python tracker 

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

[issue11586] Python/pythonrun.c: get_codec_name() typo

[STINNER Victor]

STINNER Victor  added the comment:

Fixed, thanks Ray.

--
resolution:  -> 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

[issue11586] Python/pythonrun.c: get_codec_name() typo

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 0b6f6514461e by Victor Stinner in branch '3.2':
Fix #11586: typo in initfsencoding()
http://hg.python.org/cpython/rev/0b6f6514461e

--

___
Python tracker 

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

[issue11586] Python/pythonrun.c: get_codec_name() typo

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 929aafef5b05 by Victor Stinner in branch 'default':
Fix #11586: typo in initfsencoding()
http://hg.python.org/cpython/rev/929aafef5b05

--
nosy: +python-dev

___
Python tracker 

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

[issue11342] ResourceWarning: unclosed file <_io.TextIOWrapper

[Éric Araujo]

Éric Araujo  added the comment:

The test failures you quoted are not related to distutils.

I cannot see the ResourceWarnings in 3.2 or 3.3, so I think I’m going to close 
this issue.

--
components:  -Distutils, Distutils2
status: open -> pending
versions:  -3rd party, Python 2.7, Python 3.1, Python 3.2, Python 3.3

___
Python tracker 

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

[issue10977] Concrete object C API needs abstract path for subclasses of builtin types

[Raymond Hettinger]

Raymond Hettinger  added the comment:

I'm just pointing out a problem.  Lots of existing code uses the concrete API 
and that code can break subclasses of builtin types (possibly resulting in 
segfaults).

I don't really care what is done about it.  I'm just observing that the current 
design of the concrete API is incompatible with subclassing of built-in types.

A practical consequence is that I don't see how to write a fast, light C 
version of OrderedDict that would be safe from calls to PyDict_SetItem().  
AFAICT, there is no mechanism for adding extra state to a built-in type and 
being able to guarantee even very simple invariants.

--

___
Python tracker 

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

[issue10977] Concrete object C API needs abstract path for subclasses of builtin types

[Nick Coghlan]

Nick Coghlan  added the comment:

It's largely a backwards compatibility hack, but the concrete methods also have 
an optimised fast path that the generic methods lack.

So (for example), PyList_SetItem would now mean "this is *probably* a list, so 
check for that and use the fast path if the assumption is correct, otherwise 
fall back on PyObject_SetItem".

Currently, using the concrete API means your code potentially *breaks* if the 
assumption is incorrect.

Sounds like a good idea to me, and will fix a lot of cases of bad interaction 
between concrete types and related objects.

--
nosy: +ncoghlan

___
Python tracker 

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

[issue11393] Integrate faulthandler module into Python 3.3

[Antoine Pitrou]

Antoine Pitrou  added the comment:

It would be nice if it were enabled by default for fatal errors (and asserts 
perhaps?).

--
nosy: +pitrou

___
Python tracker 

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

[issue10617] Collections ABCs can’t be linked to

[Georg Brandl]

Georg Brandl  added the comment:

I would recommend to document the classes like all other classes.  It may take 
a bit more space, but it will be a) standard and b) linkable.

--

___
Python tracker 

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

[issue3080] Full unicode import system

[Roundup Robot]

Roundup Robot  added the comment:

New changeset 7f4a4e393058 by Victor Stinner in branch 'default':
Issue #3080: imp.load_module() accepts None for the module path
http://hg.python.org/cpython/rev/7f4a4e393058

--

___
Python tracker 

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

[issue11254] distutils doesn't byte-compile .py files to __pycache__ during installation

[Éric Araujo]

Éric Araujo  added the comment:

I cleaned up the patch a bit.  In particular, I removed the dance around 
imp.cache_from_source that seemed unnecessary.  I tested with regular Python, 
with PYTHONOPTIMIZE and with PYTHONDONTWRITEBYTECODE.  Can you review and test 
too?

--
Added file: http://bugs.python.org/file21312/bug-11254.diff

___
Python tracker 

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

[issue11608] GzipFile cannot be used for streaming

[Ned Deily]

Ned Deily  added the comment:

As Ray points out, this feature is now available in Python 3.2. It was not 
backported to Python 2.7 as only bug fixes are now being accepted for Python 
2.x, not new features.

--
nosy: +ned.deily
resolution:  -> duplicate
stage:  -> committed/rejected
status: open -> closed
superseder:  -> gzip.GzipFile to accept stream as fileobj.

___
Python tracker 

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

[issue11149] [PATCH] Configure should enable -fwrapv for clang

[Brett Cannon]

Brett Cannon  added the comment:

Someone having the time to do a patch review.

--
keywords: +needs review
stage:  -> patch review

___
Python tracker 

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

[issue11613] test_subprocess fails under Windows

[Reid Kleckner]

Reid Kleckner  added the comment:

It is necessary, WaitForSingleObject takes its argument in
milliseconds.  It will make the exception message wrong, though, which
I can fix.

Reid

On Sun, Mar 20, 2011 at 1:46 PM, Santoso Wijaya  wrote:
>
> Santoso Wijaya  added the comment:
>
> The timeout value given to wait() is multiplied by 1000 before being passed 
> to TimeoutExpired constructor. The multiplication is unnecessary since we 
> take the input unit as time unit second.
>
> --
> keywords: +patch
> nosy: +santa4nt
> Added file: http://bugs.python.org/file21310/timeoutsec.patch
>
> ___
> Python tracker 
> 
> ___
>

--

___
Python tracker 

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

[issue11613] test_subprocess fails under Windows

[Santoso Wijaya]

Santoso Wijaya  added the comment:

Actually, the multiplication was necessary because 
_subprocess.WaitForSingleObject takes the timeout argument in millisecond unit. 
Defer multiplication until then.

--
Added file: http://bugs.python.org/file21311/timeoutsec.patch

___
Python tracker 

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

[issue3722] print followed by exception eats print with doctest

[Chris Withers]

Chris Withers  added the comment:

Terry,

My original post was not about *temporary* output inserted for debugging, but 
test mocks and the like which form a permanent part of the test and which 
output to stdout.

cheers,

Chris

--

___
Python tracker 

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

[issue11613] test_subprocess fails under Windows

[Santoso Wijaya]

Changes by Santoso Wijaya :


Removed file: http://bugs.python.org/file21310/timeoutsec.patch

___
Python tracker 

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

[issue3080] Full unicode import system

[Andrew Svetlov]

Changes by Andrew Svetlov :


--
nosy: +asvetlov

___
Python tracker 

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

[issue11149] [PATCH] Configure should enable -fwrapv for clang

[Ismail Donmez]

Ismail Donmez  added the comment:

Whats holding this up?

--

___
Python tracker 

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

[issue11613] test_subprocess fails under Windows

[Santoso Wijaya]

Santoso Wijaya  added the comment:

The timeout value given to wait() is multiplied by 1000 before being passed to 
TimeoutExpired constructor. The multiplication is unnecessary since we take the 
input unit as time unit second.

--
keywords: +patch
nosy: +santa4nt
Added file: http://bugs.python.org/file21310/timeoutsec.patch

___
Python tracker 

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

[issue10977] Concrete object C API needs abstract path for subclasses of builtin types

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> The problem isn't a lack of available abstract
> functions.

Then what is it exactly?
You are arguing for PyList_SetItem() to have the same semantics as
PyObject_SetItem(), but what's the point of having two functions which
do exactly the same thing? It doesn't make sense to me.

--

___
Python tracker 

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

[issue10977] Concrete object C API needs abstract path for subclasses of builtin types

[Raymond Hettinger]

Raymond Hettinger  added the comment:

> By the way, PyList_SetItem() already has an 
> abstract counterpart called PyObject_SetItem(), 
> so changing this one seems useless.

The problem isn't a lack of available abstract
functions. If some code uses PyObject_SetItem(),
then it already works for both list and dict
subclasses. The issue arises only for code that is 
already using PyDict_SetItem() or PyList_SetItem().

The concrete methods pre-date the abstract methods
and pre-date the ability to subclass built-in types.
Those concrete methods were never updated to reflect
the new reality.  They now only make sense for exact
type matches because they know nothing about 
subclasses and their invariants.


IOW, if some patch is accepted, it needs to be aimed
at the concrete functions to make them subclass aware.

--

___
Python tracker 

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

[issue11591] "python -S" should be robust against e.g. "from site import addsitedir"

[Brett Cannon]

Brett Cannon  added the comment:

This is what I get for trying to clean up site.py years ago. =)

I'm fine with the change as long as there is a very clear Misc/NEWS message 
that the semantics on import have changed (and obviously this is not 
backported).

--
assignee:  -> eric.araujo

___
Python tracker 

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

[issue11494] Confusing error message from warnings.warn

[Brett Cannon]

Brett Cannon  added the comment:

I'm not crazy about the idea of adding an inspect.isclass() check simply to 
clear up this rare error message. It's still decipherable what the problem is 
from the issubclass() message.

Closing as "won't fix".

--
resolution:  -> wont fix
status: open -> closed

___
Python tracker 

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

[issue11618] Locks broken wrt timeouts on Windows

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> > At that time, the Pythread_* functions were still in use by the GIL
> > implementation, and it made a difference judging by the commit message.
> 
> Hmm. And if some application uses thread.lock heavily, won't it still
> make a difference?

An acquire/release pair is less than one microsecond here. Compared to
the evaluation overhead of Python code, it seems not very significant.
That said, if someone can guarantee than the complex approach is
correct, why not.

--

___
Python tracker 

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

[issue11397] os.path.realpath() may produce incorrect results

[Terry J. Reedy]

Changes by Terry J. Reedy :


--
stage: test needed -> needs patch

___
Python tracker 

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

[issue11618] Locks broken wrt timeouts on Windows

[Martin v . Löwis]

Martin v. Löwis  added the comment:

> At that time, the Pythread_* functions were still in use by the GIL
> implementation, and it made a difference judging by the commit message.

Hmm. And if some application uses thread.lock heavily, won't it still
make a difference?

--

___
Python tracker 

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

[issue2200] find_executable fails to find .bat files on win32

[Martin v . Löwis]

Martin v. Löwis  added the comment:

I think the patch is incorrect. Parsing PATHEXT means that it will believe that 
all extensions listed on PATHEXT are executable. However, os.spawnv is not able 
to run them all, but only a subset. IIUC, spawnv supports (from spawnve.c)

static _TSCHAR *ext_strings[] = { _T(".cmd"), _T(".bat"), _T(".exe"), 
_T(".com") };

PATHEXT typically includes also .js and other extensions which cannot be run 
through spawnv.

--

___
Python tracker 

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

[issue11397] os.path.realpath() may produce incorrect results

[Santoso Wijaya]

Santoso Wijaya  added the comment:

Attaching a unittest that will manifest this bug.

--
keywords: +patch
Added file: http://bugs.python.org/file21309/test_issue11397.patch

___
Python tracker 

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

[issue11618] Locks broken wrt timeouts on Windows

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> Interestingly, it used to be a Semaphore up to [5e6e9e893acd]; in
> [cde4da18c4fa], Yakov Markovitch rewrote this to be the faster
> implementation we have today.

At that time, the Pythread_* functions were still in use by the GIL
implementation, and it made a difference judging by the commit message.

--

___
Python tracker 

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

[issue11618] Locks broken wrt timeouts on Windows

[Martin v . Löwis]

Martin v. Löwis  added the comment:

Interestingly, it used to be a Semaphore up to [5e6e9e893acd]; in 
[cde4da18c4fa], Yakov Markovitch rewrote this to be the faster implementation 
we have today.

--

___
Python tracker 

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