fix: move the stdio ignoring code to the correct function: the incorr…

…ect fix was regression `ct` generally

i applied the fix in a wrong way yesterday, which was leading to an exception
whenever running `readProcessOutput`, as it does use the stdio

src/frontend/lib.nim

@@ -434,12 +434,8 @@ when defined(ctIndex) or defined(ctTest):
       # debugPrint "OPTIONS: ", $(options.to(cstring))
 
       setupLdLibraryPath()
-
-      var processOptions = options
-      # important to ignore stderr, as otherwise too much of it can lead to
-      # the spawned process hanging: this is a bugfix for such a situation
-      processOptions.stdio = cstring"ignore"
-      let process = nodeStartProcess.spawn(path, args, processOptions)
+
+      let process = nodeStartProcess.spawn(path, args, options)
 
       process.stdout.setEncoding(cstring"utf8")
 
@@ -472,6 +468,11 @@ when defined(ctIndex) or defined(ctTest):
 
       let process = nodeStartProcess.spawn(path, args, options)
 
+      var processOptions = options
+      # important to ignore stderr, as otherwise too much of it can lead to
+      # the spawned process hanging: this is a bugfix for such a situation
+      processOptions.stdio = cstring"ignore"
+
       process.toJs.on("spawn", proc() =
         resolve(Result[NodeSubProcess, JsObject].ok(process)))