zshipko
commented
5 years ago How can I help with getting this to build? I would really like to be able to use it for an upcoming project!
Closed phil-opp closed 5 years ago
zshipko
commented
5 years ago How can I help with getting this to build? I would really like to be able to use it for an upcoming project!
oli-obk
commented
5 years ago I think just running cargo update should do the trick
phil-opp
commented
5 years ago On the latest nightly the build still fails due to the updates to the futures/task API in https://github.com/rust-lang/rust/pull/57992. I tried to rebase our futures-rs fork for these changes: https://github.com/embed-rs/futures-rs/tree/alloc-rebase (I used a separate branch because I didn't test it yet). We also need to update our core wrapper and the future_mutex and task_runtime modules.
oli-obk
commented
5 years ago I'm gonna finish the rebases and updates
The pull request https://github.com/m-labs/smoltcp/pull/186 was forced-pushed, so the commit referenced in the Cargo.lock does no longer exist. Updating the Cargo.lock is not possible at the moment because the new branch contains an invalid Cargo.toml: https://github.com/m-labs/smoltcp/pull/186#discussion_r254349551.
I think the best solution is to wait a bit and hope that the pull request gets merged soon. If it takes to long, we can use an own fork the branch to ensure that no force-pushes occur.