FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-28787: Fix out of tree --with-dtrace builds by stratakis · Pull Request #135 · python/cpython · GitHub

Repository navigation

bpo-28787: Fix out of tree --with-dtrace builds - #135

Merged
vstinner merged 3 commits into
python:masterfrom
stratakis:out_of_tree_dtrace
May 10, 2017
Merged

vstinner merged 3 commits into
python:masterfrom
stratakis:out_of_tree_dtrace

Conversation

Copy link
Copy Markdown
Contributor

Issue reported at: https://bugs.python.org/issue28787

This was observed after DTrace and Systemtap support was introduced for Python 3.6.0: https://bugs.python.org/issue21590

vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

It seems to be your first contribution, please add yourself to Misc/ACKS ;-)

Comment thread Makefile.pre.in Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Please remove @, I like to see what is done.

vstinner commented May 5, 2017

Copy link
Copy Markdown
Member

Please rebase also your change.

vstinner merged commit f6eae5b into python:master May 10, 2017

Copy link
Copy Markdown
Member

I merged your change, thanks @stratakis. Would you mind to backport the change? Only 3.6 is needed according to http://bugs.python.org/issue28787

I used the following commands. Without the change, make fails. With the fix, make works as expected.

cd $HOME
rm -rf build_cpython/
mkdir build_cpython/
cd build_cpython/
~/prog/python/master/configure --with-pydebug --with-dtrace
make

Copy link
Copy Markdown
Contributor Author

Thanks @Haypo . I have already backported the change at #1519 .

stratakis added a commit to stratakis/cpython that referenced this pull request May 11, 2017
* bpo-28787: Fix out of tree --with-dtrace builds

* Unsilence directory creation

* Add Misc/NEWS and Misc/ACKS entries.
(cherry picked from commit f6eae5b)
vstinner pushed a commit that referenced this pull request May 11, 2017
* bpo-28787: Fix out of tree --with-dtrace builds

* Unsilence directory creation

* Add Misc/NEWS and Misc/ACKS entries.
(cherry picked from commit f6eae5b)
akruis pushed a commit to akruis/cpython that referenced this pull request Oct 10, 2017
This commit reveals a reference leak. See issue python#135,
stackless-dev#135
akruis pushed a commit to akruis/cpython that referenced this pull request Oct 10, 2017
Fix several problems related with slp_prepare_slots():
- set Py_TPFLAGS_HAVE_STACKLESS_EXTENSION in slp_prepare_slots().
- for heap types free the allocated buffer in tp_dealloc
- Allocate the memory from the object domain, because the blocks are
  small (about 74 bytes).
akruis pushed a commit to akruis/cpython that referenced this pull request Oct 29, 2017
akruis pushed a commit to akruis/cpython that referenced this pull request Jun 19, 2018
Fix several problems related with slp_prepare_slots():
- set Py_TPFLAGS_HAVE_STACKLESS_EXTENSION in slp_prepare_slots().
- for heap types free the allocated buffer in tp_dealloc
- Allocate the memory from the object domain, because the blocks are
  small (about 74 bytes).

(cherry picked from commit e217840)
stratakis deleted the out_of_tree_dtrace branch January 3, 2020 21:28
jaraco pushed a commit that referenced this pull request Dec 2, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL