diff options
Diffstat (limited to 'gui-apps/waybar')
-rw-r--r-- | gui-apps/waybar/Manifest | 10 | ||||
-rw-r--r-- | gui-apps/waybar/files/waybar-0.9.13-libcxx-build.patch | 36 | ||||
-rw-r--r-- | gui-apps/waybar/files/waybar-0.9.17-wireplumber.patch | 374 | ||||
-rw-r--r-- | gui-apps/waybar/waybar-0.9.12.ebuild | 72 | ||||
-rw-r--r-- | gui-apps/waybar/waybar-0.9.16.ebuild | 3 | ||||
-rw-r--r-- | gui-apps/waybar/waybar-0.9.17.ebuild (renamed from gui-apps/waybar/waybar-0.9.13.ebuild) | 9 |
6 files changed, 385 insertions, 119 deletions
diff --git a/gui-apps/waybar/Manifest b/gui-apps/waybar/Manifest index b96e0f5e01ef..9b7db95dd7c0 100644 --- a/gui-apps/waybar/Manifest +++ b/gui-apps/waybar/Manifest @@ -1,9 +1,7 @@ -AUX waybar-0.9.13-libcxx-build.patch 1193 BLAKE2B 71ca297fd87a5b6fcdf5dcaa11de100b2b953e4973c537169c32ca82efdd651cdb5cc22e41bcbc5e18f6b07f4733755f1588a456f47788201ba5368daa1bc356 SHA512 18b7a7ef2d2fd18c5364a5da972a0cd2aef35be05495f365c21fbe368d8d7147bf92db1c4c14373e8cfde8b1a8f71783b527759e982a1ced753f8c9c6a783b0c -DIST waybar-0.9.12.tar.gz 183392 BLAKE2B 697a2fce74af8337dfbfc8d8823cf4f2acfda6a0dd009cea03ec2b79358a80db0935418436a51e375be2f702e2d53f9f3403ab01cb1e91e246c7e4cc01effdf5 SHA512 c035140ca0d47ae2cbc0313e25dc206c116b5b6f7a4e6192c7d40a9bd0ef18b8bfafdc8aeea2a85d670dccceda1e3942b6b3e3a451f51a6e68d25edf0ccb1b05 -DIST waybar-0.9.13.tar.gz 198160 BLAKE2B 222863b1048d2833be393875cad947647eccde3fcae335c48d4606843b3b9445717c18955351c56779196afaf9a5333ae7f718ea590521a79359372450d494a9 SHA512 d0a87044bcccdd16b1bbb20de750804c06335f11e638d37b5b80b72233e513378ba521f5c9d2c121a78a206877487628a6562cb080dd43eadae27e52d4cf6349 +AUX waybar-0.9.17-wireplumber.patch 15607 BLAKE2B eb5ad8d86cf5881bcaa4bcde7baee10c0c0c6add3005c0e13dcc86fa0e6212d1e49900642edb8ddaff14ee1cfdf338c16e6b06e8c9a94309f0220c1a39dc5703 SHA512 1ab4da1c0f1c5c62a8054a7e7ce8c81141a15abc053fbff8c3d98e78d6934b6dcacd04d52841d0ea98fe7507298d33bf896f0a4ba3a55f4610aec0f9c8e3045b DIST waybar-0.9.16.tar.gz 215154 BLAKE2B c1e655dc57d3950472e22b23bd7bb7f96dd5daa1fc1ffee871a3677486f9f41981dbea8f559f8097e9c7bc367b0bcf224119af556ca62006f0e68e380b91fdec SHA512 d89067639de8ad560cd1515bb448f47f5994281b35239f069a2012165c0f83e0a3ae861cffbb93e7636d7d95cece9424e5900d40a06b80e71768aabda5f7b54a -EBUILD waybar-0.9.12.ebuild 1686 BLAKE2B 74268b2f60c60b2e549f7c2d6fb6c4716ba904851cd3209710e0ceb909c164ba138001a37be8dfa469ce6a30a978bec84ca923167296bdc1fa06c32a1ba4950d SHA512 6e8e552d54d7cd4791dc4a42e18db655b578be3ee9d4ac5dad9406ad0b9ed15dea8c4b6101c0de0b0a24443f6d2e24dd140f054dd56704be82709d3d0501b78f -EBUILD waybar-0.9.13.ebuild 1738 BLAKE2B ea4c1d9f2d46f0f6e8379152f62b0e8bc34456ba2cb7f83c408627874e4d349d10c5201bba32b39a3b11eb97d9dc64b91a6b103bd651978e03d4bb2740e4cc8a SHA512 76033a85795bb6969e9539713244ad2beb38630eb506ee8080cd4e14c3fcffc58062497f16272c4c4647ceea5f3d71d5dcf664d189e50f822ccc2a990b4e8ec6 -EBUILD waybar-0.9.16.ebuild 1701 BLAKE2B 1def271ef1a013cb75c9cba3043667abad1bc2b3b027f2160561ce5fffcf14ce20d017bb9c2be7429560da7e84bd9133e0cc418d0e5aeb32b6101a67f969d56b SHA512 c72daf205b0867316c45dbb1dcd4d77b1aa1d0a8c7adee281a96a4f050b4458def94f7213cd32deaad9f8b4fa4f1844b922d9369642a640dc10665c13ee738de +DIST waybar-0.9.17.tar.gz 220836 BLAKE2B 78a9065f4b20c1047ce5296c8eb3b6d05ecb1e4b9994b250937e7f681cf6d7f773a5de59189544eab90ad91cfd2abf170ec76c1b125a1d7b09a22f8a7a9d43e0 SHA512 51aba7649c0e2ad3ccf22f6ede3cdf9be9c84268d31d39c5eac62e150e3b115f5560b70ebd54242fd856abec0cca5eba405f1ec153cac4cd6623523f6b19189f +EBUILD waybar-0.9.16.ebuild 1724 BLAKE2B f5ed70964a390e5ee8970ec8b90c5510d2ce8643bce7d9143e9f725fc878fdde7517d7f570ad05abbcdda5d202099400dfb88e97c2458efc9a1ef706cd299545 SHA512 0c05b7ccef38626b964b493c2aca14aefee022e85d9b963f00db9cbb04b09adf423863301f3a1ae0f2822bcb2a0677afde79d78b3acd6a13d4caa04e369afe7b +EBUILD waybar-0.9.17.ebuild 1784 BLAKE2B d1a5381088407105efe98e931343995dace4e2edae39b46cb384b4c11ebc6b13443488dd7c779a9275d6d7e8d4cbd003acdf6eacdeb95b7f91ecaf67a5d0858d SHA512 dfe159672a5a191f8299b50723ea09c9311bb5fbd826b284cdd40b48379675ef4ccc7c8ce20012d86caa40b5ceb712f8b3ef07cacc5af9c86d6073dec0e18b45 EBUILD waybar-9999.ebuild 1632 BLAKE2B a3b478c3aab3a41e0df3d06d93bc76d3fe20f187b29c9d56e61535a0cf5b0ff2a93bc40d0d90bfc88b800ee28ac7829422482728b0edb2ca0501a36d3a97228d SHA512 708c52f2030a5c72aa4c56bd0e1871633bb3641bfe3b77d518c2ac7907966663c4dcac1fccfc6ab4e16e27a96f41d09658911fae4fdf259ac822c069c62630c4 MISC metadata.xml 1280 BLAKE2B 83dcff4c9798e9a6013b1b74894095cd07802373cc9b7d0230d3ab1ef9f574f0701053314a8906fbe0074946b90763afc322e3ec921e78b5703304bc22e8f5c1 SHA512 fcd07931cee04e7050e9f7cf2e711a5ff1194f10aa602ad222e19e2708d701cd8c0fb9535c42282715abaac2c85d5456e5bc8058f0f96bd716245114316f1d17 diff --git a/gui-apps/waybar/files/waybar-0.9.13-libcxx-build.patch b/gui-apps/waybar/files/waybar-0.9.13-libcxx-build.patch deleted file mode 100644 index ad24e22fc437..000000000000 --- a/gui-apps/waybar/files/waybar-0.9.13-libcxx-build.patch +++ /dev/null @@ -1,36 +0,0 @@ -https://github.com/Alexays/Waybar/commit/d25278f710a4932798c98141bcf5b482de0e9166 - -From: Jan Beich <jbeich@FreeBSD.org> -Date: Mon, 23 May 2022 16:23:00 +0000 -Subject: [PATCH] fix(upower): add missing include for libc++ - -In file included from src/modules/upower/upower.cpp:1: -include/modules/upower/upower.hpp:25:16: error: no template named 'unordered_map' in namespace 'std' - typedef std::unordered_map<std::string, UpDevice *> Devices; - ~~~~~^ -In file included from src/modules/upower/upower_tooltip.cpp:1: -include/modules/upower/upower_tooltip.hpp:13:16: error: no template named 'unordered_map' in namespace 'std' - typedef std::unordered_map<std::string, UpDevice*> Devices; - ~~~~~^ ---- a/include/modules/upower/upower.hpp -+++ b/include/modules/upower/upower.hpp -@@ -5,6 +5,7 @@ - #include <iostream> - #include <map> - #include <string> -+#include <unordered_map> - - #include "ALabel.hpp" - #include "glibconfig.h" ---- a/include/modules/upower/upower_tooltip.hpp -+++ b/include/modules/upower/upower_tooltip.hpp -@@ -2,6 +2,8 @@ - - #include <libupower-glib/upower.h> - -+#include <unordered_map> -+ - #include "gtkmm/box.h" - #include "gtkmm/label.h" - #include "gtkmm/window.h" - diff --git a/gui-apps/waybar/files/waybar-0.9.17-wireplumber.patch b/gui-apps/waybar/files/waybar-0.9.17-wireplumber.patch new file mode 100644 index 000000000000..9db19e2a2b5d --- /dev/null +++ b/gui-apps/waybar/files/waybar-0.9.17-wireplumber.patch @@ -0,0 +1,374 @@ +Commit: https://github.com/Alexays/Waybar/commit/4e8ccf36b54cacf5281726d23ea14312a133f977 +Commit: https://github.com/Alexays/Waybar/commit/8da5425189d52ebbaab063c9289c5de5adbe0ca5 +Issue: https://github.com/Alexays/Waybar/issues/1852 +From 4e8ccf36b54cacf5281726d23ea14312a133f977 Mon Sep 17 00:00:00 2001 +From: Sasha Moak <sasha.moak@gmail.com> +Date: Thu, 12 Jan 2023 16:17:11 -0800 +Subject: [PATCH] fix(wireplumber): waybar crashes when default node changes + +In order to fix the issue, the default node name is cached rather than +the default node id. This is due to ids being unstable. So now when the +object manager is installed (ie ready), the default node name is +retrieved and stored for later. + +Now when the mixer changed signal is emitted, the id of the changed node +is used to get the node from the object manager. The nodes name is +grabbed off that node and compared against the default node name, if +they match the volume is updated. Some safeguarding has been added such +that if the node cannot be found off the object manager, it's ignored. + +Additionally, the "changed" signal on the default nodes api is now +utilized to update the default node name if it has changed. This way if +the default node changes, the module will be updated with the correct +volume and node.nick. + +This adds additional debug logging for helping diagnose wireplumber +issues. + +This also adds the wireplumber man page entry to the main waybar +supported section. +--- + include/modules/wireplumber.hpp | 10 +- + man/waybar.5.scd.in | 1 + + src/modules/wireplumber.cpp | 184 ++++++++++++++++++++++++++------ + 3 files changed, 157 insertions(+), 38 deletions(-) + +diff --git a/include/modules/wireplumber.hpp b/include/modules/wireplumber.hpp +index c0ee7f0be..fa988fcf3 100644 +--- a/include/modules/wireplumber.hpp ++++ b/include/modules/wireplumber.hpp +@@ -20,15 +20,19 @@ class Wireplumber : public ALabel { + void loadRequiredApiModules(); + void prepare(); + void activatePlugins(); +- static void updateVolume(waybar::modules::Wireplumber* self); +- static void updateNodeName(waybar::modules::Wireplumber* self); +- static uint32_t getDefaultNodeId(waybar::modules::Wireplumber* self); ++ static void updateVolume(waybar::modules::Wireplumber* self, uint32_t id); ++ static void updateNodeName(waybar::modules::Wireplumber* self, uint32_t id); + static void onPluginActivated(WpObject* p, GAsyncResult* res, waybar::modules::Wireplumber* self); + static void onObjectManagerInstalled(waybar::modules::Wireplumber* self); ++ static void onMixerChanged(waybar::modules::Wireplumber* self, uint32_t id); ++ static void onDefaultNodesApiChanged(waybar::modules::Wireplumber* self); + + WpCore* wp_core_; + GPtrArray* apis_; + WpObjectManager* om_; ++ WpPlugin* mixer_api_; ++ WpPlugin* def_nodes_api_; ++ gchar* default_node_name_; + uint32_t pending_plugins_; + bool muted_; + double volume_; +diff --git a/man/waybar.5.scd.in b/man/waybar.5.scd.in +index b1ed4c527..7566dd000 100644 +--- a/man/waybar.5.scd.in ++++ b/man/waybar.5.scd.in +@@ -277,6 +277,7 @@ A module group is defined by specifying a module named "group/some-group-name". + - *waybar-sway-scratchpad(5)* + - *waybar-sway-window(5)* + - *waybar-sway-workspaces(5)* ++- *waybar-wireplumber(5)* + - *waybar-wlr-taskbar(5)* + - *waybar-wlr-workspaces(5)* + - *waybar-temperature(5)* +diff --git a/src/modules/wireplumber.cpp b/src/modules/wireplumber.cpp +index 9a12a9b5b..9652e1e2b 100644 +--- a/src/modules/wireplumber.cpp ++++ b/src/modules/wireplumber.cpp +@@ -1,15 +1,22 @@ + #include "modules/wireplumber.hpp" + ++#include <spdlog/spdlog.h> ++ ++bool isValidNodeId(uint32_t id) { return id > 0 && id < G_MAXUINT32; } ++ + waybar::modules::Wireplumber::Wireplumber(const std::string& id, const Json::Value& config) + : ALabel(config, "wireplumber", id, "{volume}%"), + wp_core_(nullptr), + apis_(nullptr), + om_(nullptr), ++ mixer_api_(nullptr), ++ def_nodes_api_(nullptr), ++ default_node_name_(nullptr), + pending_plugins_(0), + muted_(false), + volume_(0.0), + node_id_(0) { +- wp_init(WP_INIT_ALL); ++ wp_init(WP_INIT_PIPEWIRE); + wp_core_ = wp_core_new(NULL, NULL); + apis_ = g_ptr_array_new_with_free_func(g_object_unref); + om_ = wp_object_manager_new(); +@@ -18,10 +25,15 @@ waybar::modules::Wireplumber::Wireplumber(const std::string& id, const Json::Val + + loadRequiredApiModules(); + ++ spdlog::debug("[{}]: connecting to pipewire...", this->name_); ++ + if (!wp_core_connect(wp_core_)) { ++ spdlog::error("[{}]: Could not connect to PipeWire", this->name_); + throw std::runtime_error("Could not connect to PipeWire\n"); + } + ++ spdlog::debug("[{}]: connected!", this->name_); ++ + g_signal_connect_swapped(om_, "installed", (GCallback)onObjectManagerInstalled, this); + + activatePlugins(); +@@ -33,33 +45,26 @@ waybar::modules::Wireplumber::~Wireplumber() { + g_clear_pointer(&apis_, g_ptr_array_unref); + g_clear_object(&om_); + g_clear_object(&wp_core_); ++ g_clear_object(&mixer_api_); ++ g_clear_object(&def_nodes_api_); ++ g_free(&default_node_name_); + } + +-uint32_t waybar::modules::Wireplumber::getDefaultNodeId(waybar::modules::Wireplumber* self) { +- uint32_t id; +- g_autoptr(WpPlugin) def_nodes_api = wp_plugin_find(self->wp_core_, "default-nodes-api"); ++void waybar::modules::Wireplumber::updateNodeName(waybar::modules::Wireplumber* self, uint32_t id) { ++ spdlog::debug("[{}]: updating node name with node.id {}", self->name_, id); + +- if (!def_nodes_api) { +- throw std::runtime_error("Default nodes API is not loaded\n"); ++ if (!isValidNodeId(id)) { ++ spdlog::warn("[{}]: '{}' is not a valid node ID. Ignoring node name update.", self->name_, id); ++ return; + } + +- g_signal_emit_by_name(def_nodes_api, "get-default-node", "Audio/Sink", &id); +- +- if (id <= 0 || id >= G_MAXUINT32) { +- auto err = fmt::format("'{}' is not a valid ID (returned by default-nodes-api)\n", id); +- throw std::runtime_error(err); +- } +- +- return id; +-} +- +-void waybar::modules::Wireplumber::updateNodeName(waybar::modules::Wireplumber* self) { +- auto proxy = static_cast<WpProxy*>( +- wp_object_manager_lookup(self->om_, WP_TYPE_GLOBAL_PROXY, WP_CONSTRAINT_TYPE_G_PROPERTY, +- "bound-id", "=u", self->node_id_, NULL)); ++ auto proxy = static_cast<WpProxy*>(wp_object_manager_lookup( ++ self->om_, WP_TYPE_GLOBAL_PROXY, WP_CONSTRAINT_TYPE_G_PROPERTY, "bound-id", "=u", id, NULL)); + + if (!proxy) { +- throw std::runtime_error(fmt::format("Object '{}' not found\n", self->node_id_)); ++ auto err = fmt::format("Object '{}' not found\n", id); ++ spdlog::error("[{}]: {}", self->name_, err); ++ throw std::runtime_error(err); + } + + g_autoptr(WpProperties) properties = +@@ -73,15 +78,24 @@ void waybar::modules::Wireplumber::updateNodeName(waybar::modules::Wireplumber* + auto description = wp_properties_get(properties, "node.description"); + + self->node_name_ = nick ? nick : description; ++ spdlog::debug("[{}]: Updating node name to: {}", self->name_, self->node_name_); + } + +-void waybar::modules::Wireplumber::updateVolume(waybar::modules::Wireplumber* self) { ++void waybar::modules::Wireplumber::updateVolume(waybar::modules::Wireplumber* self, uint32_t id) { ++ spdlog::debug("[{}]: updating volume", self->name_); + double vol; + GVariant* variant = NULL; +- g_autoptr(WpPlugin) mixer_api = wp_plugin_find(self->wp_core_, "mixer-api"); +- g_signal_emit_by_name(mixer_api, "get-volume", self->node_id_, &variant); ++ ++ if (!isValidNodeId(id)) { ++ spdlog::error("[{}]: '{}' is not a valid node ID. Ignoring volume update.", self->name_, id); ++ return; ++ } ++ ++ g_signal_emit_by_name(self->mixer_api_, "get-volume", id, &variant); ++ + if (!variant) { +- auto err = fmt::format("Node {} does not support volume\n", self->node_id_); ++ auto err = fmt::format("Node {} does not support volume\n", id); ++ spdlog::error("[{}]: {}", self->name_, err); + throw std::runtime_error(err); + } + +@@ -93,22 +107,121 @@ void waybar::modules::Wireplumber::updateVolume(waybar::modules::Wireplumber* se + self->dp.emit(); + } + ++void waybar::modules::Wireplumber::onMixerChanged(waybar::modules::Wireplumber* self, uint32_t id) { ++ spdlog::debug("[{}]: (onMixerChanged) - id: {}", self->name_, id); ++ ++ g_autoptr(WpNode) node = static_cast<WpNode*>(wp_object_manager_lookup( ++ self->om_, WP_TYPE_NODE, WP_CONSTRAINT_TYPE_G_PROPERTY, "bound-id", "=u", id, NULL)); ++ ++ if (!node) { ++ spdlog::warn("[{}]: (onMixerChanged) - Object with id {} not found", self->name_, id); ++ return; ++ } ++ ++ const gchar* name = wp_pipewire_object_get_property(WP_PIPEWIRE_OBJECT(node), "node.name"); ++ ++ if (g_strcmp0(self->default_node_name_, name) != 0) { ++ spdlog::debug( ++ "[{}]: (onMixerChanged) - ignoring mixer update for node: id: {}, name: {} as it is not " ++ "the default node: {}", ++ self->name_, id, name, self->default_node_name_); ++ return; ++ } ++ ++ spdlog::debug("[{}]: (onMixerChanged) - Need to update volume for node with id {} and name {}", ++ self->name_, id, name); ++ updateVolume(self, id); ++} ++ ++void waybar::modules::Wireplumber::onDefaultNodesApiChanged(waybar::modules::Wireplumber* self) { ++ spdlog::debug("[{}]: (onDefaultNodesApiChanged)", self->name_); ++ ++ uint32_t default_node_id; ++ g_signal_emit_by_name(self->def_nodes_api_, "get-default-node", "Audio/Sink", &default_node_id); ++ ++ if (!isValidNodeId(default_node_id)) { ++ spdlog::warn("[{}]: '{}' is not a valid node ID. Ignoring node change.", self->name_, ++ default_node_id); ++ return; ++ } ++ ++ g_autoptr(WpNode) node = static_cast<WpNode*>( ++ wp_object_manager_lookup(self->om_, WP_TYPE_NODE, WP_CONSTRAINT_TYPE_G_PROPERTY, "bound-id", ++ "=u", default_node_id, NULL)); ++ ++ if (!node) { ++ spdlog::warn("[{}]: (onDefaultNodesApiChanged) - Object with id {} not found", self->name_, ++ default_node_id); ++ return; ++ } ++ ++ const gchar* default_node_name = ++ wp_pipewire_object_get_property(WP_PIPEWIRE_OBJECT(node), "node.name"); ++ ++ spdlog::debug( ++ "[{}]: (onDefaultNodesApiChanged) - got the following default node: Node(name: {}, id: {})", ++ self->name_, default_node_name, default_node_id); ++ ++ if (g_strcmp0(self->default_node_name_, default_node_name) == 0) { ++ spdlog::debug( ++ "[{}]: (onDefaultNodesApiChanged) - Default node has not changed. Node(name: {}, id: {}). " ++ "Ignoring.", ++ self->name_, self->default_node_name_, default_node_id); ++ return; ++ } ++ ++ spdlog::debug( ++ "[{}]: (onDefaultNodesApiChanged) - Default node changed to -> Node(name: {}, id: {})", ++ self->name_, default_node_name, default_node_id); ++ ++ self->default_node_name_ = g_strdup(default_node_name); ++ updateVolume(self, default_node_id); ++ updateNodeName(self, default_node_id); ++} ++ + void waybar::modules::Wireplumber::onObjectManagerInstalled(waybar::modules::Wireplumber* self) { +- self->node_id_ = +- self->config_["node-id"].isInt() ? self->config_["node-id"].asInt() : getDefaultNodeId(self); ++ spdlog::debug("[{}]: onObjectManagerInstalled", self->name_); ++ ++ self->def_nodes_api_ = wp_plugin_find(self->wp_core_, "default-nodes-api"); ++ ++ if (!self->def_nodes_api_) { ++ spdlog::error("[{}]: default nodes api is not loaded.", self->name_); ++ throw std::runtime_error("Default nodes API is not loaded\n"); ++ } ++ ++ self->mixer_api_ = wp_plugin_find(self->wp_core_, "mixer-api"); ++ ++ if (!self->mixer_api_) { ++ spdlog::error("[{}]: mixer api is not loaded.", self->name_); ++ throw std::runtime_error("Mixer api is not loaded\n"); ++ } ++ ++ uint32_t default_node_id; ++ g_signal_emit_by_name(self->def_nodes_api_, "get-default-configured-node-name", "Audio/Sink", ++ &self->default_node_name_); ++ g_signal_emit_by_name(self->def_nodes_api_, "get-default-node", "Audio/Sink", &default_node_id); ++ ++ if (self->default_node_name_) { ++ spdlog::debug("[{}]: (onObjectManagerInstalled) - default configured node name: {} and id: {}", ++ self->name_, self->default_node_name_, default_node_id); ++ } + +- g_autoptr(WpPlugin) mixer_api = wp_plugin_find(self->wp_core_, "mixer-api"); ++ updateVolume(self, default_node_id); ++ updateNodeName(self, default_node_id); + +- updateVolume(self); +- updateNodeName(self); +- g_signal_connect_swapped(mixer_api, "changed", (GCallback)updateVolume, self); ++ g_signal_connect_swapped(self->mixer_api_, "changed", (GCallback)onMixerChanged, self); ++ g_signal_connect_swapped(self->def_nodes_api_, "changed", (GCallback)onDefaultNodesApiChanged, ++ self); + } + + void waybar::modules::Wireplumber::onPluginActivated(WpObject* p, GAsyncResult* res, + waybar::modules::Wireplumber* self) { ++ auto plugin_name = wp_plugin_get_name(WP_PLUGIN(p)); ++ spdlog::debug("[{}]: onPluginActivated: {}", self->name_, plugin_name); + g_autoptr(GError) error = NULL; + + if (!wp_object_activate_finish(p, res, &error)) { ++ spdlog::error("[{}]: error activating plugin: {}", self->name_, error->message); + throw std::runtime_error(error->message); + } + +@@ -118,6 +231,7 @@ void waybar::modules::Wireplumber::onPluginActivated(WpObject* p, GAsyncResult* + } + + void waybar::modules::Wireplumber::activatePlugins() { ++ spdlog::debug("[{}]: activating plugins", name_); + for (uint16_t i = 0; i < apis_->len; i++) { + WpPlugin* plugin = static_cast<WpPlugin*>(g_ptr_array_index(apis_, i)); + pending_plugins_++; +@@ -127,13 +241,13 @@ void waybar::modules::Wireplumber::activatePlugins() { + } + + void waybar::modules::Wireplumber::prepare() { +- wp_object_manager_add_interest(om_, WP_TYPE_NODE, NULL); +- wp_object_manager_add_interest(om_, WP_TYPE_GLOBAL_PROXY, NULL); +- wp_object_manager_request_object_features(om_, WP_TYPE_GLOBAL_PROXY, +- WP_PIPEWIRE_OBJECT_FEATURES_MINIMAL); ++ spdlog::debug("[{}]: preparing object manager", name_); ++ wp_object_manager_add_interest(om_, WP_TYPE_NODE, WP_CONSTRAINT_TYPE_PW_PROPERTY, "media.class", ++ "=s", "Audio/Sink", NULL); + } + + void waybar::modules::Wireplumber::loadRequiredApiModules() { ++ spdlog::debug("[{}]: loading required modules", name_); + g_autoptr(GError) error = NULL; + + if (!wp_core_load_component(wp_core_, "libwireplumber-module-default-nodes-api", "module", NULL, +From a9c9f1d705991c7f6ff9de7eac3430a219011978 Mon Sep 17 00:00:00 2001 +From: Sasha Moak <sasha.moak@gmail.com> +Date: Tue, 31 Jan 2023 17:56:58 -0800 +Subject: [PATCH] fix(wireplumber): free(): invalid pointer + +When freeing the `default_node_name_` pointer using `free`, the `&` +operator was used to try to free the reference rather than the pointer. +This caused a core dump. In order to fix this, the pointer is freed +instead (ie the `&` operator is no longer used). +--- + src/modules/wireplumber.cpp | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/src/modules/wireplumber.cpp b/src/modules/wireplumber.cpp +index fd1a0d389..4c7a2d0b1 100644 +--- a/src/modules/wireplumber.cpp ++++ b/src/modules/wireplumber.cpp +@@ -47,7 +47,7 @@ waybar::modules::Wireplumber::~Wireplumber() { + g_clear_object(&wp_core_); + g_clear_object(&mixer_api_); + g_clear_object(&def_nodes_api_); +- g_free(&default_node_name_); ++ g_free(default_node_name_); + } + + void waybar::modules::Wireplumber::updateNodeName(waybar::modules::Wireplumber* self, uint32_t id) { diff --git a/gui-apps/waybar/waybar-0.9.12.ebuild b/gui-apps/waybar/waybar-0.9.12.ebuild deleted file mode 100644 index 6fbac657f700..000000000000 --- a/gui-apps/waybar/waybar-0.9.12.ebuild +++ /dev/null @@ -1,72 +0,0 @@ -# Copyright 1999-2022 Gentoo Authors -# Distributed under the terms of the GNU General Public License v2 - -EAPI=8 - -inherit meson - -DESCRIPTION="Highly customizable Wayland bar for Sway and Wlroots based compositors" -HOMEPAGE="https://github.com/Alexays/Waybar" - -if [[ ${PV} == 9999 ]]; then - inherit git-r3 - EGIT_REPO_URI="https://github.com/Alexays/${PN^}.git" -else - SRC_URI="https://github.com/Alexays/${PN^}/archive/${PV}.tar.gz -> ${P}.tar.gz" - KEYWORDS="~amd64" -fi -S="${WORKDIR}/${PN^}-${PV}" - -LICENSE="MIT" -SLOT="0" -IUSE="mpd network +popups pulseaudio sndio tray +udev wifi" - -BDEPEND=" - >=app-text/scdoc-1.9.2 - dev-util/gdbus-codegen - virtual/pkgconfig -" -DEPEND=" - dev-cpp/cairomm:0 - dev-cpp/glibmm:2 - dev-cpp/gtkmm:3.0 - dev-libs/glib:2 - dev-libs/jsoncpp:= - dev-libs/libinput:= - dev-libs/libsigc++:2 - >=dev-libs/libfmt-7.0.0:= - <dev-libs/libfmt-9.0.0:= - >=dev-libs/spdlog-1.8.5:= - dev-libs/date:= - dev-libs/wayland - dev-libs/wayland-protocols - gui-libs/wlroots:= - x11-libs/gtk+:3[wayland] - x11-libs/libxkbcommon - mpd? ( media-libs/libmpdclient ) - network? ( dev-libs/libnl:3 ) - popups? ( gui-libs/gtk-layer-shell ) - pulseaudio? ( media-sound/pulseaudio ) - sndio? ( media-sound/sndio:= ) - tray? ( - dev-libs/libdbusmenu[gtk3] - dev-libs/libappindicator - ) - udev? ( virtual/libudev:= ) - wifi? ( || ( sys-apps/util-linux net-wireless/rfkill ) ) -" -RDEPEND="${DEPEND}" - -src_configure() { - local emesonargs=( - $(meson_feature mpd) - $(meson_feature network libnl) - $(meson_feature popups gtk-layer-shell) - $(meson_feature pulseaudio) - $(meson_feature sndio) - $(meson_feature tray dbusmenu-gtk) - $(meson_feature udev libudev) - $(meson_feature wifi rfkill) - ) - meson_src_configure -} diff --git a/gui-apps/waybar/waybar-0.9.16.ebuild b/gui-apps/waybar/waybar-0.9.16.ebuild index 006b9fc848cb..ed6e3ca97877 100644 --- a/gui-apps/waybar/waybar-0.9.16.ebuild +++ b/gui-apps/waybar/waybar-0.9.16.ebuild @@ -1,4 +1,4 @@ -# Copyright 1999-2022 Gentoo Authors +# Copyright 1999-2023 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 EAPI=8 @@ -23,6 +23,7 @@ IUSE="experimental mpd network +popups pulseaudio sndio tray +udev wifi" BDEPEND=" >=app-text/scdoc-1.9.2 + >=dev-cpp/catch-3.0.0 dev-util/gdbus-codegen virtual/pkgconfig " diff --git a/gui-apps/waybar/waybar-0.9.13.ebuild b/gui-apps/waybar/waybar-0.9.17.ebuild index c4886a844841..9d2f830e6558 100644 --- a/gui-apps/waybar/waybar-0.9.13.ebuild +++ b/gui-apps/waybar/waybar-0.9.17.ebuild @@ -1,4 +1,4 @@ -# Copyright 1999-2022 Gentoo Authors +# Copyright 1999-2023 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 EAPI=8 @@ -19,10 +19,11 @@ S="${WORKDIR}/${PN^}-${PV}" LICENSE="MIT" SLOT="0" -IUSE="mpd network +popups pulseaudio sndio tray +udev wifi" +IUSE="experimental mpd network +popups pulseaudio sndio tray +udev wifi" BDEPEND=" >=app-text/scdoc-1.9.2 + >=dev-cpp/catch-3.0.1 dev-util/gdbus-codegen virtual/pkgconfig " @@ -35,7 +36,6 @@ DEPEND=" dev-libs/libinput:= dev-libs/libsigc++:2 >=dev-libs/libfmt-7.0.0:= - <dev-libs/libfmt-9.0.0:= >=dev-libs/spdlog-1.8.5:= dev-libs/date:= dev-libs/wayland @@ -58,7 +58,7 @@ DEPEND=" RDEPEND="${DEPEND}" PATCHES=( - "${FILESDIR}"/${P}-libcxx-build.patch + "${FILESDIR}/waybar-0.9.17-wireplumber.patch" ) src_configure() { @@ -71,6 +71,7 @@ src_configure() { $(meson_feature tray dbusmenu-gtk) $(meson_feature udev libudev) $(meson_feature wifi rfkill) + $(meson_use experimental) ) meson_src_configure } |