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

[skip issue] [skip news] Update libffi dll name to libffi-8 #28203

Merged
merged 1 commit into from Sep 7, 2021
Merged

[skip issue] [skip news] Update libffi dll name to libffi-8 #28203

merged 1 commit into from Sep 7, 2021

Conversation

@giovanniwijaya
Copy link
Contributor

@giovanniwijaya giovanniwijaya commented Sep 7, 2021

In Tools/msi/lib/lib_files.wxs, the libffi dll name was still kept as libffi-7.dll, corresponding to version 3.3.0. As GH-28146 updates libffi version to 3.4.2 (and the dll name to libffi-8.dll), this causes failure when building the Windows installer.

@giovanniwijaya giovanniwijaya requested a review from as a code owner Sep 7, 2021
@giovanniwijaya giovanniwijaya changed the title Update libffi dll name to libffi-8 [skip issue] [skip news] Update libffi dll name to libffi-8 Sep 7, 2021
@zooba
Copy link
Member

@zooba zooba commented Sep 7, 2021

Thanks for this! Oversight on my part.

@zooba zooba merged commit 4dc4300 into python:main Sep 7, 2021
14 checks passed
14 checks passed
@github-actions[bot]
Check for source changes
Details
@github-actions[bot]
Windows (x86) Installer
Details
@github-actions[bot]
Check if generated files are up to date
Details
@github-actions[bot]
Windows (x64) Installer
Details
@github-actions[bot]
Windows (x86)
Details
@github-actions[bot]
Windows (x64)
Details
@github-actions[bot]
macOS
Details
@github-actions[bot]
Ubuntu
Details
@github-actions[bot]
Ubuntu SSL tests with OpenSSL (1.1.1l)
Details
@github-actions[bot]
Ubuntu SSL tests with OpenSSL (3.0.0-beta1)
Details
@github-actions[bot]
Address sanitizer Address sanitizer
Details
Azure Pipelines PR #20210907.20 succeeded
Details
@bedevere-bot
bedevere/issue-number Issue report skipped
@bedevere-bot
bedevere/news "skip news" label found
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants