Skip to content
This repository has been archived by the owner on Jun 13, 2018. It is now read-only.

Add bc to Dockerfile installs #64

Merged
merged 1 commit into from
Dec 11, 2017
Merged

Add bc to Dockerfile installs #64

merged 1 commit into from
Dec 11, 2017

Conversation

snario
Copy link
Contributor

@snario snario commented Nov 14, 2017

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

Some `seth` methods like `to-wei` depend on `bc`.
@snario
Copy link
Contributor Author

snario commented Nov 14, 2017

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
Copy link
Contributor

dbrock commented Nov 19, 2017

It's probably much better to use Nix

@dbrock
Copy link
Contributor

dbrock commented Nov 19, 2017

We should probably change the Dockerfile to install everything using Nix

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants