[issue33151] importlib.resources breaks on subdirectories

Change by Barry A. Warsaw : -- keywords: +patch pull_requests: +5993 stage: -> patch review ___ Python tracker ___

[issue33151] importlib.resources breaks on subdirectories

Barry A. Warsaw added the comment: is_resource() from zip submodules is also affected -- ___ Python tracker ___

[issue33151] importlib.resources breaks on subdirectories

New submission from Barry A. Warsaw : Found a bug when trying to read a resource from a subpackage in a zip file. I was actually surprised we didn't have a test for this AFAICT, and when I added one, it did fail. I have a PR coming soon. -- assignee: barry

[issue33111] Merely importing tkinter breaks parallel code (multiprocessing, sharedmem)

Ethan Welty added the comment: Terry Reedy: I just tried your suggestion of a main clause (code below) and it made no difference. ``` import _tkinter import numpy as np # multiprocessing.set_start_method("spawn") import multiprocessing def parallel_matmul(x): R =

[issue33144] random._randbelow optimization

Tim Peters added the comment: I'm the wrong guy to ask about that. Since I worked at Zope Corp, my natural inclination is to monkey-patch everything - but knowing full well that will offend everyone else ;-) That said, this optimization seems straightforward to me: two

[issue33149] Parser stack overflows

Change by Pablo Galindo Salgado : -- resolution: -> duplicate ___ Python tracker ___

[issue33149] Parser stack overflows

Change by Pablo Galindo Salgado : -- superseder: -> s_push: parser stack overflow MemoryError ___ Python tracker ___

[issue33144] random._randbelow optimization

Raymond Hettinger added the comment: > the optimized `_randbelow()` also avoids populating its locals > with 5 unused formal arguments Yes, that clean-up would be nice as well :-) Any thoughts on having __init__ set a flag versus using __init__subclass__ to

[issue33149] Parser stack overflows

Pablo Galindo Salgado added the comment: Duplicate of issue3971 -- nosy: +pablogsal ___ Python tracker ___

[issue30100] WeakSet should allow discard and remove on items that can't have weak references

Raymond Hettinger added the comment: > my original motivating case was objects that are hashable but not weakly > referenceable. It is probably not good design to extend discard() to handle types that aren't handled by the other methods. To me, it isn't at all

[issue33144] random._randbelow optimization

Tim Peters added the comment: I don't see anything objectionable about the class optimizing the implementation of a private method. I'll note that there's a speed benefit beyond just removing the two type checks in the common case: the optimized `_randbelow()` also avoids

[issue33150] Signature error for methods of class configparser.Interpolation

New submission from Arno-Can Uestuensoez : I am not sure whether this is already covered by an issue, it is present in 3.6.2 and 3.6.4. The class Interpolation in the configparser module causes an exception: File

[issue27212] Doc for itertools, 'islice()' implementation have unwanted behavior for recipe 'consume()'

Raymond Hettinger added the comment: New changeset c8698cff7ccc8dc730c58523c7ef4113ea8d3049 by Raymond Hettinger (Miss Islington (bot)) in branch '3.6': bpo-27212: Modify islice recipe to consume initial values preceding start (GH-6195) (GH-6267)

[issue27212] Doc for itertools, 'islice()' implementation have unwanted behavior for recipe 'consume()'

Raymond Hettinger added the comment: New changeset f328caf4caafd4521c356af8cb8a299f27603c90 by Raymond Hettinger (Miss Islington (bot)) in branch '3.7': bpo-27212: Modify islice recipe to consume initial values preceding start (GH-6195) (#GH-6266)

[issue27212] Doc for itertools, 'islice()' implementation have unwanted behavior for recipe 'consume()'

Change by miss-islington : -- pull_requests: +5992 ___ Python tracker ___

[issue27212] Doc for itertools, 'islice()' implementation have unwanted behavior for recipe 'consume()'

Change by miss-islington : -- pull_requests: +5991 ___ Python tracker ___

[issue27212] Doc for itertools, 'islice()' implementation have unwanted behavior for recipe 'consume()'

Raymond Hettinger added the comment: New changeset da1734c58d2f97387ccc9676074717d38b044128 by Raymond Hettinger (Cheryl Sabella) in branch 'master': bpo-27212: Modify islice recipe to consume initial values preceding start (GH-6195)

[issue33144] random._randbelow optimization

Raymond Hettinger added the comment: > it could also be done at instantiation time (the attached patch > uses __init_subclass__ for that purpose FWIW, a 10-25% speedup is only possible because the remaining code is already somewhat fast. All that is being

[issue33148] RuntimeError('Event loop is closed') after cancelling getaddrinfo and closing loop

Nathaniel Smith added the comment: Yeah, getaddrinfo isn't actually cancellable (because it's a blocking call that has to be run in a thread), but it's side-effect-free so if the user requests that it be cancelled then it's safe to return immediately while it keeps running in

[issue33114] random.sample() behavior is unexpected/unclear from docs

Raymond Hettinger added the comment: > However, let me argue my case one more time, and if you still disagree, feel > free to close this. Thank you for expressing your ideas so clearly. However, I'm going to go with Tim's recommendation and leave the docs as

[issue33149] Parser stack overflows

Isaac Elliott added the comment: Because of the way recursive descent parsing works, [[ is actually the minimal input required to reproduce this in python3. In python2, the bug

[issue33149] Parser stack overflows

New submission from Isaac Elliott : python3's parser stack overflows on deeply-nested expressions, for example:

[issue33148] RuntimeError('Event loop is closed') after cancelling getaddrinfo and closing loop

New submission from Vitaly Kruglikov : I see this exception on the terminal: ``` exception calling callback for Traceback (most recent call last): File

[issue32873] Pickling of typing types

Change by Ivan Levkivskyi : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue32873] Pickling of typing types

miss-islington added the comment: New changeset d0e04c82448c750d4dc27f2bddeddea74bd353ff by Miss Islington (bot) in branch '3.7': bpo-32873: Treat type variables and special typing forms as immutable by copy and pickle (GH-6216)

[issue32873] Pickling of typing types

Change by miss-islington : -- pull_requests: +5990 ___ Python tracker ___

[issue32873] Pickling of typing types

Ivan Levkivskyi added the comment: New changeset 834940375ae88bc95794226dd8eff1f25fba1cf9 by Ivan Levkivskyi in branch 'master': bpo-32873: Treat type variables and special typing forms as immutable by copy and pickle (GH-6216)

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

Change by Alexey Izbyshev : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

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

miss-islington added the comment: New changeset 57db13e582ad269d6e067fe934122207cc992739 by Miss Islington (bot) in branch '3.6': bpo-32844: Fix a subprocess misredirection of a low fd (GH5689)

[issue33146] contextlib.suppress should capture exception for inspection and filter on substrings

Raymond Hettinger added the comment: What kind useful information to you expect to get before an exception is raised? For example: with suppress(FileNotFoundError) as e: os.remove(somefile) What could *e* possibly be that would be useful. AFAICT,

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

miss-islington added the comment: New changeset 05455637f3ba9bacd459700f4feab783e5967d69 by Miss Islington (bot) in branch '3.7': bpo-32844: Fix a subprocess misredirection of a low fd (GH5689)

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

Change by miss-islington : -- pull_requests: +5989 ___ Python tracker ___

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

Change by miss-islington : -- pull_requests: +5988 ___ Python tracker ___

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

Gregory P. Smith added the comment: New changeset 0e7144b064a19493a146af94175a087b3888c37b by Gregory P. Smith (Alexey Izbyshev) in branch 'master': bpo-32844: Fix a subprocess misredirection of a low fd (GH5689)

[issue33128] PathFinder is twice on sys.meta_path

Change by Brett Cannon : -- keywords: +3.7regression nosy: +brett.cannon ___ Python tracker ___

[issue4819] Misc/cheatsheet needs updating

Mark Dickinson added the comment: Given that Misc/cheatsheet doesn't exist any more in the Python 3 repo, and Python 2 is nearing end-of-life, I suspect this should just be closed. -- stage: needs patch -> resolved status: pending -> closed

[issue1704621] interpreter crash when multiplying large lists

Guido van Rossum added the comment: @imz Please file a new issue. -- ___ Python tracker ___

[issue1704621] interpreter crash when multiplying large lists

Ivan Zakharyaschev added the comment: Hi! This is broken for tuples (but not for lists, as in the example here) in 2.7.14 for me. Should we reopen this issue and fix it better? [builder@localhost ~]$ python Python 2.7.14 (default, Nov 7 2017, 17:07:17) [GCC 6.3.1

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: Thanks for the bug report. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

miss-islington added the comment: New changeset c6147acd2ce5fa9e344f179b539f3b21b9ae1a6d by Miss Islington (bot) in branch '3.7': bpo-33141: Have dataclasses.Field pass through __set_name__ to any default argument. (GH-6260)

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

Rolf Eike Beer added the comment: So, what is the problem with this? Either the compiler knows that unaligned accesses are no problem and optimizes them away anyway, or it is kept because it would crash otherwise. I can confirm that no sparc version >= 3.5 (have not tried

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Change by miss-islington : -- pull_requests: +5987 ___ Python tracker ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: New changeset de7a2f04d6b9427d568fcb43b6f512f9b4c4bd84 by Eric V. Smith in branch 'master': bpo-33141: Have dataclasses.Field pass through __set_name__ to any default argument. (GH-6260)

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: Looks great to me. Thanks! -- ___ Python tracker ___ ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: I've updated the PR. I left those lines in, and added a different test. After all, it does need to work with real descriptors. -- ___ Python tracker

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: Yeah and I think lines 2709-2712 of TestDescriptors also needs removed. -- ___ Python tracker ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: Yes, I noticed the same thing. I'll remove the test. -- ___ Python tracker ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: Eric, looking at the PR; note that if you do this for the __set_name__ check: if inspect.ismethoddescriptor(self.default): ...an object like the one below will not get its __set_name__ called, even though PEP 487 says it should: class

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

Change by Rolf Eike Beer : -- pull_requests: +5986 ___ Python tracker ___ ___

[issue33147] Update references for RFC 3548 to RFC 4648

Paul Hoffman added the comment: I signed the contributor agreement form on 2017-06-23 -- ___ Python tracker ___

[issue33145] unaligned accesses in siphash24() lead to crashes on sparc

New submission from Serhiy Storchaka : Actually this looks like a duplicate of issue28055. -- nosy: +serhiy.storchaka resolution: -> duplicate stage: -> resolved status: open -> closed superseder: -> pyhash's siphash24 assumes alignment of the data

[issue33147] Update references for RFC 3548 to RFC 4648

New submission from Paul Hoffman : serhiy-storchaka asked me to open an issue about whether Python implements RFC 4648. As far as I can tell it does, correctly, for the parts of RFC 4648 covered in the doc. My PR was about simply updating a reference to an RFC that was

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: I don't expect there to be too much usage of Field objects, so I'm not so worried about it. If you can, try out the code in the PR. Thanks. -- ___ Python tracker

[issue33146] contextlib.suppress should capture exception for inspection and filter on substrings

New submission from Jason R. Coombs : I propose the following expansion of the interface of contextlib.suppress. Currently, when entering the context, suppress returns None. Instead, it could return an object that provides some detail about the exception. Inspiration for an

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Change by Eric V. Smith : -- keywords: +patch pull_requests: +5984 stage: -> patch review ___ Python tracker ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: I suppose one downside of that solution is __set_name__ will be called for every Field whether or not it is need. Probably can't be helped without major complications. -- ___ Python tracker

[issue33145] unaligned accesses in siphash24() lead to crashes on sparc

Change by Rolf Eike Beer : -- components: Library (Lib) nosy: Dakon priority: normal pull_requests: 5983 severity: normal status: open title: unaligned accesses in siphash24() lead to crashes on sparc versions: Python 3.5, Python 3.6, Python 3.7, Python 3.8

[issue31920] pygettext ignores directories as inputfile argument

Change by Serhiy Storchaka : -- pull_requests: +5982 stage: needs patch -> patch review ___ Python tracker ___

[issue24356] venv documentation incorrect / misleading

Change by Steven Downum : -- nosy: +steven.downum ___ Python tracker ___ ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: Sounds like a relatively easy solution then. Hopefully it makes the beta 3 so I can use it immediately- thanks! -- ___ Python tracker

[issue31201] make test: module test that failed doesn't exist

Change by Cheryl Sabella : -- keywords: +easy stage: -> needs patch title: module test that failed doesn't exist -> make test: module test that failed doesn't exist type: -> enhancement versions: +Python 3.7, Python 3.8 ___

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: Nick Coghlan suggested (on python-dev) to have Field implement __set_name__ and call through to the default value, if it exists. That approach seems to work fine. I'll generate a PR with tests sometime before today's release. --

[issue6986] _json crash on scanner/encoder initialization error

Serhiy Storchaka added the comment: New changeset 7c2d97827fd2ccd72ab7a98427f87fcfe3891644 by Serhiy Storchaka (Oren Milman) in branch 'master': bpo-6986: Add a comment to clarify a test of _json.make_encoder(). (GH-3789)

[issue31504] Documentation for return value for string.rindex is missing when search string is empty

Change by Serhiy Storchaka : -- status: open -> pending ___ Python tracker ___

[issue33144] random._randbelow optimization

New submission from Wolfgang Maier : Given that the random module goes a long way to ensure optimal performance, I was wondering why the check for a match between the random and getrandbits methods is performed per call of Random._randbelow, when it

[issue33114] random.sample() behavior is unexpected/unclear from docs

Scott Eilerman added the comment: Raymond, Tim, thanks for your replies so far. I understand (and for the most part, agree with) your points about not being able to list every behavior, and not wanting to cause uncertainty in users. However, let me argue my case

[issue24925] Allow doctest to find line number of __test__ strings if formatted as a triple quoted string.

Sanyam Khurana added the comment: Hey Jurjen, Do you mind converting your patch in a Pull request please? -- nosy: +CuriousLearner ___ Python tracker

[issue30100] WeakSet should allow discard and remove on items that can't have weak references

Change by Serhiy Storchaka : -- status: open -> pending ___ Python tracker ___

[issue24024] str.__doc__ needs an update

Serhiy Storchaka added the comment: The current docstring is correct. The signature str(object, encoding="utf-8", errors="strict") is not correct, because str(object) is not the same as str(object, encoding="utf-8", errors="strict"). >>> str([1, 2]) '[1, 2]' >>>

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Rick Teachey added the comment: hmmm... if I check the C.d class attribute it seems to return the descriptor instance object (not a field object) before any C instances have been created. i guess this is just a part of how the dataclass implementation works. i didn't realize

[issue33143] encode UTF-16 generates unexpected results

Anders Rundgren added the comment: Thanx for the superquick response! I really appreciate it. I'm obviously a Python n00b Anders -- ___ Python tracker

[issue33042] New 3.7 startup sequence crashes PyInstaller

Nick Coghlan added the comment: Just noting that while I've closed this issue as fixed, I think we still have quite a bit of work to do to get our handling of these pre-init config settings to a place we're genuinely happy with (or as happy as backwards compatibility

[issue33042] New 3.7 startup sequence crashes PyInstaller

Change by Nick Coghlan : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue33042] New 3.7 startup sequence crashes PyInstaller

Nick Coghlan added the comment: Harmut, thanks for the status update! Victor, the problem is that both sys.warnoptions and sys._xoptions get read by Py_Initialize(), so setting them afterwards is essentially pointless - while you do still change the contents of the sys

[issue20082] Misbehavior of BufferedRandom.write with raw file in append mode

Erik Bray added the comment: I keep forgetting about this (given that it's like 5 years old now). Let me see if I can make a new PR for it... -- ___ Python tracker

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: I suppose I could, when overwriting the class member, check for inspect.ismethoddescriptor and call __set_name__ myself. -- components: +Library (Lib) versions: +Python 3.8 ___ Python tracker

[issue33141] descriptor __set_name__ feature broken for dataclass descriptor fields

Eric V. Smith added the comment: That's a tough one. Because C.d is not set to a descriptor at type creation time (it's set to a Field object), the __set_name__ behavior is never invoked. It's when the @dataclass decorator is called that C.d is set to D().

[issue33143] encode UTF-16 generates unexpected results

Serhiy Storchaka added the comment: b'\x20' is the same as b' '. -- nosy: +serhiy.storchaka resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker

[issue33122] ftplib: FTP_TLS seems to have problems with sites that close the encrypted channel themselfes

Jürgen added the comment: Hi, thanks for tanking care of this issue. I am mainly working on a windows client and connect to a z/OS host. Attached you find the ftplib.py I patched to workaround this. Here is the output of the list command which ends up in an exception

[issue33143] encode UTF-16 generates unexpected results

New submission from Anders Rundgren : Python 3.5.1 (v3.5.1:37a07cee5969, Dec 6 2015, 01:54:25) [MSC v.1900 64 bit (AMD64)] on win32 Type "help", "copyright", "credits" or "license" for more information. >>> v = '\u20ac' >>> print (v) € >>> v.encode('utf-16')

[issue31947] names=None case is not handled by EnumMeta._create_ method

Change by Serhiy Storchaka : -- components: +Library (Lib) nosy: +barry, eli.bendersky, ethan.furman stage: -> patch review versions: -Python 3.4, Python 3.5 ___ Python tracker

[issue33142] Fatal Python error: Py_Initialize: Unable to get the locale encoding on Debian/Python 3.6.4 source build

Change by Andreas Jung : -- components: +Installation ___ Python tracker ___

[issue33142] Fatal Python error: Py_Initialize: Unable to get the locale encoding on Debian/Python 3.6.4 source build

Change by Andreas Jung : -- title: Fatal Python error: Py_Initialize: Unable to get the locale encoding -> Fatal Python error: Py_Initialize: Unable to get the locale encoding on Debian/Python 3.6.4 source build ___

[issue33142] Fatal Python error: Py_Initialize: Unable to get the locale encoding

New submission from Andreas Jung : Unable to build Python 3.6.4 from sources on a fresh Debian system: @plone /tmp/Python-3.6.4 $ cat /etc/os-release PRETTY_NAME="Debian GNU/Linux 9 (stretch)" NAME="Debian GNU/Linux" VERSION_ID="9" VERSION="9 (stretch)" ID=debian

[issue24132] Direct sub-classing of pathlib.Path

Change by qb-cea : -- keywords: +patch pull_requests: +5981 stage: -> patch review ___ Python tracker ___

[issue33096] ttk.Treeview.insert() does not allow to insert item with "False" iid

miss-islington added the comment: New changeset 45116d393f713bbe918f16b33a0bba28b15bc96b by Miss Islington (bot) in branch '3.6': bpo-33096: Fix ttk.Treeview.insert. (GH-6228)

[issue33096] ttk.Treeview.insert() does not allow to insert item with "False" iid

miss-islington added the comment: New changeset a7b1b847f665aafc22557917cea32ec34c9b4418 by Miss Islington (bot) in branch '3.7': bpo-33096: Fix ttk.Treeview.insert. (GH-6228)

[issue33096] ttk.Treeview.insert() does not allow to insert item with "False" iid

Change by miss-islington : -- pull_requests: +5980 ___ Python tracker ___

[issue33096] ttk.Treeview.insert() does not allow to insert item with "False" iid

Change by miss-islington : -- pull_requests: +5979 ___ Python tracker ___

[issue33096] ttk.Treeview.insert() does not allow to insert item with "False" iid

Serhiy Storchaka added the comment: New changeset 3ab44c0783eebdff687014f7d14d5dec59b6bd39 by Serhiy Storchaka (Garvit Khatri) in branch 'master': bpo-33096: Fix ttk.Treeview.insert. (GH-6228)