diff --git a/src/classes/PMsr2Data.cpp b/src/classes/PMsr2Data.cpp index 24c9110c..f9bbd177 100644 --- a/src/classes/PMsr2Data.cpp +++ b/src/classes/PMsr2Data.cpp @@ -1308,7 +1308,9 @@ bool PMsr2Data::PrepareGlobalInputFile(unsigned int tempRun, const std::string & std::ostringstream oss; oss << path << "musrfit" << " " << *fRunVectorIter << "-OneRunFit" << fFileExtension << ".msr"; std::cout << std::endl << ">> msr2data: **INFO** Calling " << oss.str() << std::endl; - system(oss.str().c_str()); + if (system(oss.str().c_str()) == -1) { + std::cerr << std::endl << "**ERROR** system call: " << oss.str().c_str() << " failed." << std::endl; + } ++fRunVectorIter; } diff --git a/src/classes/PRunDataHandler.cpp b/src/classes/PRunDataHandler.cpp index c0503e0f..4ef8c6bc 100644 --- a/src/classes/PRunDataHandler.cpp +++ b/src/classes/PRunDataHandler.cpp @@ -724,7 +724,9 @@ Bool_t PRunDataHandler::ReadWriteFilesList() sprintf(cmd, "tar -zcf %s %s", fln.Data(), fAny2ManyInfo->outPathFileName[0].Data()); else // bzip2 sprintf(cmd, "tar -jcf %s %s", fln.Data(), fAny2ManyInfo->outPathFileName[0].Data()); - system(cmd); + if (system(cmd) == -1) { + std::cerr << "**ERROR** cmd: " << cmd << " failed." << std::endl; + } } else { fln += TString(".tar"); for (UInt_t i=0; ioutPathFileName.size(); i++) { @@ -733,7 +735,9 @@ Bool_t PRunDataHandler::ReadWriteFilesList() } else { sprintf(cmd, "tar -rf %s %s", fln.Data(), fAny2ManyInfo->outPathFileName[i].Data()); } - system(cmd); + if (system(cmd) == -1) { + std::cerr << "**ERROR** cmd: " << cmd << " failed." << std::endl; + } } if (fAny2ManyInfo->compressionTag == 1) { // gzip sprintf(cmd, "gzip %s", fln.Data()); @@ -742,7 +746,9 @@ Bool_t PRunDataHandler::ReadWriteFilesList() sprintf(cmd, "bzip2 -z %s", fln.Data()); fln += ".bz2"; } - system(cmd); + if (system(cmd) == -1) { + std::cerr << "**ERROR** cmd: " << cmd << " failed." << std::endl; + } } // check if the compressed file shall be streamed to the stdout diff --git a/src/msr2data.cpp b/src/msr2data.cpp index 0fb7a621..e61e6eb6 100644 --- a/src/msr2data.cpp +++ b/src/msr2data.cpp @@ -893,7 +893,9 @@ int main(int argc, char *argv[]) std::ostringstream oss; oss << path << "musrfit" << " " << strInfile.str() << " " << musrfitOptions; std::cout << std::endl << ">> msr2data: **INFO** Calling " << oss.str() << std::endl; - system(oss.str().c_str()); + if (system(oss.str().c_str()) == -1) { + std::cerr << "**ERROR** cmd: " << oss.str().c_str() << " failed." << std::endl; + } } } @@ -982,7 +984,9 @@ int main(int argc, char *argv[]) std::ostringstream oss; oss << path << "musrfit" << " " << strInfile.str() << " " << musrfitOptions; std::cout << std::endl << ">> msr2data: **INFO** Calling " << oss.str() << std::endl; - system(oss.str().c_str()); + if (system(oss.str().c_str()) == -1) { + std::cerr << "**ERROR** cmd: " << oss.str().c_str() << " failed." << std::endl; + } } } diff --git a/src/msr2msr.cpp b/src/msr2msr.cpp index 03ef1c49..99b5ef02 100644 --- a/src/msr2msr.cpp +++ b/src/msr2msr.cpp @@ -574,10 +574,16 @@ bool msr2msr_finalize_theory(char *fln, int theoryTag, int noOfAddionalParams) // cp __temp.msr fln sprintf(str, "cp __temp.msr %s", fln); - system(str); + if (system(str) == -1) { + std::cerr << "**ERROR** cmd: " << str << " failed." << std::endl; + return false; + } // rm __temp.msr strcpy(str, "rm __temp.msr"); - system(str); + if (system(str) == -1) { + std::cerr << "**ERROR** cmd: " << str << " failed." << std::endl; + return false; + } return true; } @@ -704,7 +710,10 @@ int main(int argc, char *argv[]) // check if conversion seems to be OK if (!success) { sprintf(str, "rm -rf %s", argv[2]); - system(str); + if (system(str) == -1) { + std::cerr << "**ERROR** cmd: " << str << " failed." << std::endl; + return 0; + } } if (theoryTag != -1) { diff --git a/src/musredit_qt5/musredit/PDumpOutputHandler.cpp b/src/musredit_qt5/musredit/PDumpOutputHandler.cpp index 5e310333..29c90648 100644 --- a/src/musredit_qt5/musredit/PDumpOutputHandler.cpp +++ b/src/musredit_qt5/musredit/PDumpOutputHandler.cpp @@ -123,7 +123,14 @@ PDumpOutputHandler::~PDumpOutputHandler() #else qDebug() << msg << Qt::endl; #endif - system(cmd.toLatin1()); + if (system(cmd.toLatin1()) == -1) { +#if (QT_VERSION < QT_VERSION_CHECK(5, 14, 0)) + qDebug() << "**ERROR** " << msg << " failed." << endl; +#else + qDebug() << "**ERROR** " << msg << " failed." << Qt::endl; +#endif + + } } if (fProc) { delete fProc; diff --git a/src/musredit_qt5/musredit/PFitOutputHandler.cpp b/src/musredit_qt5/musredit/PFitOutputHandler.cpp index bdfe737e..d31bd8cc 100644 --- a/src/musredit_qt5/musredit/PFitOutputHandler.cpp +++ b/src/musredit_qt5/musredit/PFitOutputHandler.cpp @@ -126,7 +126,13 @@ PFitOutputHandler::~PFitOutputHandler() #else qDebug() << msg << Qt::endl; #endif - system(cmd.toLatin1()); + if (system(cmd.toLatin1()) == -1) { +#if (QT_VERSION < QT_VERSION_CHECK(5, 14, 0)) + qDebug() << "**ERROR** " << msg << " - failed!" << endl; +#else + qDebug() << "**ERROR** " << msg << " - failed!" << Qt::endl; +#endif + } } if (fProc) { delete fProc;