Skip to content

bpo-45598: Remove ffi header header #29352

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from
Closed

Conversation

tiran
Copy link
Member

@tiran tiran commented 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

https://bugs.python.org/issue45598

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

🤖 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-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 1, 2021
tiran added 2 commits November 3, 2021 13:59
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 tiran force-pushed the bpo-45598-ffi-macro branch from d366798 to d67f013 Compare November 3, 2021 13:05
@github-actions
Copy link

github-actions bot commented Dec 4, 2021

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

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Dec 4, 2021
@tiran
Copy link
Member Author

tiran commented Dec 4, 2021

#29747 adds configure checks for the macros.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting core review stale Stale PR or inactive for long period of time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants