Merge pull request #1836 from hawson/fix-1760-history_file

Fix for #1760: influx doesn't create $HOME/.influx_history
influxdata · Mar 3, 2015 · adee51c · adee51c
2 parents cd18255 + 0f7c69c
commit adee51c
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/cmd/influx/main.go b/cmd/influx/main.go
@@ -98,15 +98,18 @@ func main() {
 		if e != nil {
 			break
 		}
-		if !c.ParseCommand(l) {
+		if c.ParseCommand(l) {
 			// write out the history
-			if f, err := os.Create(historyFile); err == nil {
-				c.Line.WriteHistory(f)
-				f.Close()
+			if len(historyFile) > 0 {
+				c.Line.AppendHistory(l)
+				if f, err := os.Create(historyFile); err == nil {
+					c.Line.WriteHistory(f)
+					f.Close()
+				}
 			}
-			break
+		} else {
+			break // exit main loop
 		}
-		c.Line.AppendHistory(l)
 	}
 }