paritytech / substrate-up

Scripts for working with new Substrate projects
Apache License 2.0
67 stars 21 forks source link

[substrate-node-new] pipefail side effect #41

Open elenoir opened 4 years ago

elenoir commented 4 years ago

Hi,

In #40, you activated the script exiting in case a command does not end well with set -eo pipefail I faced a side effect of this after bootstrapping a fresh Virtual Machine and trying to install substrate ; indeed, my git was not configured (user.email/user.name) so the line git commit -a -m "Initial clone from template node" 2>/dev/null >/dev/null fails. As the standard and error outputs are not shown, it was a pain to understand why my node did not install.

Maybe we should remove these 2> /dev/null no? Please give me your opinion and I will submit a PR if needed.

Thanks !