'Twas brillig, and Lennart Poettering at 17/10/11 02:48 did gyre and gimble: >> 3) On Solaris, the PulseAudio GConf module does not compile since the >> "module_info" structure defined in the PulseAudio code conflicts >> with a structure with the same name in /usr/include/sys/stream.h. >> Could the PulseAudio code be updated to use a more unique PulseAudio >> specific name as suggested in the patch in this bug: >> >> https://bugs.freedesktop.org/show_bug.cgi?id=41823 >> >> Note that sys/stream.h is included by sys/types.h on Solaris, and >> sys/types.h is included in the src/modules/gconf/modules-gconf.c >> PulseAudio file. > > The GConf code probably should go away anyway, and be replaced by dconf/gsettings. While dconf/gsettings is an option, I'd be more in favour personally of using whatever pa_database* stuff we use (this was my intension in this area with module-loader). I'm certainly not against dconf or whatever, but if we do migrate module-gocnf to it, then I'd prefer to see dconf as a universal database provider in PA like gdb, tdb, plaintext etc. Having two different mechanisms of writing data to disk doesn't seem like the wises plan overall. Col -- Colin Guthrie gmane(at)colin.guthr.ie http://colin.guthr.ie/ Day Job: Tribalogic Limited http://www.tribalogic.net/ Open Source: Mageia Contributor http://www.mageia.org/ PulseAudio Hacker http://www.pulseaudio.org/ Trac Hacker http://trac.edgewall.org/
