napcok/calamares
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[libcalamares] Improve reporting on process failures

Browse Source 
- In production, cDebug() might not show up, so the log will not
  contain the lines saying what program is being run;
- Errors should at least mention the program name, but "env" or
  "chroot" is not useful, so pull that from *args*, which is
  the command we actually want to run.
This commit is contained in:
Adriaan de Groot 2020-04-06 11:17:47 +02:00
parent 752399ca6b
commit 4abb87ccca
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/libcalamares/utils/CalamaresUtilsSystem.cpp
View File

@ -181,7 +181,7 @@ System::runCommand( System::RunLocation location,
process.start();
if ( !process.waitForStarted() )
{
cWarning() << "Process failed to start" << process.error();
cWarning() << "Process" << args.first() << "failed to start" << process.error();
return ProcessResult::Code::FailedToStart;
}
@ -195,7 +195,7 @@ System::runCommand( System::RunLocation location,
? ( static_cast< int >( std::chrono::milliseconds( timeoutSec ).count() ) )
: -1 ) )
{
cWarning().noquote().nospace() << "Timed out. Output so far:\n" << process.readAllStandardOutput();
cWarning().noquote().nospace() << "Process" << args.first() << "timed out. Output so far:\n" << process.readAllStandardOutput();
return ProcessResult::Code::TimedOut;
}
@ -203,7 +203,7 @@ System::runCommand( System::RunLocation location,
if ( process.exitStatus() == QProcess::CrashExit )
{
cWarning().noquote().nospace() << "Process crashed. Output so far:\n" << output;
cWarning().noquote().nospace() << "Process" << args.first() << "crashed. Output so far:\n" << output;
return ProcessResult::Code::Crashed;
}