Have ocamlrep_ocamlpool depend on ocamlpool
commit84dc8ab7502def3e67d151aa0aafb8b924e9f410
authorJake Bailey <jakebailey@meta.com>
Mon, 21 Nov 2022 22:12:57 +0000 (21 14:12 -0800)
committerFacebook GitHub Bot <facebook-github-bot@users.noreply.github.com>
Mon, 21 Nov 2022 22:12:57 +0000 (21 14:12 -0800)
treeb535bade39d16b6e9b0b7ba2ad9e7a75038c1cdd
parenta6a20222699ac5cb0c035e6771df52701316983f
Have ocamlrep_ocamlpool depend on ocamlpool

Summary:
For some reason, we never declared a dependency on the `ocamlpool` C library from the Rust bindings library `ocamlrep_ocamlpool`, instead requiring every binary rule which links to `ocamlrep_ocamlpool` to manually declare a dependency upon `ocamlpool`. I don't see any reason why this needs to be the case.

Also, do not use `ocamlrep_caml_builtins` in `oxidized_by_ref`, since pure-Rust binaries should be able to link to `oxidized_by_ref` without stubbing out the OCaml runtime. I'm not sure why they have been able to do this in the past, but we encounter linker errors in such binaries after adding the dependency between `ocamlrep_ocamlpool` and `ocamlpool`.

Reviewed By: shayne-fletcher

Differential Revision: D41417103

fbshipit-source-id: 19201bc1c8db30eedb2cf790009d077826a61d6b
hphp/hack/src/Cargo.lock
hphp/hack/src/decl/direct_decl_parser.ml
hphp/hack/src/decl/rust_decl_ffi/rust_decl_ffi.rs
hphp/hack/src/naming/names_rust/summary.rs
hphp/hack/src/oxidized_by_ref/Cargo.toml
hphp/hack/src/oxidized_by_ref/manual/direct_decl_parser.rs
hphp/hack/src/providers/cargo/rust_provider_backend_ffi/Cargo.toml
hphp/hack/src/providers/rust_provider_backend_ffi.rs
hphp/hack/src/utils/ocaml_ffi_mock/ocaml.c