[issue32843] More revisions to test.support docs

2018-02-14 Thread Cheryl Sabella
New submission from Cheryl Sabella : Serhiy had made the following comments on the pull request for issue11015, but that PR was merged before applying his requested changes. This issue is to address his concerns. TESTFN_NONASCII - How different from TESTFN_UNICODE? PGO -

[issue32795] subprocess.check_output() with timeout does not exit if child process does not generate output after timeout

2018-02-14 Thread Fangyi Zhou
Fangyi Zhou added the comment: I choose to implement the first way. I believe this is in favour in prevention of resource leaks and zombie processes. https://github.com/python/cpython/pull/5684 -- ___ Python tracker

[issue32795] subprocess.check_output() with timeout does not exit if child process does not generate output after timeout

2018-02-14 Thread Fangyi Zhou
Change by Fangyi Zhou : -- keywords: +patch pull_requests: +5480 stage: -> patch review ___ Python tracker ___

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread miss-islington
Change by miss-islington : -- pull_requests: +5477 ___ Python tracker ___

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread miss-islington
Change by miss-islington : -- pull_requests: +5478 ___ Python tracker ___

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Andrew Svetlov
Andrew Svetlov added the comment: Well, makes sense -- ___ Python tracker ___ ___

[issue32756] argparse: parse_known_args: raising exception on unknown arg following known one

2018-02-14 Thread Yauhen
Yauhen added the comment: I am adding one more testcase in order to clarify more the problem: import argparse import sys parser = argparse.ArgumentParser(prog=sys.argv[0], add_help=False) parser.add_argument('-a', action='store_true') parser.add_argument('-b',

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Devin Bayer added the comment: I don't understand how this is about positionals vs optionals. REMAINDER is supposed to capture everything from that point forward, not just positionals. -- ___ Python tracker

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Devin Bayer added the comment: Can we at least document that argparse.REMAINDER cannot be used as the first argument? -- ___ Python tracker

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Change by Devin Bayer : -- nosy: +akvadrako ___ Python tracker ___ ___ Python-bugs-list

[issue30455] Generate C code from token.py and not vice versa

2018-02-14 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- pull_requests: -5479 ___ Python tracker ___

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread miss-islington
miss-islington added the comment: New changeset 8caee0fa572e8ced00df553a7bdca49ddaf729e8 by Miss Islington (bot) in branch '3.7': bpo-32841: Fix cancellation in awaiting asyncio.Condition (GH-5665)

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Nathaniel Smith
Nathaniel Smith added the comment: > Having shield not cancel immediately but rather wait and cancel will cause > long timed shielded operations to stall the task cancellation, usually for no > good. This isn't the general case. What I'm suggesting is that maybe it actually

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Bar Harel
Bar Harel added the comment: I don't think so. Having shield not cancel immediately but rather wait and cancel will cause long timed shielded operations to stall the task cancellation, usually for no good. This isn't the general case. However, adding another function which

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Andrew Svetlov
Andrew Svetlov added the comment: New changeset 5746510b7aef423fa4afc92b2abb919307b1dbb9 by Andrew Svetlov (Bar Harel) in branch 'master': bpo-32841: Fix cancellation in awaiting asyncio.Condition (#5665)

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Andrew Svetlov
Andrew Svetlov added the comment: New changeset a23eecab9a0b724bdfde83d159ac2415927f042a by Andrew Svetlov (Miss Islington (bot)) in branch '3.6': bpo-32841: Fix cancellation in awaiting asyncio.Condition (GH-5665) (GH-5683)

[issue32841] Asyncio.Condition prevents cancellation

2018-02-14 Thread Andrew Svetlov
Andrew Svetlov added the comment: Thanks Bar Harel -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue30455] Generate C code from token.py and not vice versa

2018-02-14 Thread Zachary Ware
Change by Zachary Ware : -- keywords: +patch pull_requests: +5479 ___ Python tracker ___

[issue32795] subprocess.check_output() with timeout does not exit if child process does not generate output after timeout

2018-02-14 Thread Eryk Sun
Eryk Sun added the comment: Work on related issues extends back 9 years to issue 5115. Also see issue 26534. -- nosy: +eryksun ___ Python tracker

[issue32820] Add bits method to ipaddress

2018-02-14 Thread Eric Osborne
Eric Osborne added the comment: Cool, I will kick it over to python-ideas. I checked in some code to handle the format string and it's a lot like what you're suggesting, so I'll leave that in there and see what happens. Thanks! eric On Tue, Feb 13, 2018 at 11:56 PM Nick

[issue32436] Implement PEP 567

2018-02-14 Thread Yury Selivanov
Change by Yury Selivanov : -- pull_requests: +5481 ___ Python tracker ___ ___

[issue32839] Add after_info as a function to tkinter

2018-02-14 Thread Cheryl Sabella
Cheryl Sabella added the comment: >>> It is internal, it was not exposed to Tkinter users before, and the API for >>> restoring the original Python callable is private. I thought `bind(sequence)` also returned these internal Tcl function names? For example, if I do a

[issue20632] Define a new __key__ protocol

2018-02-14 Thread Nick Coghlan
Nick Coghlan added the comment: Allowing for None-first and None-last ordering is a fair use case, but I'm not sure a __key__ protocol is the right answer to that - as your own example shows, it gets tricky when dealing with nested containers. It may make sense to raise

[issue32844] subprocess may incorrectly redirect a low fd to stderr if another low fd is closed

2018-02-14 Thread Alexey Izbyshev
Change by Alexey Izbyshev : -- keywords: +patch pull_requests: +5482 stage: -> patch review ___ Python tracker ___

[issue32844] subprocess may incorrectly redirect a low fd to stderr if another low fd is closed

2018-02-14 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: Note that the PR doesn't attempt to fix leaking of low dup'ed fds to the child. I'll file a separate report for that in a while. -- ___ Python tracker

[issue32840] Must install python 3.6.3 when 3.6.4 already installed

2018-02-14 Thread Ned Deily
Change by Ned Deily : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker ___

[issue32844] subprocess may incorrectly redirect a low fd to stderr if another low fd is closed

2018-02-14 Thread Alexey Izbyshev
New submission from Alexey Izbyshev : When redirecting, subprocess attempts to achieve the following state: each fd to be redirected to is less than or equal to the fd it is redirected from, which is necessary because redirection occurs in the ascending order of

[issue32845] Mac: Local modules can shadow builtins (e.g. a local `math.py` can shadow `math`)

2018-02-14 Thread Eric Cousineau
New submission from Eric Cousineau : We ran into an issue with our library; we build a submodule named `math.so`, and have a test `math_test` which is ran in the same directory as this module (since this is how we've written it with Bazel's generated Python

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread paul j3
paul j3 added the comment: Oops, I see I already mentioned 9334. Here the parsing sequence is a bit different, and the fix I suggest there would not apply here. But the underlying issue is still there - the parser has, in its first iteration, determined that the '--def'

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread paul j3
paul j3 added the comment: REMAINDER is not widely used, and probably was not tested thoroughly during development. It works for the example given in the docs. A variant, argparse.PARSER ('A...') is widely used. This is, effectively, REMAINDER ('...') that requires an

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Devin Bayer added the comment: I still don't understand how that corresponds to the described behavior of REMAINDER and what it has to do with this bug. How can REMAINDER possibly ever work if optionals take priority? However it does when it's not the first argument.

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread paul j3
paul j3 added the comment: This is another expression of the bigger problem of handling arguments that look like flags. In optparse the 'nargs' (or equivalent, it doesn't handle positionals), control how many arguments an Option takes, regardless of their form. In

[issue32845] Mac: Local modules can shadow builtins (e.g. a local `math.py` can shadow `math`)

2018-02-14 Thread Eric V. Smith
Eric V. Smith added the comment: This is the expected behavior, fortunately or not. "math" is not builtin in the sense that is used in that paragraph. -- nosy: +eric.smith ___ Python tracker

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Devin Bayer added the comment: This bug is 5 years old and you are arguing what? That it doesn't matter because it's rarely used compared to some undocumented useless alternative? It's mildly interesting to hear about some implementation detail but I really don't care. I

[issue17050] argparse.REMAINDER doesn't work as first argument

2018-02-14 Thread Devin Bayer
Change by Devin Bayer : -- nosy: -akvadrako ___ Python tracker ___ ___ Python-bugs-list

[issue32846] Deletion of large sets of strings is extra slow

2018-02-14 Thread Terry J. Reedy
New submission from Terry J. Reedy : https://metarabbit.wordpress.com/2018/02/05/pythons-weak-performance-matters/, a blog post on cpython speed, clains "deleting a set of 1 billion strings takes >12 hours". (No other details provided.) I don't have the 100+ gigabytes of

[issue32845] Mac: Local modules can shadow builtins (e.g. a local `math.py` can shadow `math`)

2018-02-14 Thread Eric V. Smith
Eric V. Smith added the comment: Removing macOS since this isn't Mac specific. -- components: +Interpreter Core -macOS ___ Python tracker

[issue32845] Mac: Local modules can shadow builtins (e.g. a local `math.py` can shadow `math`)

2018-02-14 Thread Ned Deily
Ned Deily added the comment: This had me confused for a while. But eric.smith's comment is the clue to what's going on here: > This is the expected behavior, fortunately or not. "math" is not builtin in > the sense that is used in that paragraph. The difference in behavior

[issue32847] Add DirectoryNotEmptyError subclass of OSError

2018-02-14 Thread Barry A. Warsaw
New submission from Barry A. Warsaw : I just ran across errno 39 (Directory not empty) when using Path.rename(newdir) when newdir already existed and was not empty. I realized that OSError exceptions don't have a DirectoryNotEmptyError subclass. Maybe we should add one?

[issue32839] Add after_info as a function to tkinter

2018-02-14 Thread Terry J. Reedy
Terry J. Reedy added the comment: A person who can create a tcl callback with tk.call can inquire with tk.call('after', 'info', id). That does not cover callbacks created by tcl or extensions thereof, but references to such callbacks are unlikely to be useful to anyone who

[issue32642] add support for path-like objects in sys.path

2018-02-14 Thread Jay Yin
Change by Jay Yin : -- keywords: +patch pull_requests: +5484 stage: -> patch review ___ Python tracker ___

[issue32848] Valgrind error with python

2018-02-14 Thread Arun Solomon
New submission from Arun Solomon : Hi... I am facing the problem python along with valgrind. I was getting default valgrind warnings messages. I have created the sample.py. sample.py file does not have any code. I created empty py file to confirm that my code does not

[issue32773] distutils should NOT preserve timestamps

2018-02-14 Thread Jay Yin
Change by Jay Yin : -- pull_requests: +5483 ___ Python tracker ___ ___