aboutsummaryrefslogtreecommitdiffstats
path: root/cmd/geth/chaincmd.go
diff options
context:
space:
mode:
authorFelix Lange <fjl@twurst.com>2016-03-21 21:05:22 +0800
committerFelix Lange <fjl@twurst.com>2016-04-12 21:56:49 +0800
commitdff9b4246f3ef9e6c254b57eef6d0433809f16b9 (patch)
tree71cab4093300a6e9bd1b5801f05e3507246eef7e /cmd/geth/chaincmd.go
parent83877a0f9d3d5716ee01393f10c2dfa19bb0310b (diff)
downloaddexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.gz
dexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.zst
dexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.zip
cmd/geth, cmd/utils: improve input handling
These changes make prompting behave consistently on all platforms: * The input buffer is now global. Buffering was previously set up for each prompt, which can cause weird behaviour, e.g. when running "geth account update <input.txt" where input.txt contains three lines. In this case, the first password prompt would fill up the buffer with all lines and then use only the first one. * Print the "unsupported terminal" warning only once. Now that stdin prompting has global state, we can use it to track the warning there. * Work around small liner issues, particularly on Windows. Prompting didn't work under most of the third-party terminal emulators on Windows because liner assumes line editing is always available.
Diffstat (limited to 'cmd/geth/chaincmd.go')
-rw-r--r--cmd/geth/chaincmd.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/geth/chaincmd.go b/cmd/geth/chaincmd.go
index 868ee7db1..32eacc99e 100644
--- a/cmd/geth/chaincmd.go
+++ b/cmd/geth/chaincmd.go
@@ -116,7 +116,7 @@ func exportChain(ctx *cli.Context) {
}
func removeDB(ctx *cli.Context) {
- confirm, err := utils.PromptConfirm("Remove local database?")
+ confirm, err := utils.Stdin.ConfirmPrompt("Remove local database?")
if err != nil {
utils.Fatalf("%v", err)
}