Skip to content

Conversation

methane
Copy link
Member

@methane methane commented Jun 3, 2019

--with-pydebug is commonly used to find memory leaks.
But opcache makes it harder.
So disable opcache when Py_DEBUG is defined.

https://bugs.python.org/issue37146

--with-pydebug is commonly used to find memory leaks.
But opcache makes it harder.
So disable opcache when Py_DEBUG is defined.
@methane methane requested a review from vstinner June 3, 2019 22:20
@methane methane added skip news type-bug An unexpected behavior, bug, or error labels Jun 3, 2019
@pablogsal
Copy link
Member

Check out https://github.com/python/cpython/pull/13789/files for a possible more complete fix

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. That's a reasonable workaround for https://bugs.python.org/issue37146 for beta1. We can design a better solution after beta1.

@vstinner vstinner merged commit eddef86 into python:master Jun 3, 2019
@vstinner
Copy link
Member

vstinner commented Jun 3, 2019

@methane: Thanks. I merged your PR to make Refleak buildbots green again (I hope!).

@python python deleted a comment from bedevere-bot Jun 4, 2019
@python python deleted a comment from bedevere-bot Jun 4, 2019
@python python deleted a comment from bedevere-bot Jun 4, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
--with-pydebug is commonly used to find memory leaks.
But opcache makes it harder.
So disable opcache when Py_DEBUG is defined.
@methane methane deleted the disable-opcache-on-pydebug branch April 28, 2025 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants