From 760bd5a43799aeabff50f305f659db392153c08e Mon Sep 17 00:00:00 2001 From: Albert Vaca Cintora Date: Fri, 13 Sep 2024 15:37:12 +0200 Subject: [PATCH] Revert "macOS: launch kdeconnectd with start instead of startDetached" It breaks the start for some reason. This reverts commit 2c572cf32c9093384c5ff20c446f117100424678. --- indicator/indicatorhelper_mac.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/indicator/indicatorhelper_mac.cpp b/indicator/indicatorhelper_mac.cpp index c99f907d2..c22bc6cef 100644 --- a/indicator/indicatorhelper_mac.cpp +++ b/indicator/indicatorhelper_mac.cpp @@ -135,12 +135,13 @@ int IndicatorHelper::daemonHook(QProcess &kdeconnectd) } // Start kdeconnectd, the daemon will not duplicate when there is already one - QString daemonPath = QCoreApplication::applicationDirPath() + QLatin1String("/kdeconnectd"); - if (!QFile::exists(daemonPath)) { + if (QString daemon = QCoreApplication::applicationDirPath() + QLatin1String("/kdeconnectd"); QFile::exists(daemon)) { + kdeconnectd.setProgram(daemon); + } else { QMessageBox::critical(nullptr, i18n("KDE Connect"), i18n("Cannot find kdeconnectd"), QMessageBox::Abort, QMessageBox::Abort); return -1; } - kdeconnectd.start(daemonPath, QStringList()); + kdeconnectd.startDetached(); m_splashScreen->showMessage(i18n("Loading modules") + QStringLiteral("\n"), Qt::AlignHCenter | Qt::AlignBottom, Qt::white);