Skip to content

bpo-36707: Remove "m" flag from SOABI#12931

Merged
vstinner merged 1 commit intopython:masterfrom
vstinner:pymalloc_soabi
Apr 24, 2019
Merged

bpo-36707: Remove "m" flag from SOABI#12931
vstinner merged 1 commit intopython:masterfrom
vstinner:pymalloc_soabi

Conversation

@vstinner
Copy link
Copy Markdown
Member

@vstinner vstinner commented Apr 24, 2019

./configure --with-pymalloc no longer add "m" flag to SOABI:
enabling or disabling pymalloc has no impact on the ABI.

https://bugs.python.org/issue36707

Copy link
Copy Markdown
Member

@warsaw warsaw left a comment

Choose a reason for hiding this comment

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

Small wording suggestions.

Comment thread Misc/NEWS.d/next/Build/2019-04-24-02-29-15.bpo-36707.8ZNB67.rst Outdated
@vstinner
Copy link
Copy Markdown
Member Author

@warsaw: I updated my PR, would you mind to review it again?

"./configure --with-pymalloc" no longer adds the "m" flag to SOABI
(sys.implementation.cache_tag).

Enabling or disabling pymalloc has no impact on the ABI.
@vstinner
Copy link
Copy Markdown
Member Author

Oh, I missed the comment proposing the add "the" and fixing a typo (add => adds). Both are now fixed.

@vstinner vstinner merged commit 6c44fde into python:master Apr 24, 2019
@vstinner vstinner deleted the pymalloc_soabi branch April 24, 2019 14:10
@vstinner
Copy link
Copy Markdown
Member Author

Thanks for the review @warsaw and @methane :-)

stefanor added a commit to stefanor/line_profiler that referenced this pull request Nov 3, 2020
Erotemic pushed a commit to pyutils/line_profiler that referenced this pull request Nov 3, 2020
* Test on Python 3.9

* Python 3.9 dropped gettimeofday configure checks

In bpo-38068, cPython assumes gettimeofday exists and takes two
arguments. That's a reasonable assumption that we can repeat.

See: python/cpython@f1c1903

* Drop Python 3.5 from Travis

Python 3.5 is end of life, and the test stack is no longer installing.

* The m flag was dropped in 3.8, this wasn't a once-off abnormality

See: python/cpython#12931
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.

5 participants