Update bindgen to make builds (even!) lighter #26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pulls in the latest update of bindgen, which contains some work I did in harryfei/which-rs#14 and rust-lang/rust-bindgen#1625 to further reduce the amount of indirect dependencies we have, and thus build times as well. 🙂
Basically,
bindgen
depends on thewhich
crate, which had a mandatory dependency onfailure
whichbindgen
did not use at all. I removed the dependency in those two PRs, making allbindgen
users free of having to pullfailure
as well as its own dependencies (most problematic beingbacktrace
).For us, this removes another 10 seconds of build times on my machine (from
1m28s
to1m18s
). 🙂This PR also updates the
syn
andquote
crates to their1.0
version, as I noticedbindgen
depended on them as well, and we ended up having two different versions in the dependency tree.