diff options
author | Felix Lange <fjl@twurst.com> | 2016-03-21 21:05:22 +0800 |
---|---|---|
committer | Felix Lange <fjl@twurst.com> | 2016-04-12 21:56:49 +0800 |
commit | dff9b4246f3ef9e6c254b57eef6d0433809f16b9 (patch) | |
tree | 71cab4093300a6e9bd1b5801f05e3507246eef7e /cmd/geth/js_test.go | |
parent | 83877a0f9d3d5716ee01393f10c2dfa19bb0310b (diff) | |
download | go-tangerine-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.gz go-tangerine-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.zst go-tangerine-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/js_test.go')
-rw-r--r-- | cmd/geth/js_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/geth/js_test.go b/cmd/geth/js_test.go index e0c4dacbc..efdb9ab86 100644 --- a/cmd/geth/js_test.go +++ b/cmd/geth/js_test.go @@ -94,7 +94,7 @@ func testREPL(t *testing.T, config func(*eth.Config)) (string, *testjethre, *nod t.Fatal(err) } // Create a networkless protocol stack - stack, err := node.New(&node.Config{PrivateKey: testNodeKey, Name: "test", NoDiscovery: true}) + stack, err := node.New(&node.Config{DataDir: tmp, PrivateKey: testNodeKey, Name: "test", NoDiscovery: true}) if err != nil { t.Fatalf("failed to create node: %v", err) } |