Skip to content

Commit

Permalink
Enable the engine feature on windows (#109)
Browse files Browse the repository at this point in the history
* Enable the engine feature on windows

I'm doing things on Windows and that require the `engine` feature to be enabled so that `ENGINE_by_id` is available.
This commit succeeds for me locally in a win64 msvc environment.

* fix typo

This is what happens when you edit via the webui
  • Loading branch information
wez authored Oct 11, 2021
1 parent d68b9fd commit 4f6cc35
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -176,11 +176,16 @@ impl Build {
configure.arg("no-seed");
}

if target.contains("musl") || target.contains("windows") {
if target.contains("musl") {
// This actually fails to compile on musl (it needs linux/version.h
// right now) but we don't actually need this most of the time.
// API of engine.c ld fail in Windows.
configure.arg("no-engine");
} else if target.contains("windows") {
// We can build the engine feature, but the build doesn't seem
// to correctly pick up crypt32.lib functions such as
// `__imp_CertOpenStore` when building the capieng engine.
// Let's disable just capieng.
configure.arg("no-capieng");
}

if target.contains("musl") {
Expand Down

0 comments on commit 4f6cc35

Please sign in to comment.