facebookincubator / fizz

C++14 implementation of the TLS-1.3 standard
Other
1.13k stars 158 forks source link

Paramatized rust_static_library to enable CXX support #101

Closed jdelliot closed 10 months ago

jdelliot commented 10 months ago

Summary: X-link: https://github.com/facebook/sapling/pull/763

X-link: https://github.com/facebookexperimental/rust-shed/pull/43

X-link: https://github.com/facebook/wangle/pull/221

X-link: https://github.com/facebook/openr/pull/150

X-link: https://github.com/facebook/hhvm/pull/9403

X-link: https://github.com/facebook/folly/pull/2092

X-link: https://github.com/facebook/fb303/pull/42

X-link: https://github.com/facebookincubator/velox/pull/7301

We are now using CXX (and not just bindgen/cbindgen) for building our Rust C/C++ APIS, but our OSS tooling did not ergomically support this. This change adds a single option, USE_CXX_INCLUDE, to the CMake function, rust_static_library, to enable adding the cxxbridge path to the include path.

Differential Revision: D50772544

facebook-github-bot commented 10 months ago

This pull request was exported from Phabricator. Differential Revision: D50772544

facebook-github-bot commented 10 months ago

This pull request was exported from Phabricator. Differential Revision: D50772544

facebook-github-bot commented 10 months ago

This pull request was exported from Phabricator. Differential Revision: D50772544

facebook-github-bot commented 10 months ago

This pull request has been merged in facebookincubator/fizz@536283604a2c7a2e8fb544f7794eae88d258ccb5.