Fix windows build issues

- Add dummy args param to QProcess::startDetached
- Remove logging category from inline snippet. This was removed due to
  being unused in non-windows builds.
This commit is contained in:
Alexander Lohnau 2023-07-21 14:53:23 +02:00
parent d3421a570e
commit b21f5b02e3
4 changed files with 6 additions and 5 deletions

View file

@ -173,7 +173,7 @@ int main(int argc, char *argv[])
#ifdef Q_OS_WIN
// make sure indicator shows up in the tray whenever daemon is spawned
QProcess::startDetached(QStringLiteral("kdeconnect-indicator.exe"));
QProcess::startDetached(QStringLiteral("kdeconnect-indicator.exe"), QStringList());
#endif
// kdeconnectd is autostarted, so disable session management to speed up startup

View file

@ -60,7 +60,7 @@ void IndicatorHelper::iconPathHook()
int IndicatorHelper::daemonHook(QProcess &kdeconnectd)
{
kdeconnectd.start(processes::kdeconnect_daemon);
kdeconnectd.start(processes::kdeconnect_daemon, QStringList());
return 0;
}

View file

@ -11,6 +11,7 @@
#include <core/kdeconnectplugin.h>
#ifdef Q_OS_WIN
#include <QDebug>
#include <Windows.h>
#define INFO_BUFFER_SIZE 32767
#else
@ -42,7 +43,7 @@ inline QString defaultSound()
#ifdef Q_OS_WIN
wchar_t infoBuf[INFO_BUFFER_SIZE];
if (!GetWindowsDirectory(infoBuf, INFO_BUFFER_SIZE)) {
qCWarning(KDECONNECT_PLUGIN_FINDTHISDEVICE) << "Error with getting the Windows Directory.";
qWarning() << "Error with getting the Windows Directory.";
} else {
dirPath = QString::fromStdWString(infoBuf) + QStringLiteral("/media");
if (!dirPath.isEmpty()) {

View file

@ -55,9 +55,9 @@ bool RunCommandPlugin::receivePacket(const NetworkPacket &np)
const QJsonObject commandJson = value.toObject();
qCInfo(KDECONNECT_PLUGIN_RUNCOMMAND) << "Running:" << COMMAND << ARGS << commandJson[QStringLiteral("command")].toString();
#ifdef Q_OS_WIN
QProcess::startDetached(commandJson[QStringLiteral("command")].toString());
QProcess::startDetached(commandJson[QStringLiteral("command")].toString(), QStringList());
#else
QProcess::startDetached(QStringLiteral(COMMAND), QStringList() << QStringLiteral(ARGS) << commandJson[QStringLiteral("command")].toString());
QProcess::startDetached(QStringLiteral(COMMAND), QStringList{QStringLiteral(ARGS), commandJson[QStringLiteral("command")].toString()});
#endif
return true;
} else if (np.has(QStringLiteral("setup"))) {