[issue38206] Clarify that tp_dealloc must decref for heap allocated type

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

[issue38293] Deepcopying property objects results in unexpected TypeError

Guðni Nathan added the comment: A small change: The fix should go to Lib/copy.py:198, not line 208. -- ___ Python tracker ___ ___

[issue38293] Deepcopying property objects results in unexpected TypeError

New submission from Guðni Nathan : Currently, attempting to deepcopy a property object will result in an unexpected TypeError: >>> import copy >>> obj = property() >>> new_obj = copy.deepcopy(obj) Traceback (most recent call last): File "", line 1, in File "C:\Program

[issue13214] Cmd: list available completions from the cmd.Cmd subclass and filter out EOF handler(s)

Kimball Leavitt added the comment: I know that many people do something like: def do_EOF(self, arg): return True to exit the program when you press Ctrl+d. Others might prefer something like ngie https://bugs.python.org/issue13214#msg145856: def do_EOF(self, arg): raise EOFError

[issue37415] Error build Python with Intel compiler: doesn't provide atomic_uintptr_t

Rainer Keller added the comment: We have been experiencing the same problem with compiling Python-3.7.4 on a system with CentOS 7.6 (aka glib-2.17): Intel Compiler 2019.6 implements stdatomic.h. but lacks the definition of atomic_uintptr_t. >From a configure-point of view just failing to

[issue38187] test.test_tools.test_c_analyzer fails in refleak mode

Change by Eric Snow : -- assignee: -> eric.snow resolution: -> fixed stage: patch review -> resolved status: open -> pending ___ Python tracker ___

[issue38187] test.test_tools.test_c_analyzer fails in refleak mode

Eric Snow added the comment: New changeset 6693f730e0eb77d9453f73a3da33b78a97e996ee by Eric Snow in branch 'master': bpo-38187: Fix a refleak in Tools/c-analyzer. (gh-16304) https://github.com/python/cpython/commit/6693f730e0eb77d9453f73a3da33b78a97e996ee --

[issue38006] Crash in remove() weak reference callback of weakref.WeakValueDictionary at Python exit

Christian Heimes added the comment: Łukasz, I haven't been able to verify that the last patches are sufficient to prevent Python from segfaulting. We need to wait until rc1 is out. I need a proper release or RPM build to run the tests on our internal test infrastructure. You have to release

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

Eric Snow added the comment: +1 to the suggested "whatsnew" updates. If no one beats me to it I'll work on a PR soon. -- ___ Python tracker ___

[issue37878] Sub-Interpreters : Document PyThreadState_DeleteCurrent()

Eric Snow added the comment: I've re-opened this issue to address the original point: documenting PyThreadState_DeleteCurrent(). FWIW, I don't see a problem with documenting it if we're keeping it around (which it looks like we are). We will address reverting GH-15315 in #38266.

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Eric Snow added the comment: @Joannah, sounds good. Thanks! -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Eric Snow added the comment: I *was* going to suggest that we also put an underscore prefix on the name, but I couldn't think of a reason why we would want to discourage use (other than to reduce the size of the [supported] public API). Moving it to Include/cpython/pystate.h is probably

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

STINNER Victor added the comment: If we want to make the function "fully public", I suggest to write documentation and have a simple unit test. -- ___ Python tracker ___

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Joannah Nanjekye added the comment: I will handle this later today. -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Joannah Nanjekye added the comment: +1 for moving from Include/pystate.h to Include/cpython/pystate.h -- ___ Python tracker ___

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Eric Snow added the comment: s/un-revert/revert/ -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Eric Snow added the comment: FWIW, when we un-revert we should be sure to move PyThreadState_DeleteCurrent() from Include/pystate.h to Include/cpython/pystate.h. (I suppose that could be done in a separate PR to keep the git history clear.) --

[issue38266] Revert bpo-37878: Make PyThreadState_DeleteCurrent() Internal

Eric Snow added the comment: FWIW, I support reverting the removal of PyThreadState_DeleteCurrent(). We only reverted under the assumption that no one was using this function. Clearly we were mistaken. :) I'll re-open #37878 to revive the discussion about documenting the function (which

[issue38270] Tests: Avoid MD5 or check for MD5 availablity

Change by Christian Heimes : -- pull_requests: +16015 pull_request: https://github.com/python/cpython/pull/16437 ___ Python tracker ___

[issue22107] tempfile module misinterprets access denied error on Windows

Erik Aronesty added the comment: i would like to point out that the primary reason any of this nonsense exists is because of short filename restrictions. i've replaces nearly all of my temp file creation code in all of my project to `return os.urandom(32).hex()` ... which is reliable

[issue38292] tkinter variable classes don't link to widget if matplotlib's set_cmap() function is called before the tkinter GUI is instantiated

New submission from Amir Emami : TkInter variable classes don't link to a widget (tested on Checkbutton and Scale) if matplotlib.pyplot.set_cmap() function is called before the tkinter.Tk() call which instantiates the root of a TkInter GUI. There is no problem if it is called after this,

[issue38270] Tests: Avoid MD5 or check for MD5 availablity

miss-islington added the comment: New changeset 90558158093c0ad893102158fd3c2dd9f864e82e by Miss Islington (bot) (Christian Heimes) in branch 'master': bpo-38270: More fixes for strict crypto policy (GH-16418) https://github.com/python/cpython/commit/90558158093c0ad893102158fd3c2dd9f864e82e

[issue38239] test_gdb fails on AMD64 Fedora Stable LTO 3.8 and AMD64 RHEL8 LTO 3.x: Unexpected gdb output

STINNER Victor added the comment: > AMD64 RHEL8 LTO 3.x is green again. I will keep the issue open until all > mentionned buildbots are back to green (when test_gdb pass on these workers). I checked an all buildbots mentioned in this issues are back to green (success). I close the issue.

[issue38291] Unclear status of the typing.io and typing.re pseudo-modules in docs and runtime

New submission from dgelessus : According to bpo-35089 (and the GitHub issues linked there), the typing.io and typing.re modules should no longer be used. Starting with Python 3.6, the typing documentation no longer mentions the typing.io and typing.re modules, and instead documents their

[issue38206] Clarify that tp_dealloc must decref for heap allocated type

miss-islington added the comment: New changeset 14ddca726a8cd337d0461934374b5e6bf65bf812 by Miss Islington (bot) in branch '3.8': bpo-38206: Clarify tp_dealloc requirements for heap allocated types. (GH-16248) https://github.com/python/cpython/commit/14ddca726a8cd337d0461934374b5e6bf65bf812

[issue38206] Clarify that tp_dealloc must decref for heap allocated type

Change by miss-islington : -- pull_requests: +16014 pull_request: https://github.com/python/cpython/pull/16436 ___ Python tracker ___

[issue38206] Clarify that tp_dealloc must decref for heap allocated type

miss-islington added the comment: New changeset 5faff977adbe089e1f91a5916ccb2160a22dd292 by Miss Islington (bot) (Ammar Askar) in branch 'master': bpo-38206: Clarify tp_dealloc requirements for heap allocated types. (GH-16248)

[issue22107] tempfile module misinterprets access denied error on Windows

And Clover added the comment: Attempting to answer the question "did this open call fail because the path was a directory" by implication from "do we think we ought to be able to write a file to this directory" is IMO doomed. There's no reliable way to determine whether one should be able

[issue37431] test_asyncio: test_start_tls_server_1() failed on ARMv7 Debian buster 3.7

STINNER Victor added the comment: New fail on ARMv7 Debian buster 3.x: https://buildbot.python.org/all/#/builders/176/builds/1356 == ERROR: test_start_tls_server_1 (test.test_asyncio.test_sslproto.SelectorStartTLSTests)

[issue38174] Security vulnerability in bundled expat CVE-2019-15903 (fix available in expat 2.2.8)

Change by STINNER Victor : -- pull_requests: +16013 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/16434 ___ Python tracker ___

[issue38234] The value of Py_SetPath is not used to populate the configuration

Change by Kubilay Kocak : -- nosy: +koobs ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38160] Add a "PyInterpreterState *" field to PyTypeObject.

STINNER Victor added the comment: > No, but it means that a type A *knows* that it was created in interpreter 1 > without relying on the current thread local storage. What's wrong with _PyInterpreterState_GET_UNSAFE()? This macro is based on

[issue38269] AMD64 Debian PGO 3.x, AMD64 Clang UBSan 2.7 buildbots: No space left on device

Change by STINNER Victor : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___ ___

[issue38269] AMD64 Debian PGO 3.x, AMD64 Clang UBSan 2.7 buildbots: No space left on device

STINNER Victor added the comment: > Can we close this issue? When it's an issue about specific buildbots, I prefer to first check that the buildbot is back to green. > https://buildbot.python.org/all/#/builders/47/builds/3578 AMD64 Debian PGO 3.x buildbot is back to green. >

[issue38160] Add a "PyInterpreterState *" field to PyTypeObject.

Petr Viktorin added the comment: > only if PEP 573 makes it *compulsory* for heap types to contain a pointer to > their module And that's impossible without either breaking API (or some hackery). PyType_FromSpec doesn't get the information, and it can be easily outside module

[issue38290] cleanup ababstractproperty in typing.py

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

[issue38290] cleanup ababstractproperty in typing.py

Change by miss-islington : -- pull_requests: +16012 pull_request: https://github.com/python/cpython/pull/16433 ___ Python tracker ___

[issue38290] cleanup ababstractproperty in typing.py

Change by hongweipeng : -- keywords: +patch pull_requests: +16011 stage: -> patch review pull_request: https://github.com/python/cpython/pull/16432 ___ Python tracker ___

[issue38290] cleanup ababstractproperty in typing.py

New submission from hongweipeng : `abc.ababstractproperty` was deprecated since version 3.3. https://docs.python.org/3/library/abc.html#abc.abstractproperty I checked the code and found that only `typing.py` is still used. I created a PR and replaced it with the one written in the

[issue38289] Add an option to point to an additional "site" directory

Michał Fronczyk added the comment: A link to my stackoverflow question with further rationale - https://stackoverflow.com/questions/58122752/provide-additional-location-of-site-packages-to-python?noredirect=1#comment102637846_58122752 -- ___ Python

[issue38269] AMD64 Debian PGO 3.x, AMD64 Clang UBSan 2.7 buildbots: No space left on device

Dong-hee Na added the comment: @gregory.p.smith @vstinner Looks like https://github.com/python/buildmaster-config/pull/111 is merged. Can we close this issue? -- ___ Python tracker

[issue38140] Py_tp_dictoffset / Py_tp_finalize are unsettable in stable API

Eddie Elizondo added the comment: Hey Petr, I'll get to document this and fix the windows warning over the weekend. I'll ping you on Github once it's ready -- nosy: +eelizondo ___ Python tracker

<    1   2