Skip to content

bpo-37146: disable opcache when Py_DEBUG is defined #13787

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 3, 2019

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