After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 329107 - Profile support for gconfaudiosink
Profile support for gconfaudiosink
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal enhancement
: 0.10.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 329112
 
 
Reported: 2006-01-29 17:43 UTC by Jürg Billeter
Modified: 2006-02-17 11:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add profile property to gconfaudiosink plugin (11.95 KB, patch)
2006-01-29 19:04 UTC, Jürg Billeter
none Details | Review
Patch with fixed GConf schema (12.66 KB, patch)
2006-01-30 13:13 UTC, Jürg Billeter
none Details | Review
Updated patch with fixed GConf cleanup (13.56 KB, patch)
2006-02-16 16:44 UTC, Jürg Billeter
committed Details | Review

Description Jürg Billeter 2006-01-29 17:43:51 UTC
Currently it's not possible to specify multiple default/preferred sound devices in GConf, so all applications without its own sound device chooser share the same global default.

I propose to add profile support to gconfaudiosink, so the application is able to specify whether it wants to play sound events, music/movies, or use the sound device for audio/video conferencing. I've chosen the division into these three categories according to http://mail.gnome.org/archives/utopia-list/2005-July/msg00001.html

Patch will follow soon. This bug is part of my sound configuration patches to allow sane graphical selection of sound cards in GNOME.
Comment 1 Jürg Billeter 2006-01-29 19:04:33 UTC
Created attachment 58342 [details] [review]
Patch to add profile property to gconfaudiosink plugin

First attempt to add the possibility to define multiple preferred sound devices in GConf.
Comment 2 Jürg Billeter 2006-01-30 13:13:14 UTC
Created attachment 58392 [details] [review]
Patch with fixed GConf schema

The schema for the audiosrc_description key was missing from the first patch.
Comment 3 Christian Fredrik Kalager Schaller 2006-02-16 15:50:14 UTC
I tried applying this, but it fails to apply towards current CVS.  Hunk 5 specifically fail.
Comment 4 Jürg Billeter 2006-02-16 16:44:29 UTC
Created attachment 59505 [details] [review]
Updated patch with fixed GConf cleanup

I've attached an updated patch that also fixes GConf cleanup (fixes crashes in patch of bug 329112)
Comment 5 Wim Taymans 2006-02-17 10:45:02 UTC
this breaks ABI/API, so must be scheduled for 0.11, or am I missing something?
Comment 6 Wim Taymans 2006-02-17 10:53:39 UTC
ok, this is in the plugin itself. in 0.8 this was in a public library. looks fine to me then.
Comment 7 Christian Fredrik Kalager Schaller 2006-02-17 10:57:28 UTC
ok, this patch commited to CVS. Closing bug. Thanks Jurg.