Skip to content

Add engine support for linux-gnu #63

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.

Sign up for GitHub

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

Merged
merged 6 commits into from
Jun 9, 2020
Merged

Add engine support for linux-gnu #63

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
ef1cfcb
add engine support for linux-gnu
hunterlxt Jun 6, 2020
e1153ff
address comment
hunterlxt Jun 8, 2020
ed69d10
add blacklist os
hunterlxt Jun 8, 2020
6dc76e6
add blacklist os
hunterlxt Jun 8, 2020
a280ada
To check CI build info
hunterlxt Jun 8, 2020
b8e040e
add comments
hunterlxt Jun 9, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 11 additions & 8 deletions src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,17 +102,20 @@ impl Build {
// Nothing related to zlib please
.arg("no-comp")
.arg("no-zlib")
.arg("no-zlib-dynamic")
.arg("no-zlib-dynamic");

if target.contains("musl") || target.contains("windows") {
// 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. This
// is intended for super-configurable backends and whatnot
// apparently but the whole point of this script is to produce a
// "portable" implementation of OpenSSL, so shouldn't be any harm in
// turning this off.
.arg("no-engine")
// 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");
}

if target.contains("musl") {
// MUSL doesn't implement some of the libc functions that the async
// stuff depends on, and we don't bind to any of that in any case.
.arg("no-async");
configure.arg("no-async");
}

// On Android it looks like not passing no-stdio may cause a build
// failure (#13), but most other platforms need it for things like
Expand Down