cf2613e869
- tokio 1.0 - updated reqwest, thanks to Rumo - other minor dep updates the reqwest build file has been split into two, as it was awkward to manually update the combined file, and the platform gate is now on the target in rslib/
53 lines
1.9 KiB
Markdown
53 lines
1.9 KiB
Markdown
This folder integrates Rust crates.io fetching into Bazel.
|
|
|
|
To update dependencies, ensure a local Rust environment is available
|
|
(eg install rustup), then fetch this commit of cargo raze:
|
|
|
|
https://github.com/google/cargo-raze/commit/1edfb5366d7a6a59c3f9e354a3818a649146548c
|
|
|
|
then from the folder:
|
|
|
|
```
|
|
cd impl
|
|
cargo install --path .
|
|
cargo install cargo-license
|
|
```
|
|
|
|
After updating dependencies in ../rslib/Cargo.toml, change to this
|
|
folder and run python update.py to update the external Bazel repositories
|
|
to point to the updated deps.
|
|
|
|
A couple of crates need extra work to build with Bazel, and are listed
|
|
in ../Cargo.toml. For example:
|
|
|
|
```toml
|
|
[package.metadata.raze.crates.pyo3.'*']
|
|
compile_data_attr = "glob([\"**\"])"
|
|
```
|
|
|
|
With minor version updates, you should not normally need to modify
|
|
the entries in that file.
|
|
|
|
Because update.py modifies a lot of files in remote/, it makes it difficult to
|
|
review in a PR, and the changes can sometimes break platforms like Windows. For
|
|
this reason, please don't submit PRs that do minor version bumps - those will
|
|
typically be done after stable releases. If you need a new crate for a feature
|
|
you're working on, please raise it in an issue first.
|
|
|
|
## Reqwest
|
|
|
|
Things are complicated with reqwest at the moment, because:
|
|
|
|
- we're using a fork to implement better timeouts for syncing
|
|
- we want to build it with different features on Linux (where we can't build a
|
|
wheel that links to OpenSSL), and on other platforms.
|
|
|
|
For minor version bumps, update.py should take care of updating the versions of
|
|
reqwest dependencies.
|
|
|
|
After making a big update to reqwest via an updated fork, the vendored
|
|
BUILD.reqwest.\* files may need updating. To do that, comment native-tls from
|
|
the features in rslib/Cargo.toml and run update.py, and copy the file in remote/
|
|
over the old vendored file. Then comment the other two deps out, add native-tls
|
|
back, and repeat the process.
|