Perforce: Move logging the error directly into the synchronousProcess()

Change-Id: I9d090f2da36d3868e9b4ca2a7f297be62e36e65a
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
This commit is contained in:
Jarek Kobus
2024-01-21 22:01:14 +01:00
parent a4177fb238
commit b92b0d4100

View File

@@ -1220,6 +1220,9 @@ PerforceResponse PerforcePluginPrivate::synchronousProcess(const FilePath &worki
response.stdErr = process.cleanedStdErr();
response.stdOut = process.cleanedStdOut();
response.message = process.exitMessage();
if (response.error && (flags & ErrorToWindow))
VcsOutputWindow::appendError(process.exitMessage());
return response;
}
@@ -1261,9 +1264,6 @@ PerforceResponse PerforcePluginPrivate::runP4Cmd(const FilePath &workingDir,
if (flags & ShowBusyCursor)
QGuiApplication::restoreOverrideCursor();
if (response.error && (flags & ErrorToWindow))
VcsOutputWindow::appendError(response.message);
return response;
}