Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package gnome-control-center for
openSUSE:Factory checked in at 2026-01-27 16:05:51
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gnome-control-center (Old)
and /work/SRC/openSUSE:Factory/.gnome-control-center.new.1928 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "gnome-control-center"
Tue Jan 27 16:05:51 2026 rev:278 rq:1329053 version:49.3
Changes:
--------
---
/work/SRC/openSUSE:Factory/gnome-control-center/gnome-control-center.changes
2026-01-06 17:43:36.863256140 +0100
+++
/work/SRC/openSUSE:Factory/.gnome-control-center.new.1928/gnome-control-center.changes
2026-01-27 16:06:12.410706594 +0100
@@ -1,0 +2,7 @@
+Thu Jan 22 08:51:58 UTC 2026 - Alynx Zhou <[email protected]>
+
+- Add gnome-control-center-gvc-31.patch: Fix crash with newer
+ pipewire on switching bluetooth profile
+ (glgo#GNOME/libgnome-volume-control!31).
+
+-------------------------------------------------------------------
New:
----
gnome-control-center-gvc-31.patch
----------(New B)----------
New:
- Add gnome-control-center-gvc-31.patch: Fix crash with newer
pipewire on switching bluetooth profile
----------(New E)----------
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ gnome-control-center.spec ++++++
--- /var/tmp/diff_new_pack.rvjC9E/_old 2026-01-27 16:06:13.686760759 +0100
+++ /var/tmp/diff_new_pack.rvjC9E/_new 2026-01-27 16:06:13.690760928 +0100
@@ -36,6 +36,9 @@
Source1: libgxdp-0.gitmodule.tar.zst
Source99: %{name}-rpmlintrc
+# PATCH-FIX-UPSTREAM gnome-control-center-gvc-31.patch
glgo#GNOME/libgnome-volume-control!31 [email protected] -- Fix crash with
newer pipewire on switching bluetooth profile
+Patch1: gnome-control-center-gvc-31.patch
+
### patches for Leap >= 15 plus SLE >= 15, but not TW
# PATCH-FEATURE-SLE gnome-control-center-system-never-use-gnome-software.patch
bsc#999336 [email protected] -- info: Never search for gnome-software as an
option when checking for updates on SLE and Leap 42.2, because we use
gpk-update-viewer.
Patch1001: gnome-control-center-system-never-use-gnome-software.patch
++++++ gnome-control-center-49.3.obscpio ++++++
++++++ gnome-control-center-gvc-31.patch ++++++
diff --unified --recursive --text --new-file --color
gnome-control-center-49.3/subprojects/gvc/gvc-mixer-control.c
gnome-control-center-49.3.new/subprojects/gvc/gvc-mixer-control.c
--- gnome-control-center-49.3/subprojects/gvc/gvc-mixer-control.c
2026-01-05 21:33:53.000000000 +0800
+++ gnome-control-center-49.3.new/subprojects/gvc/gvc-mixer-control.c
2026-01-22 16:39:20.347647032 +0800
@@ -239,7 +239,6 @@
GvcMixerStream *stream)
{
GList *devices, *d;
- gboolean is_network_stream;
const GList *ports;
GvcMixerUIDevice *ret;
@@ -253,43 +252,44 @@
ret = NULL;
ports = gvc_mixer_stream_get_ports (stream);
- is_network_stream = (ports == NULL);
for (d = devices; d != NULL; d = d->next) {
GvcMixerUIDevice *device = d->data;
guint stream_id = G_MAXUINT;
+ const GvcMixerStreamPort *port;
g_object_get (G_OBJECT (device),
"stream-id", &stream_id,
NULL);
- if (is_network_stream &&
- stream_id == gvc_mixer_stream_get_id (stream)) {
- g_debug ("lookup device from stream - %s - it is a
network_stream ",
+ if (stream_id != gvc_mixer_stream_get_id (stream))
+ continue;
+
+ if (!ports) {
+ g_debug ("lookup-device-from-stream - %s - is portless
",
gvc_mixer_ui_device_get_description (device));
ret = device;
break;
- } else if (!is_network_stream) {
- const GvcMixerStreamPort *port;
- port = gvc_mixer_stream_get_port (stream);
+ }
- if (stream_id == gvc_mixer_stream_get_id (stream) &&
- g_strcmp0 (gvc_mixer_ui_device_get_port (device),
+ port = gvc_mixer_stream_get_port (stream);
+ if (g_strcmp0 (gvc_mixer_ui_device_get_port (device),
port->port) == 0) {
- g_debug ("lookup-device-from-stream found
device: device description '%s', device port = '%s', device stream id %i AND
stream port = '%s' stream id '%u' and stream description '%s'",
- gvc_mixer_ui_device_get_description
(device),
- gvc_mixer_ui_device_get_port (device),
- stream_id,
- port->port,
- gvc_mixer_stream_get_id (stream),
- gvc_mixer_stream_get_description
(stream));
- ret = device;
- break;
- }
+ g_debug ("lookup-device-from-stream found device:
device description '%s', device port = '%s', device stream id %i AND stream
port = '%s' stream id '%u' and stream description '%s'",
+ gvc_mixer_ui_device_get_description (device),
+ gvc_mixer_ui_device_get_port (device),
+ stream_id,
+ port->port,
+ gvc_mixer_stream_get_id (stream),
+ gvc_mixer_stream_get_description (stream));
+ ret = device;
+ break;
+
}
}
- g_debug ("gvc_mixer_control_lookup_device_from_stream - Could not find
a device for stream '%s'",gvc_mixer_stream_get_description (stream));
+ if (!ret)
+ g_debug ("lookup-device-from-stream - Could not find a device
for stream '%s'", gvc_mixer_stream_get_description (stream));
g_list_free (devices);
@@ -340,7 +340,6 @@
gvc_mixer_control_set_default_source (GvcMixerControl *control,
GvcMixerStream *stream)
{
- GvcMixerUIDevice* input;
pa_operation *o;
g_return_val_if_fail (GVC_IS_MIXER_CONTROL (control), FALSE);
@@ -372,13 +371,6 @@
pa_operation_unref (o);
- /* source change successful, update the UI. */
- input = gvc_mixer_control_lookup_device_from_stream (control, stream);
- g_signal_emit (G_OBJECT (control),
- signals[ACTIVE_INPUT_UPDATE],
- 0,
- gvc_mixer_ui_device_get_id (input));
-
return TRUE;
}
@@ -532,7 +524,7 @@
* @profile: (allow-none): Can be %NULL if any profile present on this port is
okay
*
* Returns: This method will attempt to swap the profile on the card of
- * the device with given profile name. If successfull it will set the
+ * the device with given profile name. If successful it will set the
* preferred profile on that device so as we know the next time the user
* moves to that device it should have this profile active.
*/
@@ -584,7 +576,7 @@
* - Firstly it queries the stream from the device.
* - It assumes that if the stream is null that it cannot be a bluetooth or
network stream (they never show unless they have valid sinks and sources)
* In the scenario of a NULL stream on the device
- * - It fetches the device's preferred profile or if NUll the profile
with the highest priority on that device.
+ * - It fetches the device's preferred profile or if NULL the profile
with the highest priority on that device.
* - It then caches this device in
control->priv->cached_desired_output_id so that when the update_sink triggered
* from when we attempt to change profile we will know exactly what
device to highlight on that stream.
* - It attempts to swap the profile on the card from that device and
returns.
@@ -616,14 +608,7 @@
if (!gvc_mixer_ui_device_has_ports (output)) {
g_debug ("Did we try to move to a software/bluetooth sink ?");
- if (gvc_mixer_control_set_default_sink (control, stream)) {
- /* sink change was successful, update the UI.*/
- g_signal_emit (G_OBJECT (control),
- signals[ACTIVE_OUTPUT_UPDATE],
- 0,
- gvc_mixer_ui_device_get_id (output));
- }
- else {
+ if (gvc_mixer_control_set_default_sink (control, stream) ==
FALSE) {
g_warning ("Failed to set default sink with stream
from output %s",
gvc_mixer_ui_device_get_description
(output));
}
@@ -645,23 +630,11 @@
/* Finally if we are not on the correct stream, swap over. */
if (stream != default_stream) {
- GvcMixerUIDevice* device;
-
g_debug ("Attempting to swap over to stream %s ",
gvc_mixer_stream_get_description (stream));
- if (gvc_mixer_control_set_default_sink (control, stream)) {
- device = gvc_mixer_control_lookup_device_from_stream
(control, stream);
- g_signal_emit (G_OBJECT (control),
- signals[ACTIVE_OUTPUT_UPDATE],
- 0,
- gvc_mixer_ui_device_get_id (device));
- } else {
- /* If the move failed for some reason reset the UI. */
- device = gvc_mixer_control_lookup_device_from_stream
(control, default_stream);
- g_signal_emit (G_OBJECT (control),
- signals[ACTIVE_OUTPUT_UPDATE],
- 0,
- gvc_mixer_ui_device_get_id (device));
+ if (gvc_mixer_control_set_default_sink (control, stream) ==
FALSE) {
+ g_warning ("Failed to set default sink from output %s",
+ gvc_mixer_ui_device_get_description
(output));
}
}
}
@@ -675,7 +648,7 @@
* - Firstly it queries the stream from the device.
* - It assumes that if the stream is null that it cannot be a bluetooth or
network stream (they never show unless they have valid sinks and sources)
* In the scenario of a NULL stream on the device
- * - It fetches the device's preferred profile or if NUll the profile
with the highest priority on that device.
+ * - It fetches the device's preferred profile or if NULL the profile
with the highest priority on that device.
* - It then caches this device in
control->priv->cached_desired_input_id so that when the update_source triggered
* from when we attempt to change profile we will know exactly what
device to highlight on that stream.
* - It attempts to swap the profile on the card from that device and
returns.
@@ -729,7 +702,10 @@
if (stream != default_stream) {
g_debug ("change-input - attempting to swap over to stream %s",
gvc_mixer_stream_get_description (stream));
- gvc_mixer_control_set_default_source (control, stream);
+ if (gvc_mixer_control_set_default_source (control, stream) ==
FALSE) {
+ g_warning ("Failed to set default source from input
%s",
+ gvc_mixer_ui_device_get_description
(input));
+ }
}
}
@@ -1003,11 +979,14 @@
new_id);
input = gvc_mixer_control_lookup_device_from_stream (control,
stream);
-
- g_signal_emit (G_OBJECT (control),
- signals[ACTIVE_INPUT_UPDATE],
- 0,
- gvc_mixer_ui_device_get_id (input));
+ if (input) {
+ g_signal_emit (G_OBJECT (control),
+ signals[ACTIVE_INPUT_UPDATE],
+ 0,
+ gvc_mixer_ui_device_get_id (input));
+ } else {
+ g_warning ("Can't find input for stream-id %d",new_id);
+ }
}
}
@@ -1235,7 +1214,7 @@
GvcMixerStreamPort *stream_port,
GvcMixerStream *stream)
{
- GList *devices, *d;
+ g_autoptr (GList) devices = NULL;
guint stream_card_id;
guint stream_id;
gboolean in_possession = FALSE;
@@ -1245,12 +1224,12 @@
devices = g_hash_table_get_values (GVC_IS_MIXER_SOURCE (stream) ?
control->priv->ui_inputs : control->priv->ui_outputs);
- for (d = devices; d != NULL; d = d->next) {
+ for (GList *d = devices; d != NULL; d = d->next) {
GvcMixerUIDevice *device;
guint device_stream_id;
- gchar *device_port_name;
- gchar *origin;
- gchar *description;
+ g_autofree gchar *device_port_name = NULL;
+ g_autofree gchar *origin = NULL;
+ g_autofree gchar *description = NULL;
GvcMixerCard *card;
guint card_id;
@@ -1280,33 +1259,63 @@
stream_port->port,
stream_card_id);
- if (stream_card_id == card_id &&
- g_strcmp0 (device_port_name, stream_port->port) ==
0) {
- g_debug ("Match device with stream: We have a
match with description: '%s', origin: '%s', cached already with device id %u,
so set stream id to %i",
- description,
- origin,
- gvc_mixer_ui_device_get_id (device),
- stream_id);
-
- g_object_set (G_OBJECT (device),
- "stream-id", stream_id,
- NULL);
- in_possession = TRUE;
+ if (stream_card_id == card_id) {
+ if (g_strcmp0 (device_port_name,
stream_port->port) == 0) {
+ g_debug ("Match device with stream: We
have a match with description: '%s', origin: '%s', cached already with device
id %u, so set stream id to %i",
+ description,
+ origin,
+ gvc_mixer_ui_device_get_id
(device),
+ stream_id);
+
+ g_object_set (G_OBJECT (device),
+ "stream-id", stream_id,
+ NULL);
+ in_possession = TRUE;
+ } else {
+ /* Other UI devices on that card that
match the stream can't be valid */
+ if (device_stream_id == stream_id) {
+ g_object_set (G_OBJECT
(device),
+ "stream-id",
GVC_MIXER_UI_DEVICE_INVALID,
+ NULL);
+ g_debug ("Found another UI
device for this stream: %d, resetting", stream_id);
+ }
+ }
}
}
-
- g_free (device_port_name);
- g_free (origin);
- g_free (description);
-
- if (in_possession == TRUE)
- break;
}
- g_list_free (devices);
return in_possession;
}
+
+/* Clear stream-id from all UI devices that reference the stream that lost all
ports */
+static void
+clear_stream_from_devices (GvcMixerControl *control,
+ GvcMixerStream *stream)
+{
+ g_autoptr (GList) devices = NULL;
+ guint stream_id;
+
+ stream_id = gvc_mixer_stream_get_id (stream);
+ devices = g_hash_table_get_values (GVC_IS_MIXER_SOURCE (stream) ?
control->priv->ui_inputs : control->priv->ui_outputs);
+
+ for (GList *d = devices; d != NULL; d = d->next) {
+ guint device_stream_id;
+ GvcMixerUIDevice *device = d->data;
+
+ g_object_get (G_OBJECT (device),
+ "stream-id", &device_stream_id,
+ NULL);
+
+ if (device_stream_id == stream_id) {
+ g_object_set (G_OBJECT (device),
+ "stream-id", GVC_MIXER_UI_DEVICE_INVALID,
+ NULL);
+ g_debug ("Clearing stream-id: %d", stream_id);
+ }
+ }
+}
+
/*
* This method attempts to match a sink or source with its relevant UI device.
* GvcMixerStream can represent both a sink or source.
@@ -1327,7 +1336,8 @@
*/
static void
sync_devices (GvcMixerControl *control,
- GvcMixerStream* stream)
+ GvcMixerStream* stream,
+ gboolean is_bluetooth)
{
/* Go through ports to see what outputs can be created. */
const GList *stream_ports;
@@ -1336,6 +1346,11 @@
stream_ports = gvc_mixer_stream_get_ports (stream);
+ if (stream_ports == NULL && is_bluetooth) {
+ clear_stream_from_devices (control, stream);
+ return;
+ }
+
if (stream_ports == NULL) {
GvcMixerUIDevice *device;
GObject *object;
@@ -1447,6 +1462,20 @@
}
}
+static gboolean
+is_bluetooth (const pa_proplist *proplist)
+{
+ const char *bus = pa_proplist_gets (proplist, "device.bus");
+
+ if (!bus)
+ return FALSE;
+
+ if (g_str_equal ("bluetooth", bus))
+ return TRUE;
+
+ return FALSE;
+}
+
/*
* Called when anything changes with a sink.
*/
@@ -1459,6 +1488,8 @@
pa_volume_t max_volume;
GvcChannelMap *map;
char map_buff[PA_CHANNEL_MAP_SNPRINT_MAX];
+ GList *list = NULL;
+ gboolean is_bt;
pa_channel_map_snprint (map_buff, PA_CHANNEL_MAP_SNPRINT_MAX,
&info->channel_map);
#if 1
@@ -1475,27 +1506,10 @@
GUINT_TO_POINTER (info->index));
if (stream == NULL) {
- GList *list = NULL;
- guint i;
-
map = gvc_channel_map_new_from_pa_channel_map
(&info->channel_map);
stream = gvc_mixer_sink_new (control->priv->pa_context,
info->index,
map);
-
- for (i = 0; i < info->n_ports; i++) {
- GvcMixerStreamPort *port;
-
- port = g_slice_new0 (GvcMixerStreamPort);
- port->port = g_strdup (info->ports[i]->name);
- port->human_port = g_strdup
(info->ports[i]->description);
- port->priority = info->ports[i]->priority;
- port->available = info->ports[i]->available !=
PA_PORT_AVAILABLE_NO;
-
- list = g_list_prepend (list, port);
- }
- gvc_mixer_stream_set_ports (stream, list);
-
g_object_unref (map);
is_new = TRUE;
@@ -1505,6 +1519,19 @@
return;
}
+ for (guint i = 0; i < info->n_ports; i++) {
+ GvcMixerStreamPort *port;
+
+ port = g_slice_new0 (GvcMixerStreamPort);
+ port->port = g_strdup (info->ports[i]->name);
+ port->human_port = g_strdup (info->ports[i]->description);
+ port->priority = info->ports[i]->priority;
+ port->available = info->ports[i]->available !=
PA_PORT_AVAILABLE_NO;
+
+ list = g_list_prepend (list, port);
+ }
+ gvc_mixer_stream_set_ports (stream, g_steal_pointer (&list));
+
max_volume = pa_cvolume_max (&info->volume);
gvc_mixer_stream_set_name (stream, info->name);
gvc_mixer_stream_set_card_index (stream, info->card);
@@ -1518,6 +1545,11 @@
gvc_mixer_stream_set_base_volume (stream, (guint32) info->base_volume);
gvc_mixer_stream_set_state (stream, translate_pa_state (info->state));
+ is_bt = is_bluetooth (info->proplist);
+ /* Sync devices as the port on the stream might have changed */
+ if (!is_new)
+ sync_devices (control, stream, is_bt);
+
/* Messy I know but to set the port everytime regardless of whether it
has changed will cost us a
* port change notify signal which causes the frontend to resync.
* Only update the UI when something has changed. */
@@ -1530,7 +1562,8 @@
if (active_port == NULL ||
g_strcmp0 (active_port->port,
info->active_port->name) != 0) {
g_debug ("update sink - apparently a port
update");
- gvc_mixer_stream_set_port (stream,
info->active_port->name);
+ if (!gvc_mixer_stream_set_port (stream,
info->active_port->name))
+ g_warning ("Port update to of %d '%s'
failed", info->index, info->active_port->name);
}
}
}
@@ -1542,9 +1575,9 @@
GUINT_TO_POINTER (info->index),
g_object_ref (stream));
add_stream (control, stream);
- /* Always sink on a new stream to able to assign the right
stream id
+ /* Always sync on a new stream to able to assign the right
stream id
* to the appropriate outputs (multiple potential outputs per
stream). */
- sync_devices (control, stream);
+ sync_devices (control, stream, is_bt);
} else {
g_signal_emit (G_OBJECT (control),
signals[STREAM_CHANGED],
@@ -1584,6 +1617,7 @@
gvc_channel_map_volume_changed (map, &info->volume, FALSE);
}
+
static void
update_source (GvcMixerControl *control,
const pa_source_info *info)
@@ -1591,6 +1625,8 @@
GvcMixerStream *stream;
gboolean is_new;
pa_volume_t max_volume;
+ GList *list = NULL;
+ gboolean is_bt;
#if 1
g_debug ("Updating source: index=%u name='%s' description='%s'",
@@ -1609,26 +1645,12 @@
stream = g_hash_table_lookup (control->priv->sources,
GUINT_TO_POINTER (info->index));
if (stream == NULL) {
- GList *list = NULL;
- guint i;
GvcChannelMap *map;
map = gvc_channel_map_new_from_pa_channel_map
(&info->channel_map);
stream = gvc_mixer_source_new (control->priv->pa_context,
info->index,
map);
-
- for (i = 0; i < info->n_ports; i++) {
- GvcMixerStreamPort *port;
-
- port = g_slice_new0 (GvcMixerStreamPort);
- port->port = g_strdup (info->ports[i]->name);
- port->human_port = g_strdup
(info->ports[i]->description);
- port->priority = info->ports[i]->priority;
- list = g_list_prepend (list, port);
- }
- gvc_mixer_stream_set_ports (stream, list);
-
g_object_unref (map);
is_new = TRUE;
} else if (gvc_mixer_stream_is_running (stream)) {
@@ -1637,6 +1659,17 @@
return;
}
+ for (guint i = 0; i < info->n_ports; i++) {
+ GvcMixerStreamPort *port;
+
+ port = g_slice_new0 (GvcMixerStreamPort);
+ port->port = g_strdup (info->ports[i]->name);
+ port->human_port = g_strdup (info->ports[i]->description);
+ port->priority = info->ports[i]->priority;
+ list = g_list_prepend (list, port);
+ }
+ gvc_mixer_stream_set_ports (stream, list);
+
max_volume = pa_cvolume_max (&info->volume);
gvc_mixer_stream_set_name (stream, info->name);
@@ -1650,6 +1683,11 @@
gvc_mixer_stream_set_base_volume (stream, (guint32) info->base_volume);
g_debug ("update source");
+ is_bt = is_bluetooth (info->proplist);
+ /* Sync devices as the port on the stream might have changed */
+ if (!is_new)
+ sync_devices (control, stream, is_bt);
+
if (info->active_port != NULL) {
if (is_new)
gvc_mixer_stream_set_port (stream,
info->active_port->name);
@@ -1659,7 +1697,8 @@
if (active_port == NULL ||
g_strcmp0 (active_port->port,
info->active_port->name) != 0) {
g_debug ("update source - apparently a port
update");
- gvc_mixer_stream_set_port (stream,
info->active_port->name);
+ if (!gvc_mixer_stream_set_port (stream,
info->active_port->name))
+ g_warning ("Port update to of %d '%s'
failed", info->index, info->active_port->name);
}
}
}
@@ -1669,7 +1708,7 @@
GUINT_TO_POINTER (info->index),
g_object_ref (stream));
add_stream (control, stream);
- sync_devices (control, stream);
+ sync_devices (control, stream, is_bt);
} else {
g_signal_emit (G_OBJECT (control),
signals[STREAM_CHANGED],
@@ -1919,9 +1958,9 @@
supported_profiles = g_list_append
(supported_profiles, prof);
}
}
- g_debug ("%i profiles supported on port %s",
+ g_debug ("%i profiles supported on port %s / %s",
g_list_length (supported_profiles),
- port->description);
+ port->description, port->name);
return g_list_sort (supported_profiles, (GCompareFunc)
gvc_mixer_card_profile_compare);
}
@@ -1971,7 +2010,7 @@
gvc_mixer_ui_device_get_id (uidevice));
}
- g_debug ("update_ui_device_on_port_added, direction %u, description
'%s', origin '%s', port available %i",
+ g_debug ("update_ui_device_on_port_added, direction %u, description
'%s', origin '%s', port available %i",
direction,
port->human_port,
gvc_mixer_card_get_name (card),
diff --unified --recursive --text --new-file --color
gnome-control-center-49.3/subprojects/gvc/gvc-mixer-stream.c
gnome-control-center-49.3.new/subprojects/gvc/gvc-mixer-stream.c
--- gnome-control-center-49.3/subprojects/gvc/gvc-mixer-stream.c
2026-01-05 21:33:53.000000000 +0800
+++ gnome-control-center-49.3.new/subprojects/gvc/gvc-mixer-stream.c
2026-01-22 16:39:20.340759230 +0800
@@ -532,8 +532,6 @@
}
}
- g_assert_not_reached ();
-
return NULL;
}
@@ -626,9 +624,13 @@
GList *ports)
{
g_return_val_if_fail (GVC_IS_MIXER_STREAM (stream), FALSE);
- g_return_val_if_fail (stream->priv->ports == NULL, FALSE);
- stream->priv->ports = g_list_sort (ports, (GCompareFunc) sort_ports);
+ if (stream->priv->ports) {
+ g_list_free_full (stream->priv->ports, (GDestroyNotify)
free_port);
+ stream->priv->ports = NULL;
+ }
+ if (ports)
+ stream->priv->ports = g_list_sort (ports, (GCompareFunc)
sort_ports);
return TRUE;
}
