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

Add dapp key command to create wallets #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

nanexcool
Copy link
Contributor

dapp key --help

dapp key --keystore path (will prompt for password)

dapp key --keystore path --password file

default keystore ~/.ethereum/keystore

can be used with ETH_KEYSTORE and ETH_PASSWORD for seth compatibility

@dbrock
Copy link
Contributor

dbrock commented Mar 15, 2018

Hmmmmm I don't know if this should be in dapp!!!!!!!!??????????

Also it would have to be dapp key new (or dapp new-key), not just dapp key I think??????

@nanexcool
Copy link
Contributor Author

Yeah... it should be in seth? Here it's easy because we already have geth. In seth we'd have to either add geth or create a tiny program like ethsign that uses geth library for wallet creation, or add that capability to ethsign since it's already included, or grab some javascript library that already does this like ethereumjs-wallet

@dbrock
Copy link
Contributor

dbrock commented Mar 16, 2018

Yeah, I think the right answer is probably to either add this to ethsign or like you said make a new program that has this capability.

Or... maybe seth should also simply depend on geth itself?

I actually think for example dapp testnet might make more sense as seth testnet...

@dbrock
Copy link
Contributor

dbrock commented Mar 16, 2018

What do you think @mbrock?

@mbrock
Copy link
Contributor

mbrock commented Mar 16, 2018

Hmm, seth testnet seems reasonable to me, in a general spirit of "dapp minimalism".

The --nix-run thing in dapp is a bit weird and should probably be replaced with using normal Nix dependencies (so yeah, Seth would just depend on Geth, or the patched "Geth Unlimited" which deviates from consensus in allowing huge contracts e.g. for test suites).

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

Successfully merging this pull request may close these issues.

3 participants