[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: > bpo-38858: Allocate small integers on the heap (GH-17301) I associated this change to this issue because I had troubles when I tried to call _PyLong_Fini() in subinterpreters. I'm now trying to have different small integers per interpreter. --

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: New changeset 5dcc06f6e0d7b5d6589085692b86c63e35e2325e by Victor Stinner in branch 'master': bpo-38858: Allocate small integers on the heap (GH-17301) https://github.com/python/cpython/commit/5dcc06f6e0d7b5d6589085692b86c63e35e2325e --

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: After PR 17302 is accepted, I'll work on refactorings including a precedence algorithm to find when to parentheses. -- nosy: +Batuhan Taskaya ___ Python tracker

[issue38863] Improve is_cgi() in http.server

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: @martin.panter, @asvetlov I add the core developers who looks like to be experts on this module. Can you follow up on Siwon's PR? Thanks for your understanding. -- nosy: +asvetlov, martin.panter versions: -Python 3.6

[issue38863] Improve is_cgi() in http.server

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: Thanks, Siwon Kang, I now understood what you want to say. I left some comments on your PR 17312. -- ___ Python tracker ___

[issue38863] Improve is_cgi() in http.server

2019-11-20 Thread Siwon Kang
Siwon Kang added the comment: Hi Donghee, Since you said this is not a bug, I changed the title describing this is a matter of improvement. For your comment, I would say sorry first that I have made you confused. My mention about apache is just to give you an example for the other module

[issue38688] Python 3.8 regression: endless loop in shutil.copytree

2019-11-20 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: PR-17098 basically reverts https://bugs.python.org/issue33695. Not good. =) I think we can simply consume the iterator immediately as in: def copytree(src, ...): with os.scandir(src) as itr: entries = list(itr) return

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- pull_requests: -16785 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- pull_requests: +16800 pull_request: https://github.com/python/cpython/pull/17312 ___ Python tracker ___

[issue38871] lib2to3 generates invalid code with filter and ternary operator

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: I can reproduce on the latest master branch. ./python Tools/scripts/2to3 2.py RefactoringTool: Skipping optional fixer: buffer RefactoringTool: Skipping optional fixer: idioms RefactoringTool: Skipping optional fixer: set_literal RefactoringTool: Skipping

[issue38874] asyncio.Queue: putting items out of order when it is full

2019-11-20 Thread Junyeong Jeong
Junyeong Jeong added the comment: > The items that haven't finished the put aren't actually "in" the queue yet, > so I don't see how non-FIFO order of insertion violates any FIFO guarantees > for the contents of the queue; until the items are actually "in", they're not > sequenced for the

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Guido van Rossum
Guido van Rossum added the comment: (Don't wait for me, I am preoccupied with other things this week.) -- title: UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port -> UDP sockets created by create_datagram_endpoint() allow by

[issue38874] asyncio.Queue: putting items out of order when it is full

2019-11-20 Thread Josh Rosenberg
Josh Rosenberg added the comment: The items that haven't finished the put aren't actually "in" the queue yet, so I don't see how non-FIFO order of insertion violates any FIFO guarantees for the contents of the queue; until the items are actually "in", they're not sequenced for the purposes

[issue38874] asyncio.Queue: putting items out of order when it is full

2019-11-20 Thread Junyeong Jeong
New submission from Junyeong Jeong : Hi The python document says that asyncio.Queue is FIFO queue. But it is not true when the queue is full and multi tasks are putting items into it simultaneously. I know the document does not explicitly mention that asyncio.Queue is multi producer queue, but

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Yury Selivanov
Yury Selivanov added the comment: > Oh in that case, would you like me to close or modify GH-17311? I didn't > think you'd approve of making the more extensive changes all the way back to > 3.5. After reading the comments here I think Antoine's solution makes sense. But... let's wait a bit

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Change by Kyle Stanley : -- stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Kyle Stanley added the comment: > Yeah, I like this prposal; we can apply this to all Python's from 3.5 to 3.8. > With a proper documentation update this should be OK. Oh in that case, would you like me to close or modify GH-17311? I didn't think you'd approve of making the more extensive

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Change by Kyle Stanley : -- keywords: +patch pull_requests: +16799 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17311 ___ Python tracker ___

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Yury Selivanov
Yury Selivanov added the comment: > My preference for create_datagram_endpoint() would be: > - make the "reuse_address" parameter a no-op, and raise an error when > "reuse_address=True" is passed > - do that in 3.8 as well Yeah, I like this prposal; we can apply this to all Python's from

[issue38873] find_library for libcrypto and libssl on Catalina returns the unversioned library

2019-11-20 Thread Sarah Harvey
New submission from Sarah Harvey : I've been tracking this through a bunch of different projects now. With the release of Mac OS X Catalina, libcrypto.dylib is a dummy library that causes an automatic segfault, to prevent upstream software from relying on it. However a large amount of

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Kyle Stanley added the comment: > My preference for create_datagram_endpoint() would be: > - make the "reuse_address" parameter a no-op, and raise an error when > "reuse_address=True" is passed > - do that in 3.8 as well This solution would more elegant, but my concern is that it will

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Kyle Stanley added the comment: > I'd like to point out that it is also documented completely wrong up to this > point in time and thus people who chose True are most likely to be unaware of > the actual consequences. A user's explicit choice based on misinformation is > not really

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset d5d41d39366214c9628b2680fa18fb8d085bcdbc by Miss Islington (bot) in branch '3.8': bpo-36277: Add document for pdb debug and retval commands (GH-12872) https://github.com/python/cpython/commit/d5d41d39366214c9628b2680fa18fb8d085bcdbc

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset 97c301bfc5ea278c32dc5fabe425d8981ec5d3b1 by Miss Islington (bot) in branch '3.7': bpo-36277: Add document for pdb debug and retval commands (GH-12872) https://github.com/python/cpython/commit/97c301bfc5ea278c32dc5fabe425d8981ec5d3b1

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread Cheryl Sabella
Cheryl Sabella added the comment: @Antony.Lee, thank you for the report, @remi.lapeyre, thank you for the research, and @dvnguyen, thank you for the PR. -- nosy: +cheryl.sabella resolution: -> fixed stage: patch review -> resolved status: open -> closed

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread miss-islington
Change by miss-islington : -- pull_requests: +16797 pull_request: https://github.com/python/cpython/pull/17308 ___ Python tracker ___

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread miss-islington
Change by miss-islington : -- pull_requests: +16798 pull_request: https://github.com/python/cpython/pull/17309 ___ Python tracker ___

[issue36277] pdb's recursive debug command is not listed in the docs

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset 9391f6c3ef24f7962c534c42ccb792debdbef509 by Miss Islington (bot) (Dave Nguyen) in branch 'master': bpo-36277: Add document for pdb debug and retval commands (GH-12872)

[issue38872] Document exec symbol for codeop.compile_command

2019-11-20 Thread Cheryl Sabella
New submission from Cheryl Sabella : codeop.compile_command accepts 'exec' as a symbol, but it is not documented. Opening this bug report for an issue initially reported in PR3179. -- assignee: docs@python components: Documentation messages: 357118 nosy: cheryl.sabella, docs@python

[issue38021] pep425 tag for AIX is inadequate

2019-11-20 Thread Ned Deily
Ned Deily added the comment: PEP 425 tags are now in the Python Packaging Authority realm. People from there should also be reviewing this issue and the new PR. CC-ing Brett and Nick for guidance. Also, assuming the feature is approved, I don't think it would be appropriate to backport

[issue38857] AsyncMock issue with awaitable return_value/side_effect/wraps

2019-11-20 Thread Lisa Roach
Change by Lisa Roach : -- pull_requests: +16796 pull_request: https://github.com/python/cpython/pull/17304 ___ Python tracker ___

[issue38021] pep425 tag for AIX is inadequate

2019-11-20 Thread Michael Felt
Change by Michael Felt : -- pull_requests: +16795 pull_request: https://github.com/python/cpython/pull/17303 ___ Python tracker ___

[issue38857] AsyncMock issue with awaitable return_value/side_effect/wraps

2019-11-20 Thread Lisa Roach
Lisa Roach added the comment: New changeset 046442d02bcc6e848e71e93e47f6cde9e279e993 by Lisa Roach (Jason Fried) in branch 'master': bpo-38857: AsyncMock fix for awaitable values and StopIteration fix [3.8] (GH-17269)

[issue36054] On Linux, os.count() should read cgroup cpu.shares and cpu.cfs (CPU count inside docker container)

2019-11-20 Thread Galen Han
Change by Galen Han : -- nosy: +Galen Han ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Ned Deily
Ned Deily added the comment: (Provisionally marking this as a security-related deferred blocker issue for backporting) -- nosy: +ned.deily priority: normal -> deferred blocker type: behavior -> security versions: +Python 3.5, Python 3.6, Python 3.7, Python 3.8

[issue38823] Improve stdlib module initialization error handling.

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset e5d1f734db135e284af8e8868e7ccc85355952b9 by Miss Islington (bot) (Brandt Bucher) in branch 'master': bpo-38823: Clean up _xxtestfuzz initialization. (GH-17216) https://github.com/python/cpython/commit/e5d1f734db135e284af8e8868e7ccc85355952b9

[issue17582] xml.etree.ElementTree does not preserve whitespaces in attributes

2019-11-20 Thread Emiliano Heyns
Emiliano Heyns added the comment: I don't see newlines currently preserved in attributes: elem = ET.parse(StringIO('')).getroot() print(ET.tostring(elem)) -- components: -XML nosy: +Emiliano Heyns versions: -Python 3.4, Python 3.5, Python 3.6

[issue38871] lib2to3 generates invalid code with filter and ternary operator

2019-11-20 Thread Zoran Simic
New submission from Zoran Simic : This code snippet exposes a small edge case in lib2to3, where syntactically invalid code is generated: data = [1, 2, 3, 4, 5] x = filter(lambda x: True if x > 2 else False, data) print(x) lib2to3 transforms 2nd line above to: x = [x for x in data if True if

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Pablo Galindo Salgado
Pablo Galindo Salgado added the comment: After PR17302 is merged we need to fix the following cosmetic issues indicated by Victor: (*) unparse adds many useless parentheses. The algorithm seems naive. For example, it adds "()" to "class _AddedDllDirectory():". It also adds parenthesis around

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Brandt Bucher
Change by Brandt Bucher : -- nosy: +brandtbucher ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Ivan Levkivskyi
Change by Ivan Levkivskyi : -- nosy: +levkivskyi ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38868] Shutil cannot delete a folder that contains an .ini file

2019-11-20 Thread Eryk Sun
Eryk Sun added the comment: What is the system error code (winerror) of the PermissionError? -- nosy: +eryksun ___ Python tracker ___

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Jukka Väisänen
Jukka Väisänen added the comment: > We just chose an unfortunate default for one of them I'd like to point out that it is also documented completely wrong up to this point in time and thus people who chose True are most likely to be unaware of the actual consequences. A user's explicit

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Pablo Galindo Salgado
Change by Pablo Galindo Salgado : -- keywords: +patch pull_requests: +16794 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17302 ___ Python tracker

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread Kyle Stanley
Kyle Stanley added the comment: > There are some platforms (Linux pre-3.9 kernels) that don't have > SO_REUSEPORT. I wish I could say I don't care about such platforms; alas, I > just had to compile Python 3.7 on a system running a 2.6 kernel last month at > a client site. Based on

[issue38870] Expose ast.unparse in the ast module

2019-11-20 Thread Pablo Galindo Salgado
New submission from Pablo Galindo Salgado : As discussed in https://mail.python.org/archives/list/python-...@python.org/thread/JAQDBMC23HW2PQ27HQNJ7G244T423IDD/ I propose to expose the unparse.py tool as part of the standard library in the ast module. The exposed function will have the

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +16793 pull_request: https://github.com/python/cpython/pull/17301 ___ Python tracker ___

[issue38869] Unexpectedly variable result

2019-11-20 Thread Zachary Ware
Zachary Ware added the comment: I don't see anything here to indicate a bug in CPython or the standard library, so I'm closing the issue. I would recommend asking for help with this on python-l...@python.org (comp.lang.python) or StackOverflow. If you can reduce it to a bug in either

[issue38869] Unexpectedly variable result

2019-11-20 Thread Ian Carr-de Avelon
New submission from Ian Carr-de Avelon : I can't understand why the result of changes() in the example file changes. I get: [[6.90642211e-310] [1.01702662e-316] [1.58101007e-322]] [[0.] [0.] [0.]] with an Ubuntu 14 system that has had a lot of changes made. I've checked the same happens

[issue38864] dbm: Can't open database with bytes-encoded filename

2019-11-20 Thread John Goerzen
John Goerzen added the comment: As has been pointed out to me, the surrogateescape method could be used here; however, it is a bit of an odd duckling itself, and the system's open() call accepts bytes; couldn't this as well? -- ___ Python tracker

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread David Cuthbert
David Cuthbert added the comment: On the completely deprecate reuse_address and rewrite/force folks to use reuse_port proposals, I'm a bit dubious of this approach. Right now, we have two knobs that directly correspond to (potential) kernel-level socket parameters, SO_REUSEADDR and

[issue38848] compileall fails when the platform lacks a functional sem_open()

2019-11-20 Thread Xavier de Gaye
Change by Xavier de Gaye : -- keywords: +patch pull_requests: +16792 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/17300 ___ Python tracker

[issue38841] [asyncio] bind() on a unix socket raises PermissionError on Android for a non-root user

2019-11-20 Thread Xavier de Gaye
Change by Xavier de Gaye : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: +Python 3.8 ___ Python tracker ___

[issue38853] set.repr breaches docstring contract

2019-11-20 Thread Steven D'Aprano
Steven D'Aprano added the comment: > My confusion stems from the fact that I expected the unpacking of a > set to return the same output as that obtained from the unpacking of a > list. Why did you expect that? Sets aren't lists. Lists are ordered, so they hold their items in a specific

[issue37228] UDP sockets created by create_datagram_endpoint() allow by default multiple processes to bind the same port

2019-11-20 Thread David Cuthbert
David Cuthbert added the comment: Jukka -- Fair enough; will reword this a bit. I'm trying to keep the DeprecationWarning short enough so people's eyes don't glaze over; will see what wordsmithing I can do here. (Once you get beyond a certain length, the number of folks who actually read

[issue38841] [asyncio] bind() on a unix socket raises PermissionError on Android for a non-root user

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset b762375ba28893be73f166576aecde44ad2f4001 by Miss Islington (bot) in branch '3.8': bpo-38841: Skip asyncio test_create_datagram_endpoint_existing_sock_unix (GH-17294)

[issue38868] Shutil cannot delete a folder that contains an .ini file

2019-11-20 Thread Svetlana Vodianova
Svetlana Vodianova added the comment: I posted my problem on Stack Overflow, but as of right now haven't received any answers. Link: https://stackoverflow.com/questions/58922332/shutil-cannot-delete-a-folder-with-a-hidden-desktop-ini-file To summarize my problem: If you use copytree to

[issue38848] compileall fails when the platform lacks a functional sem_open()

2019-11-20 Thread Xavier de Gaye
Xavier de Gaye added the comment: Changing the title to "compileall fails when the platform lacks a functional sem_open()" as the problem lies in the compileall module itself. Nosying Antoine as the author of issue #36786. compileall fails on android API 24:

[issue38868] Shutil cannot delete a folder that contains an .ini file

2019-11-20 Thread Svetlana Vodianova
New submission from Svetlana Vodianova : I posted my problem on Stack Overflow, but as of right now haven't received any answers. Link: https://stackoverflow.com/questions/58922332/shutil-cannot-delete-a-folder-with-a-hidden-desktop-ini-file To summarize my problem: If you use copytree to

[issue38841] [asyncio] bind() on a unix socket raises PermissionError on Android for a non-root user

2019-11-20 Thread miss-islington
Change by miss-islington : -- pull_requests: +16791 pull_request: https://github.com/python/cpython/pull/17299 ___ Python tracker ___

[issue38841] [asyncio] bind() on a unix socket raises PermissionError on Android for a non-root user

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset 559bad1a70ed50cc9caa7cb303b6ac1fe6a34af3 by Miss Islington (bot) (xdegaye) in branch 'master': bpo-38841: Skip asyncio test_create_datagram_endpoint_existing_sock_unix (GH-17294)

[issue38843] Document argparse behaviour when custom namespace object already has the field set

2019-11-20 Thread Raymond Hettinger
Change by Raymond Hettinger : -- assignee: docs@python -> rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38823] Improve stdlib module initialization error handling.

2019-11-20 Thread Brandt Bucher
Change by Brandt Bucher : -- pull_requests: +16790 pull_request: https://github.com/python/cpython/pull/17298 ___ Python tracker ___

[issue38546] test_concurrent_futures: reap_children() warnings on RHEL7 and RHEL8 buildbots

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: AMD64 Debian PGO 3.x: https://buildbot.python.org/all/#/builders/47/builds/3851 test_submit_keyword (test.test_concurrent_futures.ProcessPoolSpawnProcessPoolExecutorTest) ... 0.62s Warning -- reap_children() reaped child process 32627 Warning --

[issue33125] Windows 10 ARM64 platform support

2019-11-20 Thread Steve Dower
Steve Dower added the comment: I also filed issue38867 specifically for enabling Tkinter on Windows ARM. -- ___ Python tracker ___

[issue38867] Enable Tkinter on Windows ARM

2019-11-20 Thread Steve Dower
New submission from Steve Dower : (Split out from issue33125) We currently do not have ARM/ARM64 builds of Tcl/Tk for Windows, so we cannot enable tkinter on this platform. When builds of the dependencies become possible (in 8.6.10, apparently), we can enable it again. --

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: New changeset b93f31fcd9f10b213c614d4944baf9ca2df2016c by Victor Stinner in branch 'master': bpo-38858: Fix Py_Finalize() when called from a subinterpreter (GH-17297) https://github.com/python/cpython/commit/b93f31fcd9f10b213c614d4944baf9ca2df2016c

[issue33125] Windows 10 ARM64 platform support

2019-11-20 Thread Steve Dower
Steve Dower added the comment: I've merged the change to master, but note that the release is not enabled, so we aren't actually releasing ARM64 builds yet (though they will be compiled [but not tested] in PRs - nothing should be different from other Windows builds though). There's no harm

[issue33125] Windows 10 ARM64 platform support

2019-11-20 Thread Steve Dower
Steve Dower added the comment: New changeset de148f263fba75cd10d2cb010fe9c495cee4ec83 by Steve Dower in branch 'master': bpo-33125: Add support for building and releasing Windows ARM64 packages (GH-16828) https://github.com/python/cpython/commit/de148f263fba75cd10d2cb010fe9c495cee4ec83

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: Yes, IMHO, but this code is related to the http.server.CGIHTTPRequestHandler. This code looks like to be executed on the http.server.CGIHTTPRequestHandler not the apache server. According to docs, This defaults to ['/cgi-bin', '/htbin'] and describes

[issue28533] Replace asyncore

2019-11-20 Thread Jackson Riley
Jackson Riley added the comment: Lib/test/test_pyclbr.py - subissue issue38866 A trivial one! -- nosy: +jacksonriley ___ Python tracker ___

[issue38866] test_pyclbr replace asyncore

2019-11-20 Thread Jackson Riley
New submission from Jackson Riley : sub-issue of (issue28533)[https://bugs.python.org/issue28533] -- components: Tests files: pyclbr.patch keywords: patch messages: 357087 nosy: jacksonriley priority: normal severity: normal status: open title: test_pyclbr replace asyncore Added file:

[issue38765] `ast.AST._attributes` is used by `ast.dump()` but not documented

2019-11-20 Thread Brett Cannon
Brett Cannon added the comment: It's also possible the leading underscore was used to prevent name collisions. Unfortunately I don't know the history of _attributes, so without someone digging through the git history or personally knowing the reason behind it I don't know whether we should

[issue38500] Provide a way to get/set PyInterpreterState.frame_eval without needing to access interpreter internals

2019-11-20 Thread Brett Cannon
Brett Cannon added the comment: OK, then my vote is to provide getter and setter methods that are underscore-prefixed to mark them as "internal" with clear comments specifying that they are part of PEP 523 and thus should not be needlessly broken in a bugfix release. --

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +16789 pull_request: https://github.com/python/cpython/pull/17297 ___ Python tracker ___

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Siwon Kang added the comment: Thank you for your message and the info about 21323. I agree with the idea that cgi files are conventionally placed at the cgi-bin of the root but there is no explicit regulation so other servers, apache for example, handle this kind of sub directories

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: New changeset fff7bbfdb6b7c143b73b6b4b6b40e828c101110c by Victor Stinner in branch 'master': bpo-38858: Add _Py_IsMainInterpreter(tstate) (GH-17293) https://github.com/python/cpython/commit/fff7bbfdb6b7c143b73b6b4b6b40e828c101110c --

[issue38840] incorrect __all__ list in multiprocessing.managers module

2019-11-20 Thread Xavier de Gaye
Change by Xavier de Gaye : -- keywords: +patch pull_requests: +16788 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/17296 ___ Python tracker

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: In addition, the code is not about the path on the file system, but about the web request path. https://bugs.python.org/msg216960 will help you to understand. -- ___ Python tracker

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Dong-hee Na
Dong-hee Na added the comment: CGI programs are stored in a directory which must be configured in the web server. The path is typically SERVER_ROOT/cgi-bin, so the URL looks like http://www.domain/cgi-bin/script So IMHO, is_cgi's assumption is correct. IMHO, this is not the wrong code.

[issue38865] Can Py_Finalize() be called if the current interpreter is not the main interpreter?

2019-11-20 Thread STINNER Victor
New submission from STINNER Victor : Programs/_testembed.c contains the following test used by test_embed: static int test_audit_subinterpreter(void) { Py_IgnoreEnvironmentFlag = 0; PySys_AddAuditHook(_audit_subinterpreter_hook, NULL); _testembed_Py_Initialize();

[issue38853] set.repr breaches docstring contract

2019-11-20 Thread Cat Chenal
Cat Chenal added the comment: Thank you for pointing out my lack of clarity: I apologize. I probably should not have split this issue in two (issue38855). My confusion stems from the fact that I expected the unpacking of a set to return the same output as that obtained from the unpacking of

[issue38841] [asyncio] bind() on a unix socket raises PermissionError on Android for a non-root user

2019-11-20 Thread Xavier de Gaye
Change by Xavier de Gaye : -- keywords: +patch pull_requests: +16787 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/17294 ___ Python tracker

[issue38864] dbm: Can't open database with bytes-encoded filename

2019-11-20 Thread John Goerzen
New submission from John Goerzen : This simple recipe fails: >>> import dbm >>> dbm.open(b"foo") Traceback (most recent call last): File "", line 1, in File "/usr/lib/python3.7/dbm/__init__.py", line 78, in open result = whichdb(file) if 'n' not in flag else None File

[issue38858] new_interpreter() should reuse more Py_InitializeFromConfig() code

2019-11-20 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +16786 pull_request: https://github.com/python/cpython/pull/17293 ___ Python tracker ___

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- keywords: +patch pull_requests: +16785 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17292 ___ Python tracker ___

[issue34716] MagicMock.__divmod__ should return a pair

2019-11-20 Thread Jackson Riley
Change by Jackson Riley : -- keywords: +patch pull_requests: +16784 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17291 ___ Python tracker ___

[issue38692] add a pidfd child process watcher

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: As I already proposed previously, I proposed PR 17290 to simply skip test_pidfd_open() if os.pidfd_open() fails with a PermissionError. I don't propose to change os.pidfd_open(), only the test unit for practical reasons: not bother users who have to use a

[issue38692] add a pidfd child process watcher

2019-11-20 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +16783 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/17290 ___ Python tracker ___

[issue38692] add a pidfd child process watcher

2019-11-20 Thread STINNER Victor
STINNER Victor added the comment: I reopen the issue since the discussion restarted. -- status: closed -> open ___ Python tracker ___

[issue22107] tempfile module misinterprets access denied error on Windows

2019-11-20 Thread Erik Aronesty
Erik Aronesty added the comment: This is the fist of what I'm using: https://gist.github.com/earonesty/a052ce176e99d5a659472d0dab6ea361 Seems OK for my use cases. There's probably issues with relying on __del__ this way. But it solves the Windows close/reopen problem, too. --

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
New submission from Siwon Kang : is_cgi() in CGIHTTPRequestHandler class separates given path into (dir, rest) then checks if dir is in cgi_directories. However, it divides based on the first seen '/', multi-level directories like /sub/dir/cgi-bin/hello.py is divided into head=/sub,

[issue38576] CVE-2019-18348: CRLF injection via the host part of the url passed to urlopen()

2019-11-20 Thread Justin Capella
Justin Capella added the comment: Can't see the specifics of that "restricted" redhat bug, but this was interesting bug and I wanted to ask if perhaps the domain in such cases should be IDN / punycoded ://xn--n28h.ws/ for example is ://.la -- nosy: +b1tninja

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset 836f137f7ae0799de937e5281cb1da2bfdb8a69d by Miss Islington (bot) in branch '3.8': bpo-38821: Fix crash in argparse when using gettext (GH-17192) https://github.com/python/cpython/commit/836f137f7ae0799de937e5281cb1da2bfdb8a69d --

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread miss-islington
miss-islington added the comment: New changeset ecb2afc1bc1c2f3f9f99b09dd866100c3c7dcab7 by Miss Islington (bot) in branch '3.7': bpo-38821: Fix crash in argparse when using gettext (GH-17192) https://github.com/python/cpython/commit/ecb2afc1bc1c2f3f9f99b09dd866100c3c7dcab7 --

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread Tal Einat
Change by Tal Einat : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: +Python 3.7 ___ Python tracker ___

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread Tal Einat
Tal Einat added the comment: Thanks for the report and the PR with a fix, Federico! As discussed on in the PR comments, the PR did not include tests since there are currently no tests for argparse i18n. Please create a new issue for that, referencing this one. --

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread miss-islington
Change by miss-islington : -- pull_requests: +16782 pull_request: https://github.com/python/cpython/pull/17289 ___ Python tracker ___

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread miss-islington
Change by miss-islington : -- pull_requests: +16781 pull_request: https://github.com/python/cpython/pull/17288 ___ Python tracker ___

[issue38821] argparse calls ngettext with deprecated non-integer value

2019-11-20 Thread Tal Einat
Tal Einat added the comment: New changeset be5c79e0338005d675a64ba6e5b137e850d556d1 by Tal Einat (Federico Bond) in branch 'master': bpo-38821: Fix crash in argparse when using gettext (GH-17192) https://github.com/python/cpython/commit/be5c79e0338005d675a64ba6e5b137e850d556d1 --

  1   2   >