search

rustls / rustls-ffi

Use Rustls from any language
Other
132 stars 30 forks source link

Document static lib dependencies #213

Closed jsha closed 2 years ago

jsha commented 2 years ago

Fixes #133

kevinburke commented 2 years ago

See #237, which will protect against regressions

kevinburke commented 2 years ago

FYI, on Mac, I get a different list now:

-framework Security -liconv -lSystem -lresolv -lc -lm -liconv

I have no idea why -liconv is presented twice.

jsha commented 2 years ago

FYI, on Mac, I get a different list now:

That's the same as what's in this PR, I just added -lrustls at the front.

I have no idea why -liconv is presented twice.

Same! Maybe file an issue on Rust? Seems likely to be a bug.

kevinburke commented 2 years ago

Ah, I should have read the note above the list. "The order and any duplication can be significant."

note: Link against the following native artifacts when linking against this static library. The order and any duplication can be significant on some platforms.

note: native-static-libs: -framework Security -liconv -lSystem -lresolv -lc -lm -liconv

    Finished dev [unoptimized + debuginfo] target(s) in 11.51s