Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-100540: Remove unused Modules/_ctypes/libffi_osx/#100543

Merged
zware merged 1 commit into
python:mainpython/cpython:mainfrom
zware:remove_libffi_osxzware/cpython:remove_libffi_osxCopy head branch name to clipboard
Dec 28, 2022
Merged

gh-100540: Remove unused Modules/_ctypes/libffi_osx/#100543
zware merged 1 commit into
python:mainpython/cpython:mainfrom
zware:remove_libffi_osxzware/cpython:remove_libffi_osxCopy head branch name to clipboard

Conversation

@zware

@zware zware commented Dec 26, 2022

Copy link
Copy Markdown
Member

It was an ancient, modified copy of libffi that has not been in use
since GH-22855.

It was an ancient, modified copy of libffi that has not been in use
since pythonGH-22855.
@zware zware added 3.12 only security fixes OS-mac build The build process and cross-build topic-ctypes extension-modules C modules in the Modules dir skip news labels Dec 26, 2022
@zware zware self-assigned this Dec 26, 2022
@zware zware requested a review from a team December 26, 2022 21:29
@zware

zware commented Dec 26, 2022

Copy link
Copy Markdown
Member Author

The Azure Pipelines failure is due to patchcheck wanting to destroy the tsv values in Tools/c-analyzer/cpython/_parser.py since a mention of libffi_osx was removed from it.

@zware

zware commented Dec 26, 2022

Copy link
Copy Markdown
Member Author

!buildbot .*mac

@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @zware for commit ad202d7 🤖

The command will test the builders whose names match following regular expression: .*mac

The builders matched are:

  • ARM64 macOS PR
  • x86-64 macOS PR

@zware zware merged commit 025b5c3 into python:main Dec 28, 2022
@zware zware deleted the remove_libffi_osx branch December 28, 2022 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.12 only security fixes build The build process and cross-build extension-modules C modules in the Modules dir OS-mac skip news topic-ctypes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.