Remove the "waitForStarted()" function call in the external process launcher functions, and adjust the manual check for process started/running/finished to also account for the "Starting" State (only breaks out for the "NotRunning" process state now).

1 files changed, 5 insertions, 5 deletions

diff --git a/libLumina/LuminaUtils.cpp b/libLumina/LuminaUtils.cpp
index 7b13adb9..987e3c5d 100644
--- a/libLumina/LuminaUtils.cpp
+++ b/libLumina/LuminaUtils.cpp
@@ -36,9 +36,9 @@ int LUtils::runCmd(QString cmd, QStringList args){
   }else{
     proc.start(cmd, args);
   }
-  if(!proc.waitForStarted(30000)){ return 1; } //process never started - max wait of 30 seconds
+  //if(!proc.waitForStarted(30000)){ return 1; } //process never started - max wait of 30 seconds
   while(!proc.waitForFinished(300)){
-    if(proc.state() != QProcess::Running){ break; } //somehow missed the finished signal
+    if(proc.state() == QProcess::NotRunning){ break; } //somehow missed the finished signal
     QCoreApplication::processEvents();
   }
   int ret = proc.exitCode();
@@ -58,9 +58,9 @@ QStringList LUtils::getCmdOutput(QString cmd, QStringList args){
   }else{
     proc.start(cmd,args);	  
   }
-  if(!proc.waitForStarted(30000)){ return QStringList(); } //process never started - max wait of 30 seconds
-  while(!proc.waitForFinished(500)){
-    if(proc.state() != QProcess::Running){ break; } //somehow missed the finished signal
+  //if(!proc.waitForStarted(30000)){ return QStringList(); } //process never started - max wait of 30 seconds
+  while(!proc.waitForFinished(300)){
+    if(proc.state() != QProcess::NotRunning){ break; } //somehow missed the finished signal
     QCoreApplication::processEvents();
   }
   QStringList out = QString(proc.readAllStandardOutput()).split("\n");