parse: fix arg number check

This should fix issue #1196 (Can't launch a command line process from Qt). The check was bad because it took stdin into account, but it really shouldn't. License: MIT Signed-off-by: Christian Couder <[email protected]>
ipfs · May 17, 2015 · 53a80e2 · 53a80e2
1 parent c93d583
commit 53a80e2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/commands/cli/parse.go b/commands/cli/parse.go
@@ -228,8 +228,8 @@ func parseArgs(inputs []string, stdin *os.File, argDefs []cmds.Argument, recursi
 	// if we have more arg values provided than argument definitions,
 	// and the last arg definition is not variadic (or there are no definitions), return an error
 	notVariadic := len(argDefs) == 0 || !argDefs[len(argDefs)-1].Variadic
-	if notVariadic && numInputs > len(argDefs) {
-		return nil, nil, fmt.Errorf("Expected %v arguments, got %v: %v", len(argDefs), numInputs, inputs)
+	if notVariadic && len(inputs) > len(argDefs) {
+		return nil, nil, fmt.Errorf("Expected %v arguments, got %v: %v", len(argDefs), len(inputs), inputs)
 	}
 
 	stringArgs := make([]string, 0, numInputs)