interledger / interledger.github.io

🙅 Even earlier version of Interledger website
38 stars 38 forks source link

Fix spin-up-local-network.md and build #148

Closed nkmr-jp closed 3 years ago

nkmr-jp commented 3 years ago

I did the tutorial on this page and got an error with the following command, so I fixed the documentation. https://interledger.org/spin-up-local-network.html

$ docker build -f ./docker/ilp-node.dockerfile -t interledgerrs/ilp-node --build-arg CARGO_BUILD_OPTION="--release --features monitoring --manifest-path ./crates/ilp-node/Cargo.toml" --build-arg RUST_BIN_DIR_NAME="release" https://github.com/interledger-rs/interledger-rs.git#ko-stream-reliability
unable to prepare context: unable to 'git clone' to temporary context directory: error fetching: fatal: couldn't find remote ref ko-stream-reliability
: exit status 128

When I ran dactyl_build -S as described in the README, I found other files with differences besides the edited ones. Like this (3f510c2).

So, I just built the edited file as dactyl_build -S --only spin-up-local-network.md.

nkmr-jp commented 3 years ago

I've created a separate issue #149

kincaidoneil commented 3 years ago

Thank you for this! I believe we can nix that command altogether and just add docker pull interledgerrs/ilp-node to to the step above? (which should be much faster)

nkmr-jp commented 3 years ago

Thanks for the advice! modify documentation to use pre-built ilp-node docker image. (ad41f2b) I checked the operation and it worked. (which was much faster.)