Merge pull request #343 from andreaferretti/fix305

Fixed https://github.com/nim-lang/nimble/issues/305

src/nimblepkg/tools.nim

@@ -21,12 +21,16 @@ proc doCmd*(cmd: string, showOutput = false) =
   stderr.flushFile()
 
   displayDebug("Executing", cmd)
+  if showOutput:
+    let exitCode = execCmd(cmd)
+    displayDebug("Finished", "with exit code " & $exitCode)
+    if exitCode != QuitSuccess:
+      raise newException(NimbleError,
+          "Execution failed with exit code $1\nCommand: $2" %
+          [$exitCode, cmd])
+  else:
     let (output, exitCode) = execCmdEx(cmd)
     displayDebug("Finished", "with exit code " & $exitCode)
-  # TODO: Improve to show output in real-time.
-  if showOutput:
-    display("Output:", output, priority = HighPriority)
-  else:
     displayDebug("Output", output)
 
     if exitCode != QuitSuccess:

tests/tester.nim

@@ -194,7 +194,7 @@ test "can use nimscript's setCommand with flags":
     let (output, exitCode) = execNimble("--debug", "cr")
     let lines = output.strip.splitLines()
     check exitCode == QuitSuccess
-    check "Hello World".normalize in lines[^2].normalize
+    check inLines(lines, "Hello World")
 
 test "can use nimscript with repeated flags (issue #329)":
   cd "nimscript":