Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Update substrate-node-template readme to add ./.local folder #12901

Closed
wants to merge 1 commit into from

Conversation

ltfschoen
Copy link
Contributor

@ltfschoen ltfschoen commented Dec 12, 2022

Add an instruction in the "Run in Docker" instructions to create that folder (i.e. mkdir .local) before running the scripts like ./scripts/docker_run.sh, otherwise users get the following error when they clone the repo https://github.com/substrate-developer-hub/substrate-node-template and just run ./scripts/docker_run.sh

ERROR: for substrate-node-template_dev_run  Cannot create container for service dev: invalid mount config for type "bind": bind source path does not exist: /Users/me/substrate-node-template/.local

ERROR: for dev  Cannot create container for service dev: invalid mount config for type "bind": bind source path does not exist: /Users/me/substrate-node-template/.local

Alternative 1 create a /.local folder in the readme for users to use.

Alternative 2 suggest users use the Substrate docker instructions instead so they use https://github.com/paritytech/substrate/blob/master/docker/substrate_builder.Dockerfile, and just tweak the instructions to use the node-template part of it

…local directory if they want to use the template Docker instructions

Add an instruction in the "Run in Docker" instructions to create that folder (i.e. `mkdir .local`) before running the scripts like `./scripts/docker_run.sh`, otherwise users get the following error:

```
ERROR: for substrate-node-template_dev_run  Cannot create container for service dev: invalid mount config for type "bind": bind source path does not exist: /Users/me/substrate-node-template/.local

ERROR: for dev  Cannot create container for service dev: invalid mount config for type "bind": bind source path does not exist: /Users/me/substrate-node-template/.local
```

Alternatively create a /.local folder in the readme for users to use
@bkchr
Copy link
Member

bkchr commented Dec 12, 2022

Alternative 2 suggest users use the Substrate docker instructions instead so they use https://github.com/paritytech/substrate/blob/master/docker/substrate_builder.Dockerfile, and just tweak the instructions to use the node-template part of it

This is the correct way to go.

@bkchr
Copy link
Member

bkchr commented Dec 12, 2022

Maybe you could sync up with @nukemandan in #12741

@stale
Copy link

stale bot commented Jan 11, 2023

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Jan 11, 2023
@stale stale bot closed this Jan 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants