Skip to content

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

Merged
vstinner merged 1 commit intopython:masterfrom
methane:disable-opcache-on-pydebug
Jun 3, 2019
Merged

bpo-37146: disable opcache when Py_DEBUG is defined#13787
vstinner merged 1 commit intopython:masterfrom
methane:disable-opcache-on-pydebug

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

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