Merge pull request #1519 from ravenexp/fix-windows-shared

Always set `Py_SHARED` when building for Windows
This commit is contained in:
Yuji Kanagawa 2021-03-26 01:39:17 +09:00 committed by GitHub
commit eaf516d19a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 5 deletions

View File

@ -886,13 +886,17 @@ fn abi3_without_interpreter() -> Result<()> {
println!("cargo:rustc-cfg=py_sys_config=\"WITH_THREAD\"");
println!("cargo:python_flags={}", flags);
// Unfortunately, on windows we can't build without at least providing
// python.lib to the linker. While maturin tells the linker the location
// of python.lib, we need to do the renaming here, otherwise cargo
// complains that the crate using pyo3 does not contains a `#[link(...)]`
// attribute with pythonXY.
if env::var("CARGO_CFG_TARGET_FAMILY")? == "windows" {
// Unfortunately, on windows we can't build without at least providing
// python.lib to the linker. While maturin tells the linker the location
// of python.lib, we need to do the renaming here, otherwise cargo
// complains that the crate using pyo3 does not contains a `#[link(...)]`
// attribute with pythonXY.
println!("cargo:rustc-link-lib=pythonXY:python3");
// Match `get_config_from_interpreter()` and `windows_hardcoded_cross_compile()`:
// assume "Py_ENABLE_SHARED" to be set on Windows.
println!("cargo:rustc-cfg=Py_SHARED");
}
Ok(())