Skip to content

bpo-45598: Remove ffi header header - #29352

Closed
tiran wants to merge 3 commits into
python:mainfrom
tiran:bpo-45598-ffi-macro
Closed

bpo-45598: Remove ffi header header#29352
tiran wants to merge 3 commits into
python:mainfrom
tiran:bpo-45598-ffi-macro

Conversation

@tiran

@tirantiran commented Nov 1, 2021

Copy link
Copy Markdown
Member

libffi supports ffi_prep_ciff, ffi_prep_closure_loc, and ffi_closure_alloc
for more than a decade. Assume that the feature exists on supported
platforms and only keep checks for old macOS libffi.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue45598

@tirantiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 1, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @tiran for commit d36679854d0faf997fb387d50b99087f307be33d 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 1, 2021
libffi supports ffi_prep_ciff, ffi_prep_closure_loc, and ffi_closure_alloc
for more than a decade. Assume that the feature exists on supported
platforms and only keep checks for old macOS libffi.
Signed-off-by: Christian Heimes <christian@python.org>
@tiran
tiranforce-pushed the bpo-45598-ffi-macro branch from d366798 to d67f013CompareNovember 3, 2021 13:05
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actionsgithub-actionsBot added the stale Stale PR or inactive for long period of time. label Dec 4, 2021
@tiran

tiran commented Dec 4, 2021

Copy link
Copy Markdown
MemberAuthor

#29747 adds configure checks for the macros.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core reviewstaleStale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tiran@bedevere-bot@the-knights-who-say-ni