From 50c47c785d70916a8e0bd53c1a49d9b239a3c3ab Mon Sep 17 00:00:00 2001 From: Ryan Huffman Date: Fri, 8 Dec 2017 17:45:42 -0800 Subject: [PATCH] Fix BakeAssetTask not handling case where oven process is not found --- assignment-client/src/assets/BakeAssetTask.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/assignment-client/src/assets/BakeAssetTask.cpp b/assignment-client/src/assets/BakeAssetTask.cpp index 427f41c494..69123e4909 100644 --- a/assignment-client/src/assets/BakeAssetTask.cpp +++ b/assignment-client/src/assets/BakeAssetTask.cpp @@ -115,7 +115,12 @@ void BakeAssetTask::run() { qDebug() << "Starting oven for " << _assetPath; _ovenProcess->start(path, args, QIODevice::ReadOnly); - _ovenProcess->waitForStarted(); + if (!_ovenProcess->waitForStarted(-1)) { + _didFinish.store(true); + QString errors = "Oven process failed to start"; + emit bakeFailed(_assetHash, _assetPath, errors); + return; + } _ovenProcess->waitForFinished(); }