From 1b9afa41f731eff20cbafa53c0129145d68b4c3b Mon Sep 17 00:00:00 2001 From: Zhenguo Niu Date: Mon, 19 Mar 2018 16:20:38 +0800 Subject: [PATCH] cmd/geth: small change of copydb source path check This changes to check the first arg length instead of checking if there's only one arg for the required source path to follow more conventions. --- cmd/geth/chaincmd.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cmd/geth/chaincmd.go b/cmd/geth/chaincmd.go index c9ab72b6df..8cbef220ac 100644 --- a/cmd/geth/chaincmd.go +++ b/cmd/geth/chaincmd.go @@ -305,7 +305,8 @@ func exportChain(ctx *cli.Context) error { func copyDb(ctx *cli.Context) error { // Ensure we have a source chain directory to copy - if len(ctx.Args()) != 1 { + dbPath := ctx.Args().First() + if len(dbPath) == 0 { utils.Fatalf("Source chaindata directory path argument missing") } // Initialize a new chain for the running node to sync into @@ -316,7 +317,7 @@ func copyDb(ctx *cli.Context) error { dl := downloader.New(syncmode, chainDb, new(event.TypeMux), chain, nil, nil) // Create a source peer to satisfy downloader requests from - db, err := ethdb.NewLDBDatabase(ctx.Args().First(), ctx.GlobalInt(utils.CacheFlag.Name), 256) + db, err := ethdb.NewLDBDatabase(dbPath, ctx.GlobalInt(utils.CacheFlag.Name), 256) if err != nil { return err }