[issue25654] test_multiprocessing_spawn ResourceWarning with -Werror

Serhiy Storchaka added the comment: Here is an alternative patch that uses closefd=False. -- Added file: http://bugs.python.org/file41075/multiprocessing_no_close_fd.patch ___ Python tracker

[issue25654] test_multiprocessing_spawn ResourceWarning with -Werror

Serhiy Storchaka added the comment: Good point! The question is whether we want to see resource warnings when override standard streams with open files (usually without closefd=False)? -- ___ Python tracker

[issue25626] Gzip fails for file over 2**32 bytes

Martin Panter added the comment: Okay. For the gzip module, I cannot easily test this myself. Quickly looking at other cases, I guess it would look something like this, but I need to spend some time understanging the bigmemtest decorator properly: @unittest.skipIf(sys.maxsize < _4G, "Requires

[issue25654] test_multiprocessing_spawn ResourceWarning with -Werror

Serhiy Storchaka added the comment: You can commit your patch right now (it shouldn't make things worse), and continue to work on additional tests. -- ___ Python tracker

[issue25626] Gzip fails for file over 2**32 bytes

Serhiy Storchaka added the comment: Could you please add tests for other two functions? And tests for the gzip module? -- ___ Python tracker ___

[issue25663] Make rlcompleter avoid duplicate global names

Serhiy Storchaka added the comment: Nice work. An alternative approach is to make "matches" a dict. And instead of if match not in seen: seen.add(word) matches.append(match) use just matches[word] = match I don't know what approach is better. Added other minor

[issue25665] typing.NamedTuple instances are not picklable.

New submission from Ashwini Chaudhary: Currently namedtuple(https://hg.python.org/cpython/file/3.5/Lib/collections/__init__.py#l418) sets the `__module__` attribute by looking up `__name__` in calling frame's globals. As in the case of `typing.NamedTuple` it is always going to be 'typing'

[issue25665] typing.NamedTuple instances are not picklable.

Changes by Serhiy Storchaka : -- nosy: +gvanrossum, rhettinger ___ Python tracker ___

[issue12346] Python source code build fails with old mercurial

Luke Erlacher added the comment: This is not fixed in 2.7.10. -- nosy: +Luke Erlacher ___ Python tracker ___

[issue25666] Python unexpectedly ignores a signal after fork

New submission from jon orebro: Description: I found a slight problem with signal handling. It seems that if you have a signal handler setup for a signal, right after a fork the child ignores that signal for a short time. This is regardless of what the signal handler is setup to do. This can

[issue25665] typing.NamedTuple instances are not picklable.

Ashwini Chaudhary added the comment: Attached patch. -- keywords: +patch Added file: http://bugs.python.org/file41073/issue25665.patch ___ Python tracker

[issue20438] inspect: Deprecate getfullargspec?

Vedran Čačić added the comment: > Also, it is worth noting, that Signature API does not provide 100% of functionality that deprecated APIs have. It is important to do a soft deprecation of outdated APIs in 3.5 to gather users feedback, and improve Signature object. Well, here is a feedback

[issue25626] Gzip fails for file over 2**32 bytes

Serhiy Storchaka added the comment: You can commit your patch right now (it shouldn't make things worse) and add new tests later. -- ___ Python tracker

[issue25634] Add a dedicated subclass for attribute missing errors

Jun Wang added the comment: I think this is a common problem while using both __getattr__ and descriptor/property. A descriptor example: class Descriptor(): def __get__(self, instance, owner=None): raise AttributeError('Implicitly suppressed') class A(): d =

[issue25634] Add a dedicated subclass for attribute missing errors

R. David Murray added the comment: The more detailed discussion should happen on python-ideas. I'm going to close this for now, but if there's a consensus there about what action to take this issue could be reopened. (Or a new one created, whatever makes the most sense at that time.)

[issue25654] test_multiprocessing_spawn ResourceWarning with -Werror

Serhiy Storchaka added the comment: Oh, I forgot about issue21779. And there was at least yet one duplicate issue23827. Thanks for the reminder Davin. The patch in issue21779 fixes only symptoms, makes failed test more lenient. I prefer to got rid of resource warnings at all. --

[issue25673] unittest assertLessEqual not working properly with lists

New submission from kehlert: I attached a file that explains the issue. Basically, assertLessEqual only seems to compare the first elements of the two lists and ignores the others. Thus a test can pass when it shouldn't. -- files: example.py messages: 254920 nosy: kehlert priority:

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: No, I mentioned the idea of a warning in the thread on the python-tulip mailing list, but decided not to do it after all. -- ___ Python tracker

[issue25672] set SSL_MODE_RELEASE_BUFFERS

Cory Benfield added the comment: Ok, I've just uploaded an initial draft of the patch for review. -- keywords: +patch Added file: http://bugs.python.org/file41083/ssl.patch ___ Python tracker

[issue21779] test_multiprocessing_spawn fails when ran with -Werror

Serhiy Storchaka added the comment: Proposed patch cures only the symptom, the failure of one rigid test. A warning still is emitted in applications that use multiprocessing. Closed in favor of duplicate issue25654 with patches that are intended to fix the cause of warnings. In any case thank

[issue25673] unittest assertLessEqual not working properly with lists

Ezio Melotti added the comment: This is how comparison works for sequences and it's not a bug: >>> a = [1, 2] >>> b = [2, 1] >>> a <= b True >>> a = [2, 1] >>> b = [1, 2] >>> a <= b False See https://docs.python.org/3/tutorial/datastructures.html#comparing-sequences-and-other-types

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Justin Mayfield added the comment: I should have commented more on the run_once removal. The depiction given in its docstring seemed inconsistent with the new way stop works and I found no callers, so it seemed like it was best left out to avoid confusion. No worries though, I didn't get to

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Justin Mayfield added the comment: Ha, email race. Regarding rev 2, the updated docstring and scheduled stop looks good along with alleviating the confusion I mentioned. I'm not sure about your warning comment; Perhaps that's a patch I didn't lay eyes on. Cheers. --

[issue25672] Unconditionally set SSL_MODE_RELEASE_BUFFERS

New submission from Cory Benfield: Originally raised by Ben Bangert on the python-dev mailing list. It turns out that OpenSSL has a mode setting, SSL_MODE_RELEASE_BUFFERS, that can be set by a call to SSK_CTX_set_mode. This mode can potentially reduce connection overhead by nearly 18kB *per

[issue25533] Make pkgutil.iter_modules() yield built-in modules

Brett Cannon added the comment: I say let it go and make sure the docs clearly document that only modules found off of sys.path are supported. Otherwise I would look at why walk_packages() and pydoc feel the need to import every module and if import simply needs to be tweaked to support this

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: OK, here's another revision of the patch, setting the timeout passed to the selector to 0 when the loop is pre-stopped. -- Added file: http://bugs.python.org/file41084/issue25593_revised_3.patch ___ Python tracker

[issue25660] tabs don't work correctly in python repl

Yury Selivanov added the comment: Attached is a patch that uses different logic for tabulation. Instead of returning '\t' from the completion callback, it instead calls explicit readline API: "readline.insert_text('\t'); readline.redisplay()" Please review. -- keywords: +patch Added

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: OK, another revision, keep the mock selector. -- Added file: http://bugs.python.org/file41086/issue25593_revised_4.patch ___ Python tracker

[issue23788] test_urllib2_localnet.test_bad_address fails: OSError not raised by urlopen

Steve Harris added the comment: I experienced the same problem on my system: OS: Xubuntu/Linux 12.04 $ uname -a Linux ives 3.2.0-94-generic #134-Ubuntu SMP Fri Nov 6 18:17:08 UTC 2015 i686 i686 i386 GNU/Linux $ gcc --version gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3

[issue9504] signal.signal/signal.alarm not working as expected

Martin Panter added the comment: With Python’s current technique for signal handling, I think there is always going to be a window where a signal can arrive at the low level after checking the Python flag and before starting a blocking call. If you need to robustly handle a signal maybe it is

[issue25666] Python unexpectedly ignores a signal after fork

Martin Panter added the comment: Here is a modified version of the script that is not a slow fork bomb. In the original, if time.sleep(600) fails to be interrupted, the children end up continuing the loop and forking more children. I tried Python 3.5, 2.7 and 3.4. I am seeing the signal

[issue9504] signal.signal/signal.alarm not working as expected

Martin Panter added the comment: Sorry, EINTR changes were PEP 475. -- ___ Python tracker ___ ___

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Changes by Andrew Svetlov : -- nosy: +asvetlov ___ Python tracker ___ ___

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Changes by Andrew Svetlov : -- versions: +Python 3.6 ___ Python tracker ___ ___

[issue25670] ast.NodeTransformer: remove duplicate getattr()

Serhiy Storchaka added the comment: LGTM! -- assignee: -> haypo components: +Library (Lib) nosy: +serhiy.storchaka stage: -> commit review ___ Python tracker

[issue23441] rlcompleter: tab on empty prefix => insert spaces

Yury Selivanov added the comment: Please see my patch for Issue 25660 -- instead of returning '\t' from the completion function, I use readline API to modify the input buffer directly. -- ___ Python tracker

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Roundup Robot added the comment: New changeset 9b3144716d17 by Guido van Rossum in branch '3.4': Issue #25593: Change semantics of EventLoop.stop(). https://hg.python.org/cpython/rev/9b3144716d17 New changeset 158cc5701488 by Guido van Rossum in branch '3.5': Issue #25593: Change semantics of

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: Hopefully this is it! -- assignee: -> gvanrossum resolution: later -> fixed status: open -> closed ___ Python tracker

[issue25672] set SSL_MODE_RELEASE_BUFFERS

Changes by Ethan Furman : -- nosy: +alex, christian.heimes, dstufft, giampaolo.rodola, janssen, pitrou title: Unconditionally set SSL_MODE_RELEASE_BUFFERS -> set SSL_MODE_RELEASE_BUFFERS ___ Python tracker

[issue25672] Unconditionally set SSL_MODE_RELEASE_BUFFERS

Cory Benfield added the comment: Oh, one further requirement: we should *not* set this mode for OpenSSL releases 1.x through 1.0.1g, which have a NULL pointer dereference vulnerability (CVE 2014-0198). Thanks to Marc-Andre Lemburg for spotting this. See also:

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: New patch. Update test_utils.run_once() to use the recommended idiom. On second thought I don't like issuing a warning when stop() is called before the loop runs -- a warning seems overkill for something so minor. But I'm okay with no longer recommending

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Justin Mayfield added the comment: I see. Seems like good discussion over there. I joined up. -- ___ Python tracker ___

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: Whoops. Hopefully this one's right. -- Added file: http://bugs.python.org/file41087/issue25593_revised_5.patch ___ Python tracker

[issue25656] multiprocessing.dummy: pool.map hangs on empty list

Changes by Tomer : -- nosy: +jnoller, sbt ___ Python tracker ___ ___ Python-bugs-list

[issue25674] test_ssl (test_algorithms) failures on bolen-ubuntu slaves: sha256.tbs-internet.com unknown host

David Bolen added the comment: Ah, it appears that the transient_internet context manager in the test causes it to be skipped if the host is unknown. So mine was just "lucky" in that it fell back to connecting somewhere else. I've removed my resolver search path on bolen-ubuntu which will

[issue25676] Bug

Changes by Zachary Ware : Removed file: http://bugs.python.org/file41088/Garcinia Cambogia Free Trial UK.html ___ Python tracker ___

[issue25083] Python can sometimes create incorrect .pyc files

Brett Cannon added the comment: As I said previously, tzickel, this is an edge case in Python 2.7 which means it has a low priority for most core developers. Realize that we are also working towards a Python 3.5.1 release and a 3.4.4 release as well (which will be the first and last bugfix

[issue25674] test_ssl (test_algorithms) failures on bolen-ubuntu slaves: sha256.tbs-internet.com unknown host

New submission from David Bolen: It appears that the test host (sha256.tbs-internet.com) used by test_algorithms in test_ssl.py no longer exists. It was showing up as a certificate failure in the test because it ended up falling back to a resolv.conf search path which yielded a host that did

[issue18620] multiprocessing page leaves out important part of Pool example

Changes by Raymond Hettinger : -- assignee: docs@python -> rhettinger ___ Python tracker ___

[issue25675] doc for BaseEventLoop.run_in_executor() says its a coroutine, but it is not

Changes by Guido van Rossum : -- keywords: +easy priority: normal -> low ___ Python tracker ___

[issue25675] doc for BaseEventLoop.run_in_executor() says its a coroutine, but it is not

Changes by Nicolas Demarchi : -- nosy: +gilgamezh ___ Python tracker ___ ___

[issue25675] doc for BaseEventLoop.run_in_executor() says its a coroutine, but it is not

New submission from Ezequiel Alfie: Doc says: coroutine BaseEventLoop.run_in_executor(executor, func, *args) .. This method is a coroutine. However, inspecting the code, I find that this function is not a coroutine. Also this function returns a Future instance, this fact should be

[issue25676] Spam

Changes by Zachary Ware : -- components: -Windows resolution: -> not a bug stage: -> resolved status: open -> closed title: Bug -> Spam type: enhancement -> versions: -Python 2.6 ___ Python tracker

[issue25083] Python can sometimes create incorrect .pyc files

tzickel added the comment: Meador Inge any other questions regarding the issue ? I can't believe 2.7.11 is coming out soon, and nobody is taking this issue seriously enough... -- ___ Python tracker

[issue25676] Bug

New submission from Richard Tupper: resolved -- components: Windows files: Garcinia Cambogia Free Trial UK.html messages: 254940 nosy: omskinbourbe1977, paul.moore, steve.dower, tim.golden, zach.ware priority: normal severity: normal status: open title: Bug type: enhancement versions:

[issue25674] test_ssl (test_algorithms) failures on bolen-ubuntu slaves: sha256.tbs-internet.com unknown host

Zachary Ware added the comment: So the mystery of that buildbot is solved, but I think that test needs some work. -- keywords: +buildbot nosy: +alex, christian.heimes, dstufft, giampaolo.rodola, janssen, pitrou stage: -> needs patch ___ Python

[issue25646] Distutils and Windows SDK 7.0 / Windows SDK 7.1 / Visual C++ Build Tools 2015

JGoutin added the comment: Hello, I modified "msvc9compiler.py" with following changes : - Add automatic compatibility with standalone VC++ compilers from Windows SDK 7.1. (Not for 6.1/7.0, because that finally don't work as good as it seem on first view and there is already MSVC++ for

[issue25664] Unexpected UnicodeDecodeError in logging module

Changes by Serhiy Storchaka : -- nosy: +vinay.sajip ___ Python tracker ___ ___

[issue25670] ast.NodeTransformer: remove duplicate getattr()

New submission from STINNER Victor: ast.NodeTransformer.generic_visit() calls getattr() whereas iter_fields() already returns the attribute. The getattr() is useless, attached patch removes it. Sadly, it looks like test_ast doesn't test this class at all :-( The class is tested *indirectly*

[issue25669] unittest.assertEqual() on un-equal types inheriting from collections.Mapping

Nathan Herring added the comment: FWIW, it's also pretty easy to write self.assertEqual({'key': 'value'}, dict(foo)) and get the right behavior, but it'd be nice if it was done automagically by type checking. -- ___ Python tracker

[issue25668] Deadlock in logging caused by a possible race condition with "format"

Florent Viard added the comment: I understand that it should have been the reason of this. But in my opinion, it is very bad to possibly have calculations and user space arbitrary operations inside the logging lock. If you look at my proposition, you can do the format after the filter, but

[issue25671] Fix venv activate.fish to maintain $status

New submission from Nathan West: Many fish_prompts use display the $status (fish's equivalent to $?) somewhere in the prompt. Currently, venv's activate.fish overwrites and wraps the user's fish_prompt, the wrapping doesn't preserve this $status. This patch ensures that the $status is

[issue25677] Syntax error caret confused by indentation

New submission from Michael Layzell: It appears that syntax errors generated by checking the AST will get confused by indentation and place the caret incorrectly. With no indentation: === 1 + 1 = 2 === File "/Users/mlayzell/test.py", line 1 1 + 1 = 2 ^ SyntaxError: can't assign to

[issue25665] typing.NamedTuple instances are not picklable.

Roundup Robot added the comment: New changeset 33df0056c148 by Guido van Rossum in branch '3.5': Issue #25665: Make NamedTuple picklable. https://hg.python.org/cpython/rev/33df0056c148 New changeset 8a32d44b8359 by Guido van Rossum in branch 'default': Issue #25665: Make NamedTuple picklable.

[issue25665] typing.NamedTuple instances are not picklable.

Guido van Rossum added the comment: Fixed it! Thanks for the report *and* the patch. I wrote a different test though. -- assignee: -> gvanrossum resolution: -> fixed status: open -> closed versions: +Python 3.6 ___ Python tracker

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: I'm going to fix up the patch and apply it so this can make 3.5.1 rc1. -- ___ Python tracker ___

[issue25670] ast.NodeTransformer: remove duplicate getattr()

STINNER Victor added the comment: Oh, the getattr() is as old as the Lib/ast.py module: change 7d824fd131ad. -- ___ Python tracker ___

[issue25583] os.makedirs with exist_ok=True raises PermissionError on Windows 7^

Roundup Robot added the comment: New changeset 05d6ddf2b7c2 by Martin Panter in branch '3.4': Issue #25583: Avoid incorrect errors raised by os.makedirs(exist_ok=True) https://hg.python.org/cpython/rev/05d6ddf2b7c2 New changeset 515f76bf1254 by Martin Panter in branch '3.5': Issue #25583: Merge

[issue25583] os.makedirs with exist_ok=True raises PermissionError on Windows 7^

Martin Panter added the comment: None of the Windows buildbots are failing my particular test. There are other failures, but they look unrelated, so I am calling this fixed. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed

[issue25669] unittest.assertEqual() on un-equal types inheriting from collections.Mapping

Changes by Raymond Hettinger : -- assignee: -> rhettinger nosy: +rhettinger ___ Python tracker ___

[issue20898] Missing 507 response description

SilentGhost added the comment: Re msg233465: it doesn't seem like a particularly good justification to remove something that is not hurting anyone. The problem now is that because http.HTTPStatus is an enumeration, it cannot be extended, therefore when someone encounters these non-standard

[issue7676] IDLE shell shouldn't use TABs

Changes by irdb : -- nosy: +irdb ___ Python tracker ___ ___ Python-bugs-list mailing

[issue25672] set SSL_MODE_RELEASE_BUFFERS

Benjamin Peterson added the comment: It might be better to do a runtime OpenSSL version check in case someone upgrades or downgrades to an vulnerable version without recompiling Python. -- nosy: +benjamin.peterson ___ Python tracker

[issue25670] ast.NodeTransformer: remove duplicate getattr()

Roundup Robot added the comment: New changeset 7789f338447f by Victor Stinner in branch '3.5': Issue #25670: Remove duplicate getattr() from ast.NodeTransformer https://hg.python.org/cpython/rev/7789f338447f New changeset 2d90f4b27743 by Victor Stinner in branch 'default': Merge 3.5 (issue

[issue25668] Deadlock in logging caused by a possible race condition with "format"

R. David Murray added the comment: Using the lock to cover as little as possible makes sense, but there may be a reason format is covered by the lock. I don't know the code well enough to say, we'll have to wait for Vinay. -- ___ Python tracker

[issue25667] Supply dual-stack (IPv4/IPv6) socket bind routine

New submission from Jason R. Coombs: When binding to a socket for creating a service, the socket API provides a mechanism for selecting only IPv4 or IPv6, but the simple operation of binding to all interfaces in a dual-stack environment is not easy, as is revealed in issue20215 and

[issue25667] Supply dual-stack (IPv4/IPv6) socket bind routine

R. David Murray added the comment: I don't think it should require a PEP, it looks like a fairly straightforward small enhancement. What is the reason to not put it in the socket module? -- nosy: +r.david.murray stage: -> needs patch type: -> enhancement

[issue25668] Deadlock in logging caused by a possible race condition with "format"

R. David Murray added the comment: But not doing the format until the last moment is part of the design of logging (as low overhead as possible unless a message is actually emitted). I suspect you are just going to have to pre-calculate that variable if you want to log it. Let's see what

[issue25669] unittest.assertEqual() on un-equal types inheriting from collections.Mapping

New submission from Nathan Herring: We have some code that generates objects that inherit from Mapping that are not nearly as straightforward to instantiate in a test. It's much easier to have something like the follows: foo = … # some collections.Mapping subtype self.assertEqual({'key':

[issue25654] test_multiprocessing_spawn ResourceWarning with -Werror

Davin Potts added the comment: Note issue21779 has been around for quite some time with its own patch specifically for multiprocessing, making its tests more robust. I am hesitant to embrace the patch suggested here. -- nosy: +davin ___ Python

[issue25668] Deadlock in logging caused by a possible race condition with "format"

New submission from Florent Viard: When an user, use logging, to try to display an object that uses some threading locks, there could be a race condition and the logging module will deadlock. So, any thread that will try to use logging, will then be stuck forever. Please see the following

[issue25668] Deadlock in logging caused by a possible race condition with "format"

Florent Viard added the comment: Looking at the code, this issue makes sense in logging/__init__.py (+738): def handle(self, record): """ Conditionally emit the specified logging record. Emission depends on filters which may have been added to the handler.

[issue25666] Python unexpectedly ignores a signal after fork

R. David Murray added the comment: Are you sure this is a python issue and not an OS issue? That is, does an equivalent C program work correctly? Since the operation is a fork, I don't think there's anything that python does that would cause the signal to be ignored. The comment block in

[issue20438] inspect: Deprecate getfullargspec?

R. David Murray added the comment: Please open a new issue for that observation/request. -- ___ Python tracker ___

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: Here's a better patch. - Renamed _stopped to _stopping. - Restore test_utils.run_once() and add a test for it. - Change logic so if _stopping is True upon entering run_forever(), it will run once. Please try it out!! --

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

Guido van Rossum added the comment: Here's the file. -- Added file: http://bugs.python.org/file41080/issue25593_revised.patch ___ Python tracker ___