delvtech / hyperdrive-rs

Rust SDK for the Hyperdrive AMM.
https://docs.rs/crate/hyperdrive-math/latest
Apache License 2.0
2 stars 0 forks source link

`build.rs` does not always work #141

Closed dpaiton closed 4 months ago

dpaiton commented 4 months ago

on fresh installs we need to add a forge install step so that the Hyperdrive dependencies (openzeppelin, etc) are installed. When running make build on a fresh ubuntu environment it fails because this step is missing.

the need_to_build function also does not always work correctly. When doing a LOCAL_DEVELOPMENT=true build I was frequently getting unresolved import errors:

image

which are happening because it's not building. I could consistently fix the problem by forcing need_to_build to always return Ok(true).

dpaiton commented 4 months ago

the --root flag should be helpful in the script (forge install --root path/to/hyperdrive)

sentilesdal commented 4 months ago

fixed in #148