◐ Shell
clean mode source ↗

gh-100540: Remove unused Modules/_ctypes/libffi_osx/ by zware · Pull Request #100543 · python/cpython

zware

Skip to content

Navigation Menu

Sign in

Appearance settings

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

Sign up

Appearance settings

Conversation

@zware

@zware zware commented

Dec 26, 2022

•

edited by bedevere-bot

Loading

Copy link Copy Markdown

Member

It was an ancient, modified copy of libffi that has not been in use
since GH-22855.
erlend-aasland reacted with thumbs up emoji 
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

Reviewers

@erlend-aasland erlend-aasland erlend-aasland approved these changes

Assignees

@zware zware

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zware @bedevere-bot @erlend-aasland