alexcrichton / curl-rust

Rust bindings to libcurl
MIT License
1.01k stars 234 forks source link

Add environment variable to find GSSAPI #373

Closed theawless closed 3 years ago

theawless commented 3 years ago

This change allows us to set the location for GSSAPI. This would be useful on systems where the library is not found at the usual location /usr or if the client wants to set it manually. I tried to make the behaviour similar to DEP_NGHTTP2_ROOT.

alexcrichton commented 3 years ago

FWIW the DEP variables are not meant to be set by users, they're set by Cargo because dependencies of this crate are named nghttp2 and such (via the links key in the manifest). If this isn't going through Cargo for dependencies, could the env var be named differently?

theawless commented 3 years ago

Would GSSAPI_ROOT be okay?

alexcrichton commented 3 years ago

Seems fine by me!

theawless commented 3 years ago

Updated the commits. Thank you for your time!.