From 9dd9c464936ca5003240531bd45c8f2dc6032415 Mon Sep 17 00:00:00 2001 From: Claudio Cambra Date: Mon, 6 Jun 2022 19:05:12 +0200 Subject: [PATCH] Do not compile things that require the app bundle when building the client without the app bundle on macOS Signed-off-by: Claudio Cambra --- CMakeLists.txt | 1 + src/gui/systray.cpp | 6 +++--- src/gui/updater/updater.cpp | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 0176eaf9af3e2..d6c978f72b227 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -105,6 +105,7 @@ if(APPLE AND BUILD_OWNCLOUD_OSX_BUNDLE) set(OWNCLOUD_OSX_BUNDLE "${APPLICATION_NAME}.app") set(LIB_INSTALL_DIR "${APPLICATION_NAME}.app/Contents/MacOS") set(BIN_INSTALL_DIR "${APPLICATION_NAME}.app/Contents/MacOS") + add_definitions(-DBUILD_OWNCLOUD_OSX_BUNDLE) endif() diff --git a/src/gui/systray.cpp b/src/gui/systray.cpp index db04b01a43dda..7de521ca7ec52 100644 --- a/src/gui/systray.cpp +++ b/src/gui/systray.cpp @@ -99,7 +99,7 @@ Systray::Systray() qmlRegisterType("com.nextcloud.desktopclient", 1, 0, "WheelHandler"); -#ifdef Q_OS_MACOS +#if defined(Q_OS_MACOS) && defined(BUILD_OWNCLOUD_OSX_BUNDLE) setUserNotificationCenterDelegate(); checkNotificationAuth(MacNotificationAuthorizationOptions::Default); // No provisional auth, ask user explicitly first time registerNotificationCategories(QString(tr("Download"))); @@ -307,7 +307,7 @@ void Systray::showMessage(const QString &title, const QString &message, MessageI QDBusConnection::sessionBus().asyncCall(method); } else #endif -#ifdef Q_OS_OSX +#if defined(Q_OS_MACOS) && defined(BUILD_OWNCLOUD_OSX_BUNDLE) if (canOsXSendUserNotification()) { sendOsXUserNotification(title, message); } else @@ -319,7 +319,7 @@ void Systray::showMessage(const QString &title, const QString &message, MessageI void Systray::showUpdateMessage(const QString &title, const QString &message, const QUrl &webUrl) { -#ifdef Q_OS_MACOS +#if defined(Q_OS_MACOS) && defined(BUILD_OWNCLOUD_OSX_BUNDLE) sendOsXUpdateNotification(title, message, webUrl); #else // TODO: Implement custom notifications (i.e. actionable) for other OSes Q_UNUSED(webUrl); diff --git a/src/gui/updater/updater.cpp b/src/gui/updater/updater.cpp index af8fad47b26ca..cb3dd310738ca 100644 --- a/src/gui/updater/updater.cpp +++ b/src/gui/updater/updater.cpp @@ -141,7 +141,7 @@ Updater *Updater::create() return nullptr; } -#if defined(Q_OS_MAC) && defined(HAVE_SPARKLE) +#if defined(Q_OS_MACOS) && defined(HAVE_SPARKLE) && defined(BUILD_OWNCLOUD_OSX_BUNDLE) return new SparkleUpdater(url); #elif defined(Q_OS_WIN32) // Also for MSI