-
Notifications
You must be signed in to change notification settings - Fork 4
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
inconsistencies in flags #3
Comments
Also |
Interestingly, things work on my mac without the "cdylib" here: https://github.com/zshipko/ocaml-rust-starter/blob/master/Cargo.toml#L8 but don't work with it |
Thanks for the report!
That is actually in
These are needed on linux
I'm happy to review a PR with that update
What things don't work? Can you please provide a little more information or an error message? |
here's the error I get when I try to compile with cdylib:
|
I've found that using the following in a fn main() {
if cfg!(any(target_os = "macos", target_os = "ios")) {
println!("cargo:rustc-link-arg=-Wl,-undefined,dynamic_lookup");
}
} works well as long as you have rust-lang/cargo#9557 (Rust 1.56.0). (I'm added iOS since I suspect it has the same problem) |
The docs says you should add some flags to
Cargo.toml
on MacOS:yet we don't do it here: https://github.com/zshipko/ocaml-rust-starter/blob/master/Cargo.toml
yet (bis) it works fine on my mac without these build flags.
Second, why the C flags here? https://github.com/zshipko/ocaml-rust-starter/blob/master/src/dune#L17
It works fine on my mac without these :o
The text was updated successfully, but these errors were encountered: