Maarten: I updated the patch as attached to address your comments. This has "#ifdef __sun" used more properly and a comment. Also note that I added a comment to the bug to answer your question that LOG_INFO and LOG_WARNING are defined in the system syslog.h header, not in PulseAudio. Brian On 10/16/11 08:01 AM, Maarten Bosmans wrote: > 2011/10/15 Brian Cameron<brian.cameron at oracle.com>: >> 4) Some straightforward compile issues that seem appropriate to fix for >> building on Solaris is in the patch in this bug: >> >> https://bugs.freedesktop.org/show_bug.cgi?id=41538 > > This one looks good, see also my comment attached to the patch. > > Maarten > _______________________________________________ > pulseaudio-discuss mailing list > pulseaudio-discuss at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: pulse-fix.diff URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20111019/16736e5f/attachment.ksh>