dapphub / dapp

This repository has been moved to dapphub/dapptools
https://dapp.tools/dapp
249 stars 49 forks source link

Add `bc` to Dockerfile installs #64

Closed snario closed 6 years ago

snario commented 6 years ago

Some seth methods like to-wei depend on bc.

Before

node@c693e002dd7b:/src$ seth --to-wei 1 gwei
/usr/local/libexec/seth/seth---to-wei: line 18: bc: command not found

After

node@c693e002dd7b:/src$ seth --to-wei 1 gwei
1000000000
snario commented 6 years ago

Actually there are a few more things missing like ethabi and jshon, let me come back to this in a bit. Maybe the Docker container doesn't need to have support for seth?

dbrock commented 6 years ago

It's probably much better to use Nix

dbrock commented 6 years ago

We should probably change the Dockerfile to install everything using Nix