Skip to content
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

re-enable FFI support #2529

Merged
merged 1 commit into from
Sep 2, 2022
Merged

re-enable FFI support #2529

merged 1 commit into from
Sep 2, 2022

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Sep 2, 2022

tov/libffi-rs#58 landed so the license should no longer be an issue. :)

Fixes #2526

…ion"

This reverts commit 5f3545e.
With tov/libffi-rs#58 landed, we no longer depend on abort_on_panic.
@RalfJung RalfJung changed the title re-enabled FFI support re-enable FFI support Sep 2, 2022
@RalfJung
Copy link
Member Author

RalfJung commented Sep 2, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 2, 2022

📌 Commit 2f348ab has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Sep 2, 2022

⌛ Testing commit 2f348ab with merge 92c24c4...

@bors
Copy link
Collaborator

bors commented Sep 2, 2022

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 92c24c4 to master...

@bors bors merged commit 92c24c4 into rust-lang:master Sep 2, 2022
@RalfJung RalfJung deleted the yesffi branch September 2, 2022 13:45
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.

FFI support disabled due to licensing situation
2 participants