From 09fc1d3d5261f01cc0e7dcebe49f43532dd81666 Mon Sep 17 00:00:00 2001 From: Eric Hameleers Date: Sun, 12 Aug 2018 13:01:53 +0200 Subject: Prepare port of plasma-pa from gconf to gsettings But we have to wait until post 5.13.4 because the patch does not yet apply. --- kde/patch/plasma-pa.patch | 5 + .../plasma-pa/plasma-pa_enable_gsettings.patch | 583 +++++++++++++++++++++ 2 files changed, 588 insertions(+) create mode 100644 kde/patch/plasma-pa.patch create mode 100644 kde/patch/plasma-pa/plasma-pa_enable_gsettings.patch (limited to 'kde') diff --git a/kde/patch/plasma-pa.patch b/kde/patch/plasma-pa.patch new file mode 100644 index 0000000..2349f09 --- /dev/null +++ b/kde/patch/plasma-pa.patch @@ -0,0 +1,5 @@ +# Port from gconf to gsettings (wait until post 5.13.4) +# because PA 12 dropped support for gconf and this broke the advanced settings +# in the PA audio configuration: +#cat $CWD/patch/plasma-pa/plasma-pa_enable_gsettings.patch | patch -p1 --verbose || { touch ${SLACK_KDE_BUILD_DIR}/${PKGNAME}.failed ; continue ; } + diff --git a/kde/patch/plasma-pa/plasma-pa_enable_gsettings.patch b/kde/patch/plasma-pa/plasma-pa_enable_gsettings.patch new file mode 100644 index 0000000..ef212d5 --- /dev/null +++ b/kde/patch/plasma-pa/plasma-pa_enable_gsettings.patch @@ -0,0 +1,583 @@ +# https://phabricator.kde.org/D14147 +# Port from gconf to gsettings +# Step 1 was to make gconf optional in 5.13.4 +# ---------------------------------- + +diff --git a/CMakeLists.txt b/CMakeLists.txt +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -22,9 +22,6 @@ + include(FindPkgConfig) + pkg_check_modules(GCONF gconf-2.0) + pkg_check_modules(GOBJECT gobject-2.0) +-if (GCONF_FOUND AND GOBJECT_FOUND) +- set(HAVE_GCONF TRUE) +-endif() + + find_package(Qt5 ${QT_MIN_VERSION} REQUIRED COMPONENTS + Core +@@ -44,6 +41,21 @@ + find_package(PulseAudio 5.0.0 REQUIRED) + find_package(Canberra REQUIRED) + find_package(GLIB2 REQUIRED) ++find_package(GIO) ++ ++if(USE_GCONF AND USE_GSETTINGS) ++ message(FATAL_ERROR "USE_GCONF and USE_GSETTINGS cannot be used at the same time") ++endif() ++ ++# if GSetting is available and user does not force GConf build use GSettings ++if(GIO_FOUND AND GLIB2_FOUND AND NOT USE_GCONF) ++ set(USE_GSETTINGS TRUE) ++endif() ++ ++# don't use GConf if not availabe ++if (NOT GCONF_FOUND OR NOT GOBJECT_FOUND) ++ set(USE_GCONF FALSE) ++endif() + + configure_file(config.h.cmake ${CMAKE_CURRENT_BINARY_DIR}/config.h) + +diff --git a/cmake/FindGIO.cmake b/cmake/FindGIO.cmake +new file mode 100644 +--- /dev/null ++++ b/cmake/FindGIO.cmake +@@ -0,0 +1,72 @@ ++# - Try to find the GIO libraries ++# Once done this will define ++# ++# GIO_FOUND - system has GIO ++# GIO_INCLUDE_DIR - the GIO include directory ++# GIO_LIBRARIES - GIO library ++# ++# Copyright (c) 2010 Dario Freddi ++# ++# Redistribution and use is allowed according to the terms of the BSD license. ++# For details see the accompanying COPYING-CMAKE-SCRIPTS file. ++ ++if(GIO_INCLUDE_DIR AND GIO_LIBRARIES) ++ # Already in cache, be silent ++ set(GIO_FIND_QUIETLY TRUE) ++endif(GIO_INCLUDE_DIR AND GIO_LIBRARIES) ++ ++if (NOT WIN32) ++ include(UsePkgConfig) ++ pkgconfig(gio-2.0 _LibGIOIncDir _LibGIOLinkDir _LibGIOLinkFlags _LibGIOCflags) ++endif(NOT WIN32) ++ ++MESSAGE(STATUS "gio include dir: ${_LibGIOIncDir}") ++ ++# first try without default paths to respect PKG_CONFIG_PATH ++ ++find_path(GIO_MAIN_INCLUDE_DIR glib.h ++ PATH_SUFFIXES glib-2.0 ++ PATHS ${_LibGIOIncDir} ++ NO_DEFAULT_PATH) ++ ++find_path(GIO_MAIN_INCLUDE_DIR glib.h ++ PATH_SUFFIXES glib-2.0 ++ PATHS ${_LibGIOIncDir} ) ++ ++MESSAGE(STATUS "found gio main include dir: ${GIO_MAIN_INCLUDE_DIR}") ++ ++# search the glibconfig.h include dir under the same root where the library is found ++find_library(GIO_LIBRARIES ++ NAMES gio-2.0 ++ PATHS ${_LibGIOLinkDir} ++ NO_DEFAULT_PATH) ++ ++find_library(GIO_LIBRARIES ++ NAMES gio-2.0 ++ PATHS ${_LibGIOLinkDir}) ++ ++ ++get_filename_component(GIOLibDir "${GIO_LIBRARIES}" PATH) ++ ++find_path(GIO_INTERNAL_INCLUDE_DIR glibconfig.h ++ PATH_SUFFIXES glib-2.0/include ++ PATHS ${_LibGIOIncDir} "${GIOLibDir}" ${CMAKE_SYSTEM_LIBRARY_PATH} ++ NO_DEFAULT_PATH) ++ ++find_path(GIO_INTERNAL_INCLUDE_DIR glibconfig.h ++ PATH_SUFFIXES glib-2.0/include ++ PATHS ${_LibGIOIncDir} "${GIOLibDir}" ${CMAKE_SYSTEM_LIBRARY_PATH}) ++ ++set(GIO_INCLUDE_DIR "${GIO_MAIN_INCLUDE_DIR}") ++ ++# not sure if this include dir is optional or required ++# for now it is optional ++if(GIO_INTERNAL_INCLUDE_DIR) ++ set(GIO_INCLUDE_DIR ${GIO_INCLUDE_DIR} "${GIO_INTERNAL_INCLUDE_DIR}") ++endif(GIO_INTERNAL_INCLUDE_DIR) ++ ++include(FindPackageHandleStandardArgs) ++find_package_handle_standard_args(GIO DEFAULT_MSG GIO_LIBRARIES GIO_MAIN_INCLUDE_DIR) ++ ++mark_as_advanced(GIO_INCLUDE_DIR GIO_LIBRARIES) ++ +diff --git a/config.h.cmake b/config.h.cmake +--- a/config.h.cmake ++++ b/config.h.cmake +@@ -1,3 +1,4 @@ + /* config.h. Generated by cmake from config.h.cmake */ + +-#cmakedefine01 HAVE_GCONF ++#cmakedefine01 USE_GSETTINGS ++#cmakedefine01 USE_GCONF +diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt +--- a/src/CMakeLists.txt ++++ b/src/CMakeLists.txt +@@ -37,11 +37,16 @@ + set_property(SOURCE qml/dbus/osdService.xml APPEND PROPERTY CLASSNAME OsdServiceInterface) + qt5_add_dbus_interface(dbus_SRCS qml/dbus/osdService.xml osdservice) + +-if (HAVE_GCONF) ++if (USE_GCONF) + include_directories(${GCONF_INCLUDE_DIRS} ${GOBJECT_INCLUDE_DIRS}) + set(cpp_SRCS ${cpp_SRCS} gconfitem.cpp) + endif() + ++if (USE_GSETTINGS) ++ include_directories(${GIO_INCLUDE_DIRS} ${GOBJECT_INCLUDE_DIRS}) ++ set(cpp_SRCS ${cpp_SRCS} gsettingsitem.cpp) ++endif() ++ + add_library(plasma-volume-declarative SHARED ${dbus_SRCS} ${cpp_SRCS} ${qml_SRCS}) + target_link_libraries(plasma-volume-declarative + Qt5::Core +@@ -52,12 +57,18 @@ + ${PULSEAUDIO_LIBRARY} + ${PULSEAUDIO_MAINLOOP_LIBRARY} + ${CANBERRA_LIBRARIES} ++ ${GOBJECT_LIBRARIES} + ) + +-if (HAVE_GCONF) ++if (USE_GCONF) + target_link_libraries(plasma-volume-declarative + ${GCONF_LDFLAGS} +- ${GOBJECT_LDFLAGS} ++ ) ++endif() ++ ++if (USE_GSETTINGS) ++ target_link_libraries(plasma-volume-declarative ++ ${GIO_LIBRARIES} + ) + endif() + +diff --git a/src/gsettingsitem.h b/src/gsettingsitem.h +new file mode 100644 +--- /dev/null ++++ b/src/gsettingsitem.h +@@ -0,0 +1,59 @@ ++/* ++ * Copyright (C) 2018 Nicolas Fella ++ * ++ * This library is free software; you can redistribute it and/or ++ * modify it under the terms of the GNU Lesser General Public License ++ * version 2.1 as published by the Free Software Foundation. ++ * ++ * This library is distributed in the hope that it will be useful, but ++ * WITHOUT ANY WARRANTY; without even the implied warranty of ++ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU ++ * Lesser General Public License for more details. ++ * ++ * You should have received a copy of the GNU Lesser General Public ++ * License along with this library; if not, write to the Free Software ++ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA ++ * 02110-1301 USA ++ * ++ */ ++ ++#ifndef GSETTINGSITEM_H ++#define GSETTINGSITEM_H ++ ++#include ++#include ++#include ++ ++#include ++ ++class GSettingsItem : public QObject ++{ ++ Q_OBJECT ++ ++ public: ++ ++ explicit GSettingsItem(const QString &key, QObject *parent = nullptr); ++ virtual ~GSettingsItem() override; ++ ++ QVariant value(const QString &key) const; ++ void set(const QString &key, const QVariant &val); ++ ++ ++Q_SIGNALS: ++ void subtreeChanged(); ++ ++private: ++ GSettings *m_settings; ++ ++static void settingChanged(GSettings *settings, const gchar *key, gpointer data) ++{ ++ Q_UNUSED(settings) ++ Q_UNUSED(key) ++ ++ GSettingsItem *self = static_cast(data); ++ Q_EMIT self->subtreeChanged(); ++} ++ ++}; ++ ++#endif // GCONFITEM_H +diff --git a/src/gsettingsitem.cpp b/src/gsettingsitem.cpp +new file mode 100644 +--- /dev/null ++++ b/src/gsettingsitem.cpp +@@ -0,0 +1,93 @@ ++/* ++ * Copyright (C) 2018 Nicolas Fella ++ * ++ * This library is free software; you can redistribute it and/or ++ * modify it under the terms of the GNU Lesser General Public License ++ * version 2.1 as published by the Free Software Foundation. ++ * ++ * This library is distributed in the hope that it will be useful, but ++ * WITHOUT ANY WARRANTY; without even the implied warranty of ++ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU ++ * Lesser General Public License for more details. ++ * ++ * You should have received a copy of the GNU Lesser General Public ++ * License along with this library; if not, write to the Free Software ++ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA ++ * 02110-1301 USA ++ * ++ */ ++ ++#include ++#include ++#include ++ ++#include ++ ++#include "gsettingsitem.h" ++#include "debug.h" ++ ++QVariant GSettingsItem::value(const QString &key) const ++{ ++ GVariant *gvalue = g_settings_get_value(m_settings, key.toLatin1().data()); ++ ++ QVariant toReturn; ++ ++ switch (g_variant_classify(gvalue)) { ++ case G_VARIANT_CLASS_BOOLEAN: ++ toReturn = QVariant((bool)g_variant_get_boolean(gvalue)); ++ break; ++ case G_VARIANT_CLASS_STRING: ++ toReturn = QVariant(QString::fromUtf8(g_variant_get_string(gvalue, NULL))); ++ break; ++ default: ++ qCDebug(PLASMAPA()) << "Unhandled variant type in value()"; ++ } ++ ++ g_variant_unref(gvalue); ++ ++ return toReturn; ++} ++ ++void GSettingsItem::set(const QString &key, const QVariant &val) ++{ ++ ++ // It might be hard to detect the right GVariant type from ++ // complext QVariant types such as string lists or more detailed ++ // types such as integers (GVariant has different sizes), ++ // therefore we get the current value for the key and convert ++ // to QVariant using the GVariant type ++ GVariant *oldValue = g_settings_get_value(m_settings, key.toLatin1().data()); ++ GVariant *newValue; ++ ++ switch (g_variant_type_peek_string(g_variant_get_type(oldValue))[0]) { ++ case G_VARIANT_CLASS_BOOLEAN: ++ newValue = g_variant_new_boolean(val.toBool()); ++ break; ++ case G_VARIANT_CLASS_STRING: ++ newValue = g_variant_new_string(val.toString().toUtf8().constData()); ++ break; ++ default: ++ qCDebug(PLASMAPA()) << "Unhandled variant type in set()"; ++ } ++ ++ if (newValue) ++ g_settings_set_value(m_settings, key.toLatin1().data(), newValue); ++ ++ g_variant_unref(oldValue); ++} ++ ++GSettingsItem::GSettingsItem(const QString &key, QObject *parent) ++ : QObject (parent) ++{ ++ ++ m_settings = g_settings_new_with_path("org.freedesktop.pulseaudio.module-group", key.toLatin1().data()); ++ ++ g_signal_connect(m_settings, "changed", G_CALLBACK(GSettingsItem::settingChanged), this); ++} ++ ++GSettingsItem::~GSettingsItem() ++{ ++ g_settings_sync(); ++ if (m_settings) ++ g_object_unref(m_settings); ++} +diff --git a/src/kcm/package/contents/ui/Advanced.qml b/src/kcm/package/contents/ui/Advanced.qml +--- a/src/kcm/package/contents/ui/Advanced.qml ++++ b/src/kcm/package/contents/ui/Advanced.qml +@@ -73,7 +73,7 @@ + text: i18n("Add virtual output device for simultaneous output on all local sound cards") + checked: moduleManager.combineSinks + onCheckedChanged: moduleManager.combineSinks = checked; +- enabled: moduleManager.loadedModules.indexOf("module-gconf") != -1 ++ enabled: moduleManager.configModuleLoaded + visible: moduleManager.settingsSupported + } + +@@ -84,16 +84,16 @@ + text: i18n("Automatically switch all running streams when a new output becomes available") + checked: moduleManager.switchOnConnect + onCheckedChanged: moduleManager.switchOnConnect = checked; +- enabled: moduleManager.loadedModules.indexOf("module-gconf") != -1 ++ enabled: moduleManager.configModuleLoaded + visible: moduleManager.settingsSupported + } + + Label { + Layout.alignment: Qt.AlignHCenter + enabled: false + font.italic: true +- text: i18n("Requires 'module-gconf' PulseAudio module") +- visible: moduleManager.settingsSupported && moduleManager.loadedModules.indexOf("module-gconf") == -1 ++ text: i18n("Requires %1 PulseAudio module", moduleManager.configModuleName) ++ visible: moduleManager.settingsSupported && !moduleManager.configModuleLoaded + } + + Header { +diff --git a/src/modulemanager.h b/src/modulemanager.h +--- a/src/modulemanager.h ++++ b/src/modulemanager.h +@@ -29,18 +29,18 @@ + // Properties need fully qualified classes even with pointers. + #include "client.h" + +-class GConfItem; +- + namespace QPulseAudio + { +-class GConfModule; ++class ConfigModule; + + class ModuleManager : public QObject + { + Q_OBJECT + Q_PROPERTY(bool settingsSupported READ settingsSupported CONSTANT) + Q_PROPERTY(bool combineSinks READ combineSinks WRITE setCombineSinks NOTIFY combineSinksChanged) + Q_PROPERTY(bool switchOnConnect READ switchOnConnect WRITE setSwitchOnConnect NOTIFY switchOnConnectChanged) ++ Q_PROPERTY(bool configModuleLoaded READ configModuleLoaded NOTIFY loadedModulesChanged) ++ Q_PROPERTY(QString configModuleName READ configModuleName CONSTANT) + Q_PROPERTY(QStringList loadedModules READ loadedModules NOTIFY loadedModulesChanged) + public: + explicit ModuleManager(QObject *parent = nullptr); +@@ -52,6 +52,8 @@ + bool switchOnConnect() const; + void setSwitchOnConnect(bool switchOnConnect); + QStringList loadedModules() const; ++ bool configModuleLoaded() const; ++ QString configModuleName() const; + + Q_SIGNALS: + void combineSinksChanged(); +@@ -61,9 +63,9 @@ + private: + void updateLoadedModules(); + +- GConfModule *m_combineSinks; +- GConfModule *m_switchOnConnect; +- GConfModule *m_deviceManager; ++ ConfigModule *m_combineSinks; ++ ConfigModule *m_switchOnConnect; ++ ConfigModule *m_deviceManager; + QStringList m_loadedModules; + }; + +diff --git a/src/modulemanager.cpp b/src/modulemanager.cpp +--- a/src/modulemanager.cpp ++++ b/src/modulemanager.cpp +@@ -23,40 +23,54 @@ + #include "module.h" + #include "../config.h" + +-#if HAVE_GCONF ++#if USE_GSETTINGS ++#include "gsettingsitem.h" ++ ++#define PA_SETTINGS_PATH_MODULES "/org/freedesktop/pulseaudio/module-groups" ++#endif ++ ++#if USE_GCONF + #include "gconfitem.h" +-#define PA_GCONF_ROOT "/system/pulseaudio" +-#define PA_GCONF_PATH_MODULES PA_GCONF_ROOT"/modules" ++#define PA_SETTINGS_PATH_MODULES "/system/pulseaudio/modules" + #endif + + #include + + namespace QPulseAudio + { + +-#if HAVE_GCONF +-class GConfModule : public GConfItem ++#if USE_GCONF || USE_GSETTINGS ++ ++#if USE_GSETTINGS ++class ConfigModule : public GSettingsItem ++#elif USE_GCONF ++class ConfigModule : public GConfItem ++#endif + { + public: +- GConfModule(const QString &configName, const QString &moduleName, QObject *parent); ++ ConfigModule(const QString &configName, const QString &moduleName, QObject *parent); + bool isEnabled() const; + void setEnabled(bool enabled, const QVariant &args=QVariant()); + private: + QString m_moduleName; + }; + +-GConfModule::GConfModule(const QString &configName, const QString &moduleName, QObject *parent) : +- GConfItem(QStringLiteral(PA_GCONF_PATH_MODULES"/") + configName, parent), ++ConfigModule::ConfigModule(const QString &configName, const QString &moduleName, QObject *parent) : ++#if USE_GSETTINGS ++ GSettingsItem(QStringLiteral(PA_SETTINGS_PATH_MODULES"/") + configName + QStringLiteral("/"), parent), ++#elif USE_GCONF ++ GConfItem(QStringLiteral(PA_SETTINGS_PATH_MODULES"/") + configName, parent), ++#endif + m_moduleName(moduleName) + { + } + +-bool GConfModule::isEnabled() const ++bool ConfigModule::isEnabled() const + { + return value(QStringLiteral("enabled")).toBool(); + } + +-void GConfModule::setEnabled(bool enabled, const QVariant &args) ++void ConfigModule::setEnabled(bool enabled, const QVariant &args) + { + set(QStringLiteral("locked"), true); + +@@ -69,20 +83,21 @@ + } + set(QStringLiteral("locked"), false); + } +-#endif + ++#endif + + ModuleManager::ModuleManager(QObject *parent) : + QObject(parent) + { +-#if HAVE_GCONF +- m_combineSinks = new GConfModule(QStringLiteral("combine"), QStringLiteral("module-combine"), this); +- m_switchOnConnect = new GConfModule(QStringLiteral("switch-on-connect"), QStringLiteral("module-switch-on-connect"), this); +- m_deviceManager = new GConfModule(QStringLiteral("device-manager"), QStringLiteral("module-device-manager"), this); ++#if USE_GCONF || USE_GSETTINGS + +- connect(m_combineSinks, &GConfItem::subtreeChanged, this, &ModuleManager::combineSinksChanged); +- connect(m_switchOnConnect, &GConfItem::subtreeChanged, this, &ModuleManager::switchOnConnectChanged); +- connect(m_deviceManager, &GConfItem::subtreeChanged, this, &ModuleManager::switchOnConnectChanged); ++ m_combineSinks = new ConfigModule(QStringLiteral("combine"), QStringLiteral("module-combine"), this); ++ m_switchOnConnect = new ConfigModule(QStringLiteral("switch-on-connect"), QStringLiteral("module-switch-on-connect"), this); ++ m_deviceManager = new ConfigModule(QStringLiteral("device-manager"), QStringLiteral("module-device-manager"), this); ++ ++ connect(m_combineSinks, &ConfigModule::subtreeChanged, this, &ModuleManager::combineSinksChanged); ++ connect(m_switchOnConnect, &ConfigModule::subtreeChanged, this, &ModuleManager::switchOnConnectChanged); ++ connect(m_deviceManager, &ConfigModule::subtreeChanged, this, &ModuleManager::switchOnConnectChanged); + #endif + + QTimer *updateModulesTimer = new QTimer(this); +@@ -100,48 +115,48 @@ + + bool ModuleManager::settingsSupported() const + { +-#if HAVE_GCONF ++#if USE_GCONF || USE_GSETTINGS + return true; + #else + return false; + #endif + } + + bool ModuleManager::combineSinks() const + { +-#if HAVE_GCONF ++#if USE_GCONF || USE_GSETTINGS + return m_combineSinks->isEnabled(); + #else + return false; + #endif + } + + void ModuleManager::setCombineSinks(bool combineSinks) + { +-#if HAVE_GCONF ++#if USE_GCONF || USE_GSETTINGS + m_combineSinks->setEnabled(combineSinks); + #else +- Q_UNUSED(combineSinks) ++ Q_UNUSED(combineSinks() + #endif + } + + bool ModuleManager::switchOnConnect() const + { ++#if USE_GCONF || USE_GSETTINGS + //switch on connect and device-manager do the same task. Only one should be enabled + + //Note on the first run m_deviceManager will appear to be disabled even though it's actually running + //because there is no gconf entry, however m_switchOnConnect will only exist if set by Plasma PA + //hence only check this entry +-#if HAVE_GCONF + return m_switchOnConnect->isEnabled() ; + #else + return false; + #endif + } + + void ModuleManager::setSwitchOnConnect(bool switchOnConnect) + { +-#if HAVE_GCONF ++#if USE_GCONF || USE_GSETTINGS + m_deviceManager->setEnabled(!switchOnConnect); + m_switchOnConnect->setEnabled(switchOnConnect); + #else +@@ -164,4 +179,19 @@ + Q_EMIT loadedModulesChanged(); + } + ++bool ModuleManager::configModuleLoaded() const ++{ ++ return m_loadedModules.contains(configModuleName()); ++} ++ ++QString ModuleManager::configModuleName() const ++{ ++#if USE_GCONF ++ return QStringLiteral("module-gconf"); ++#elif USE_GSETTINGS ++ return QStringLiteral("module-gsettings"); ++#else ++ return QString(); ++#endif ++} + } + -- cgit v1.2.3