[issue33322] Overridden __getitem__ not called on use of slice syntax when inheriting from tuple

New submission from Jacob Thalman : class MyTuple(tuple): def __getitem__(self, item): print "Getting {}".format(item) t = MyTuple((1, 2)) t[0] -> "Getting 0" t[1] -> "Getting 1" t[slice(None)] -> "Getting slice(None, None, None)" t[:] -> (1, 2) t[slice(None, 1)] -> "Getting slice(No

[issue33277] Deprecate __loader__, __package__, __file__, and __cached__ on modules

Nick Coghlan added the comment: As a concrete proposal for 3.8, how about: 1. Add the following utility functions to importlib.util (names adjusted to match PEP 451 rather than my initial suggestions above): def get_location(module): try: return module.__file__

[issue33318] Move folding tuples of constants into compiler.c from peephole.c

Nick Coghlan added the comment: If I recall Eugene Toder's previous AST-level compilation patches correctly, they handle this problem by putting the AST optimisation step *after* the output of the Py_CF_ONLY_AST step. This meant that if you're running the source->AST step explicitly, you stil

[issue33131] Upgrade to pip 10 for Python 3.7

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

[issue33309] Unittest Mock objects do not freeze arguments they are called with

Change by Terry J. Reedy : -- nosy: +ezio.melotti, michael.foord, rbcollins ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue33303] ElementTree Comment text isn't escaped

Change by Terry J. Reedy : -- versions: -Python 3.4, Python 3.5 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue33289] tkinter askcolor returning floats for r, g, b values instead of ints

Terry J. Reedy added the comment: For future reference, 3.4 and 3.5 only get security fixes. The quoted line of code is from the main part of tkinter.colorchooser,Chooser._fixresult: r, g, b = widget.winfo_rgb(result) return (r/256, g/256, b/256), str(result) where tkinter.Mis

[issue33284] Increase test coverage for numbers.py

Change by Terry J. Reedy : -- stage: patch review -> needs patch type: -> enhancement ___ Python tracker ___ ___ Python-bugs-list ma

[issue33284] Increase test coverage for numbers.py

Terry J. Reedy added the comment: Barry, thank you for your first submission. You propose to test numbers.Complex.__bool__ def __bool__(self): """True if self != 0. Called for bool(self).""" return self != 0 by adding the following to Lib/test/test_abstract_numbers. +

[issue33274] minidom removeAttributeNode returns None

Ari Krupnik added the comment: I guess the main question if whether minidom wants to adhere to the standard or be pythonic, and it's not up to me to decide, although personally I like standards. The common use case for DOM functions returning the relevant nodes is for the caller to chain cal

[issue33275] glob.glob should explicitly note that results aren't sorted

Terry J. Reedy added the comment: How about adding a sentence to the end of the first paragraph. glob.glob(pathname, *, recursive=False) Return a possibly-empty list of path names that match pathname, which must be a string containing a path specification. pathname can be either absolute

[issue33274] minidom removeAttributeNode returns None

Terry J. Reedy added the comment: It is standard in the Python stdlib that mutation methods usually return None and never echo an input argument. If one can pass a node to element.removeAttributeNode(node), there is no need to echo it back. So I suspect that the current behavior is intended

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: Sorry for the breakage. I'm glad the fix was that greppable and easy given I had no idea where to start otherwise. :) This is in for 3.7. The PR for 3.6 is pending, i want to let it out in the next 3.7 builds for a while before i merge it into 3.6 even th

[issue33321] Add a Linux clang ubsan undefined behavior sanitizer buildbot

New submission from Gregory P. Smith : CPython needs to be regularly built and vetted for C undefined behavior. CFLAGS=-fsanitize=undefined LDFLAGS=-fsanitize=undefined CC=clang LD=clang ./configure is all it takes to start with. if you do this today, test_ctypes will knock your socks off wit

[issue33276] Clarify that __path__ can't be set to just anything

Change by miss-islington : -- pull_requests: +6252 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33276] Clarify that __path__ can't be set to just anything

Change by miss-islington : -- pull_requests: +6253 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33276] Clarify that __path__ can't be set to just anything

Change by Brett Cannon : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___ P

[issue33131] Upgrade to pip 10 for Python 3.7

miss-islington added the comment: New changeset 47d57bfd3d6ceb08f98fcff14fe3c8bb41ca0664 by Miss Islington (bot) in branch '3.6': bpo-33131: Upgrade ensurepip to bundle pip 10.0.1 (GH-6546) https://github.com/python/cpython/commit/47d57bfd3d6ceb08f98fcff14fe3c8bb41ca0664 --

[issue33254] Have importlib.resources.contents() return an interable instead of an iterator

Change by Brett Cannon : -- title: importlib.resources.contents() incorrectly yields an empty list -> Have importlib.resources.contents() return an interable instead of an iterator ___ Python tracker

[issue991266] Cookie.py does not correctly quote Morsels

Change by Berker Peksag : -- pull_requests: +6251 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue28627] [alpine] shutil.copytree fail to copy a direcotry with broken symlinks

Martin Panter added the comment: This looks like it may be covered by Issue 31940, about the “shutil.copystat” API. See Anthony’s initial proposal at . Max: I think you need the “else” branch to reraise the exception if “errno” doesn’t match. Per

[issue25427] Remove the pyvenv script in Python 3.8

Change by Brett Cannon : -- resolution: -> fixed stage: commit review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue25427] Remove the pyvenv script in Python 3.8

Brett Cannon added the comment: New changeset a8c342465b977115b88a4f130833a2fd4c4e7356 by Brett Cannon in branch 'master': bpo-25427: Remove pyvenv (GH-5962) https://github.com/python/cpython/commit/a8c342465b977115b88a4f130833a2fd4c4e7356 -- ___

[issue33276] Clarify that __path__ can't be set to just anything

Change by Brett Cannon : -- keywords: +patch pull_requests: +6250 stage: needs patch -> patch review ___ Python tracker ___ ___ Pytho

[issue33131] Upgrade to pip 10 for Python 3.7

miss-islington added the comment: New changeset 4caba7a94024d8af2eed26300f13ff4433dcb05b by Miss Islington (bot) in branch '2.7': bpo-33131: Upgrade ensurepip to bundle pip 10.0.1 (GH-6546) https://github.com/python/cpython/commit/4caba7a94024d8af2eed26300f13ff4433dcb05b --

[issue33131] Upgrade to pip 10 for Python 3.7

miss-islington added the comment: New changeset b221107d32370670905fefb569b3104748f92afe by Miss Islington (bot) in branch '3.7': bpo-33131: Upgrade ensurepip to bundle pip 10.0.1 (GH-6546) https://github.com/python/cpython/commit/b221107d32370670905fefb569b3104748f92afe -- nosy: +mi

[issue32886] new Boolean ABC in numbers module

Raymond Hettinger added the comment: > If I get you correctly, then the minimal `Boolean` ABC would only need > to contain the `__bool__` method, without the bitwise operations Once reduced to just having `__bool__`, I think there is no value added and would prefer to leave this out. We reall

[issue33144] random._randbelow optimization

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

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Ned Deily added the comment: Thanks, Greg, looking better! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue33131] Upgrade to pip 10 for Python 3.7

Change by miss-islington : -- pull_requests: +6249 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33277] Deprecate __loader__, __package__, __file__, and __cached__ on modules

Raymond Hettinger added the comment: +1 from me. The visual clutter in dir() has proven to be somewhat distracting in introduction-to-python courses. -- nosy: +rhettinger ___ Python tracker

[issue33131] Upgrade to pip 10 for Python 3.7

Ned Deily added the comment: Since 2.7 was upgraded, I think 3.6.x should be upgraded to 10.0.x, as well. -- versions: +Python 2.7, Python 3.6 ___ Python tracker ___ __

[issue33131] Upgrade to pip 10 for Python 3.7

Change by miss-islington : -- pull_requests: +6248 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33131] Upgrade to pip 10 for Python 3.7

Change by miss-islington : -- pull_requests: +6247 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33131] Upgrade to pip 10 for Python 3.7

Paul Moore added the comment: New changeset 0399cf9b5e370516e3d0aed6a63ff74aff5eadb5 by Paul Moore in branch 'master': bpo-33131: Upgrade ensurepip to bundle pip 10.0.1 (GH-6546) https://github.com/python/cpython/commit/0399cf9b5e370516e3d0aed6a63ff74aff5eadb5 -- ___

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: New changeset 11659d00b9185c8f02ea6b642fa475a80e21f1a9 by Gregory P. Smith (Miss Islington (bot)) in branch '3.7': bpo-33312: update Tools/gdb/libpython.py to match. (GH-6549) https://github.com/python/cpython/commit/11659d00b9185c8f02ea6b642fa475a80e21f1a9

[issue33318] Move folding tuples of constants into compiler.c from peephole.c

Antoine Pitrou added the comment: I'm not convinced by this. One of the points of separating the optimizer from the compiler is that people wanting to study/extend the compiler don't have to filter through optimization-related routines. Separating initial code generation from optimization p

[issue33318] Move folding tuples of constants into compiler.c from peephole.c

Raymond Hettinger added the comment: +0 It looks like there is a small net win. Conceptually, constant folding is better done upstream where more semantic information is available (rather than downstream when the meaning has to be deduced from the opcodes). The only drawback is that if unfo

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Change by miss-islington : -- pull_requests: +6246 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: New changeset 53f67d401df40486fd0fb8fbcf9da725cd37290c by Gregory P. Smith in branch 'master': bpo-33312: update Tools/gdb/libpython.py to match. (GH-6548) https://github.com/python/cpython/commit/53f67d401df40486fd0fb8fbcf9da725cd37290c -- __

[issue33315] Allow queue.Queue to be used in type annotations

Josh Rosenberg added the comment: None of the actual classes outside of the typing module support this either to my knowledge. You can't do: from collections import deque a: deque[int] nor can you do: a: list[int] Adding Queue to the typing module might make sense (feel free to

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Change by Gregory P. Smith : -- pull_requests: +6245 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue33319] `subprocess.run` documentation doesn't tell is using `stdout=PIPE` safe

Josh Rosenberg added the comment: If the goal is just to suppress stdout, that's what passing subprocess.DEVNULL is for (doesn't exist in Py2, but opening os.devnull and passing that is a slightly higher overhead equivalent). subprocess.run includes a call to communicate as part of its defaul

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: I see one line in Tools/gdb/libpython.py that may be related. i'll try changing that. the only way i have to _test_ it is to merge it into master. -- ___ Python tracker

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: ned: yes, it supposedly has. test_gdb. in a manner i cannot reproduce or debug without error output that indicates anything about the problem. test_gdb passes on my systems. -- ___ Python tracker

[issue21314] Document '/' in signatures

Change by Jakub Wilk : -- nosy: +jwilk ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: New changeset 392520bd78cd18639a27e5d2803c2e1c2bd593a8 by Gregory P. Smith (Miss Islington (bot)) in branch '3.7': bpo-33312: Fix clang ubsan out of bounds warnings in dict. (GH-6537) (GH-6543) https://github.com/python/cpython/commit/392520bd78cd18639a27e5d

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Ned Deily added the comment: Greg, this change has broken some buildbots: http://buildbot.python.org/all/#/builders/85/builds/934 http://buildbot.python.org/all/#/builders/21/builds/951 -- nosy: +ned.deily ___ Python tracker

[issue33277] Deprecate __loader__, __package__, __file__, and __cached__ on modules

Nick Coghlan added the comment: -1 from me for __package__, as setting that's the preferred way for directly executed scripts to fix their explicit relative imports (rather than trying to retroactively fix their spec). (https://docs.python.org/3/library/importlib.html also doesn't say anythin

[issue33312] ubsan undefined behavior sanitizer flags struct _dictkeysobject (PyDictKeysObj)

Gregory P. Smith added the comment: If you think this should be written differently, please propose it in a PR so we can see what you are suggesting. An unbounded member at the end of a struct is quite a common practice in C. ex: PyBytesObject -- _

[issue32232] building extensions as builtins is broken in 3.7

Nick Coghlan added the comment: Xavier's changes should fix the reported compile error, while keeping the increased isolation of the interpreter core from the optional extension modules. If the latter change causes a detectable performance regression, then I think that would make more sense a

[issue32232] building extensions as builtins is broken in 3.7

miss-islington added the comment: New changeset 18cd82815b9bdeabc5ac03aa7f58102109ccf368 by Miss Islington (bot) in branch '3.7': bpo-32232: by default, Setup modules are no longer built with -DPy_BUILD_CORE (GH-6489) https://github.com/python/cpython/commit/18cd82815b9bdeabc5ac03aa7f58102109

[issue33320] Clarify who can add the "skip issue" label on GitHub

Change by Andrés Delfino : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue32232] building extensions as builtins is broken in 3.7

Change by miss-islington : -- pull_requests: +6244 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue32232] building extensions as builtins is broken in 3.7

Nick Coghlan added the comment: New changeset 063db62aab4041ac47798e7e48b27b2f2bef21c5 by Nick Coghlan (xdegaye) in branch 'master': bpo-32232: by default, Setup modules are no longer built with -DPy_BUILD_CORE (GH-6489) https://github.com/python/cpython/commit/063db62aab4041ac47798e7e48b27b2

[issue33131] Upgrade to pip 10 for Python 3.7

Change by Paul Moore : -- keywords: +patch pull_requests: +6243 stage: needs patch -> patch review ___ Python tracker ___ ___ Python-

[issue32232] building extensions as builtins is broken in 3.7

Nick Coghlan added the comment: I also resolved my own puzzlement from the PR comments regarding why _weakrefs.c was fine with being built as a regular extension module: it's because the C level access API for __weakrefs__ is published through the regular C API, not as a core-only internal AP

[issue32232] building extensions as builtins is broken in 3.7

Xavier de Gaye added the comment: To answer my own comment in PR 6489 about the interference of the pyconfig.h macros with the build of std lib extension modules, the py_config_macros.py script prints the list of pyconfig.h.in macros that are being used by the standard library extension modul

[issue33320] Clarify who can add the "skip issue" label on GitHub

Andrés Delfino added the comment: This is the PR: https://github.com/python/devguide/pull/359 -- ___ Python tracker ___ ___ Python-b

[issue33320] Clarify who can add the "skip issue" label on GitHub

Change by Andrés Delfino : -- pull_requests: -6242 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue33320] Clarify who can add the "skip issue" label on GitHub

Change by Andrés Delfino : -- keywords: +patch pull_requests: +6242 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-li

[issue33264] Remove to-be-deprecated urllib.request.urlretrieve function reference from HOWTO

Senthil Kumaran added the comment: Indeed. Thank you! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue33320] Clarify who can add the "skip issue" label on GitHub

New submission from Andrés Delfino : IMHO, the wording of https://devguide.python.org/pullrequest/ is somewhat confusing regarding the "skip issue" label. I thought that I (with no commit access) should add the label, but, as far I understand, that is a responsibility of developers with commit

[issue33264] Remove to-be-deprecated urllib.request.urlretrieve function reference from HOWTO

Andrés Delfino added the comment: I believe this issue can be closed, right? -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue33298] Wrap only constants with _PyCode_ConstantKey() in the compiler.

Change by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue33318] Move folding tuples of constants into compiler.c from peephole.c

Change by Serhiy Storchaka : -- keywords: +patch pull_requests: +6241 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-

[issue33319] `subprocess.run` documentation doesn't tell is using `stdout=PIPE` safe

New submission from Pekka Klärck : I'm porting old scripts from Python 2.7 to 3.6 and plan to change `subprocess.call()` to `subprocess.run()` at the same time. When using `call()` I've used `tempfile.TemporaryFile` as stdout because it's documentation has this warning: Note: Do not use s

[issue33319] `subprocess.run` documentation doesn't tell is using `stdout=PIPE` safe

Change by Pekka Klärck : -- assignee: -> docs@python components: +Documentation nosy: +docs@python ___ Python tracker ___ ___ Python

[issue33298] Wrap only constants with _PyCode_ConstantKey() in the compiler.

Serhiy Storchaka added the comment: New changeset d70c2a6894d80410b6f5a3129d6b1062ea8fd4e4 by Serhiy Storchaka in branch 'master': bpo-33298: Wrap only constants with _PyCode_ConstantKey() in the compiler. (GH-6512) https://github.com/python/cpython/commit/d70c2a6894d80410b6f5a3129d6b1062ea8f

[issue33318] Move folding tuples of constants into compiler.c from peephole.c

New submission from Serhiy Storchaka : Explicit tuples of constants are folded in the AST optimizer since 3.7. But peephole.c still contains the code for folding tuples of constants, because there are tuples created by the compiler. For example "[1, 2, *a]", "f(1, 2, *a)", "def f(a=1, b=2)" al

[issue33317] `repr()` of string in NFC and NFD forms does not differ

Pekka Klärck added the comment: Thanks for pointing out `ascii()`. Seems to do exactly what I want. `repr()` showing combining characters would, in my opinion, still be useful to avoid problems like I demonstrated with unittest and pytest. I doubt it's a good idea with them to use `ascii()` i

[issue33317] `repr()` of string in NFC and NFD forms does not differ

Serhiy Storchaka added the comment: Use ascii() in Python 3 if you want the behavior of repr() in Python 2. It escapes all non-ascii characters. But escaping only combining characters in addition to non-printable characters in repr() looks an interesting idea. -- components: +Interpr

[issue33317] `repr()` of string in NFC and NFD forms does not differ

Pekka Klärck added the comment: Forgot to mention that this doesn't affect Python 2: >>> a = u'hyv\xe4' >>> b = u'hyva\u0308' >>> print(repr(a)) u'hyv\xe4' >>> print(repr(b)) u'hyva\u0308' In addition to hoping `repr()` would be enhanced in future Python 3 versions, I'm also looking for a wa

[issue33317] `repr()` of string in NFC and NFD forms does not differ

New submission from Pekka Klärck : If I have two strings that look the same but have different Unicode form, it's very hard to see where the problem actually is: >>> a = 'hyv\xe4' >>> b = 'hyva\u0308' >>> print(a) hyvä >>> print(b) hyvä >>> a == b False >>> print(repr(a)) 'hyvä' >>> print(repr