From 8768e39d3c8e74e583eca3897cc6de53a99c3dde Mon Sep 17 00:00:00 2001 From: Hannes Domani Date: Mon, 24 Sep 2018 17:41:27 +0200 Subject: Valgrind: Add target executable suffix to heob command line The suffix is needed in case the executable name contains a dot, because then the .exe suffix is not added automatically by CreateProcess(). Change-Id: Ief2cfeaa8dd8ebbfb71f69575ee9574a4f2156d3 Reviewed-by: Orgad Shaneh Reviewed-by: Eike Ziller --- src/plugins/valgrind/memchecktool.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/plugins/valgrind/memchecktool.cpp b/src/plugins/valgrind/memchecktool.cpp index 6078cd240c..e67c1b3762 100644 --- a/src/plugins/valgrind/memchecktool.cpp +++ b/src/plugins/valgrind/memchecktool.cpp @@ -780,8 +780,9 @@ void MemcheckTool::heobAction() TaskHub::requestPopup(); return; } - if (!QFile::exists(executable) - && !QFile::exists(Utils::HostOsInfo::withExecutableSuffix(executable))) { + if (!QFile::exists(executable)) + executable = Utils::HostOsInfo::withExecutableSuffix(executable); + if (!QFile::exists(executable)) { const QString msg = tr("Heob: Cannot find %1.").arg(executable); TaskHub::addTask(Task::Error, msg, Debugger::Constants::ANALYZERTASK_ID); TaskHub::requestPopup(); -- cgit v1.2.3