From b0e0b3ecda895d1bf8eb8607b4fda16fe4aad472 Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Wed, 14 Jul 2021 15:20:37 +0200 Subject: [PATCH] CMakePM: Use Utils::FilePath for accessing cmake tools When storing the cmake tools we otherwise drop auto detected ones when failing to access them locally. This makes the cmake tools on docker devices persist on Windows as long the docker daemon is running. Change-Id: I5235cf00aff49359a2e20ddc1667bcbb99337092 Reviewed-by: hjk --- src/plugins/cmakeprojectmanager/cmaketoolsettingsaccessor.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/plugins/cmakeprojectmanager/cmaketoolsettingsaccessor.cpp b/src/plugins/cmakeprojectmanager/cmaketoolsettingsaccessor.cpp index b1dde077b67..6e38e67a0d0 100644 --- a/src/plugins/cmakeprojectmanager/cmaketoolsettingsaccessor.cpp +++ b/src/plugins/cmakeprojectmanager/cmaketoolsettingsaccessor.cpp @@ -211,9 +211,9 @@ void CMakeToolSettingsAccessor::saveCMakeTools(const QList &cmakeTo int count = 0; for (const CMakeTool *item : cmakeTools) { - QFileInfo fi = item->cmakeExecutable().toFileInfo(); + Utils::FilePath fi = item->cmakeExecutable(); - if (fi.isExecutable()) { + if (fi.isExecutableFile()) { QVariantMap tmp = item->toMap(); if (tmp.isEmpty()) continue;