Skip to content

Commit

Permalink
Change paraview check call. Refs #5379
Browse files Browse the repository at this point in the history
It did call -V which ends up starting a GUI briefly and is quite slow on
OS X. Changed to --help as this is faster and we don't grab anything from
the version string anyway.
  • Loading branch information
martyngigg committed May 23, 2012
1 parent 7a02be1 commit dcdbf93
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Code/Mantid/Framework/Kernel/src/ConfigService.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1671,7 +1671,7 @@ bool ConfigServiceImpl::quickParaViewCheck() const

try
{
//Try to run "paraview -V", which will succeed if ParaView is installed.
//Try to run "paraview --help", which will succeed if ParaView is installed.
std::string paraviewDir = getString("paraview.path");
std::string cmd = "paraview";
if(!paraviewDir.empty())
Expand All @@ -1680,7 +1680,7 @@ bool ConfigServiceImpl::quickParaViewCheck() const
cmd = paraviewExe.toString();
}
std::vector<std::string> args;
args.push_back("-V");
args.push_back("--help");
Poco::Pipe outPipe;
Poco::ProcessHandle ph = Poco::Process::launch(cmd, args, 0, &outPipe, 0);
const int rc = ph.wait();
Expand Down

0 comments on commit dcdbf93

Please sign in to comment.