[pulseaudio-discuss] Description for virtual sinks

2009-03-24 Thread Maarten Bosmans
Is there any particular reason that it is not possible to set the description when making a sink with module-remap-sink (or his friends module-combine, module-tunnel-sink, etc.)? It seems that the description is always derived from the master sink, but I would like to customize it, because this is

Re: [pulseaudio-discuss] Description for virtual sinks

2009-03-26 Thread Maarten Bosmans
It's good to think about a generic way to set all these properties, on all sources, sinks, etc. This way we avoid all the separate patches, like the ones for module-remap-sink and airport. On Thu, Mar 26, 2009 at 3:40 PM, Lennart Poettering lenn...@poettering.net wrote: The only non-trivial

[pulseaudio-discuss] module-combine master parameter

2009-03-26 Thread Maarten Bosmans
It looks like the parameter master for module-combine doesn't work, it gives an error upon loading the module. Only listing all the sinks under slaves works though. I wanted to update the documentation on http://pulseaudio.org/wiki/Modules#module-combine, but am not sure about the details. Is it

Re: [pulseaudio-discuss] Description for virtual sinks

2009-03-26 Thread Maarten Bosmans
On Thu, Mar 26, 2009 at 7:38 PM, Lennart Poettering lenn...@poettering.net wrote: On Thu, 26.03.09 19:16, Maarten Bosmans (mkbosm...@gmail.com) wrote: Another approach is separate set-*-property commands. I would really like to be able to fire up pacmd and just say:   set-sink-input-property

Re: [pulseaudio-discuss] module-combine master parameter

2009-03-28 Thread Maarten Bosmans
On Fri, Mar 27, 2009 at 12:16 AM, Lennart Poettering lenn...@poettering.net wrote: On Thu, 26.03.09 20:43, Maarten Bosmans (mkbosm...@gmail.com) wrote: It looks like the parameter master for module-combine doesn't work, it gives an error upon loading the module. Only listing all the sinks

[pulseaudio-discuss] [PATCH] fix some typos in doxygen comments

2009-03-29 Thread Maarten Bosmans
these bugged me for a while now. Maarten --- src/pulse/introspect.h | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/pulse/introspect.h b/src/pulse/introspect.h index ec8a2c0..d7ef1c0 100644 --- a/src/pulse/introspect.h +++ b/src/pulse/introspect.h @@

Re: [pulseaudio-discuss] Description for virtual sinks

2009-03-30 Thread Maarten Bosmans
On Fri, Mar 27, 2009 at 1:07 AM, Lennart Poettering lenn...@poettering.net wrote: On Thu, 26.03.09 22:34, Maarten Bosmans (mkbosm...@gmail.com) wrote: Are you willing to except patches to extent the pulse CLI with this functionality? I was thinking of adding the following commands: set-{sink

Re: [pulseaudio-discuss] use 7.1 audio chip as 2x stereo?

2009-03-31 Thread Maarten Bosmans
You have to manually edit the default startup script for Pulseaudio. In Ubuntu this is the file /etc/pulse/default/pa. Use module-remap-sink (http://pulseaudio.org/wiki/Modules#module-remap-sink) like this: load-module module-remap-sink master=sinkname channels=2 channel_map=left,right

Re: [pulseaudio-discuss] Microphone input filter

2009-06-03 Thread Maarten Bosmans
On Tue, Jun 2, 2009 at 1:35 PM, Toby Collett tcollett+li...@plan9.net.nz wrote: I am trying to use the internal microphone in my laptop to make VOIP calls (actually skype mostly). The quality of the sound is not good at all with large amounts of background noise when I have the levels up high

Re: [pulseaudio-discuss] Sending audio from source to multiple sinks dynamically

2009-06-17 Thread Maarten Bosmans
On Tue, Jun 16, 2009 at 2:08 AM, Timothy J Massey tmas...@obscorp.com wrote: From what I can tell, with PA you can pick only which (single) sink a source is connected to.  If you pick a new sink for the source, it is removed from whatever sink it was using before.  And AFAICT, if you want to

Re: [pulseaudio-discuss] Automatic selection of an audio sink depending on value of ENV variable

2009-10-19 Thread Maarten Bosmans
2009/10/19 Leszek Koltunski les...@koltunski.pl: Now, I would make me one happy duck if I could somehow tell Pulse to automatically route audio to hw:1.0 whenever I am working on the TV, that is, whenever DISPLAY is 1.0. It looks like PULSE_SINK environment variable is exactly what you need,

Re: [pulseaudio-discuss] Moving speakers around using AC3

2009-11-28 Thread Maarten Bosmans
Yes, in this way you don't have to switch profiles at all, you just end up with to sinks, one for the soundcard 5.1 output and a virtual stereo sink from module-remap. Pulseaudio remembers when you move streams from one sink to another (using pavucontrol, for example), so the movie player would

Re: [pulseaudio-discuss] Stream volumes as the universal volume adjustment method

2010-01-08 Thread Maarten Bosmans
2010/1/8 Tanu Kaskinen ta...@iki.fi: pe, 2010-01-08 kello 01:02 +0100, Mads Kiilerich kirjoitti: So if I adjust the volume with one application playing and then start another application then the other application will use its default volume and the relative volume will thus have changed?

Re: [pulseaudio-discuss] system-wide daemon

2010-02-10 Thread Maarten Bosmans
2010/2/9 olin.pulse@shivers.mail0.org: Maybe I'm wrong. I can't figure out *what* the model is, really. When I click on padevchooser's Configure Local Sound Server entry, I get a window whose Network Server tab lets me enable network access to local sound devices. Furthermore, I can set

Re: [pulseaudio-discuss] system-wide daemon

2010-02-10 Thread Maarten Bosmans
Bah, wrong thread 2010/2/10 Maarten Bosmans mkbosm...@gmail.com: 2010/2/9  olin.pulse@shivers.mail0.org: Maybe I'm wrong. I can't figure out *what* the model is, really. When I click on padevchooser's Configure Local Sound Server entry, I get a window whose Network Server tab lets me

Re: [pulseaudio-discuss] system-wide daemon

2010-02-11 Thread Maarten Bosmans
2010/2/11 Lennart Poettering lenn...@poettering.net: On Wed, 10.02.10 10:45, Maarten Bosmans (mkbosm...@gmail.com) wrote: The other mode is the system-wide daemon mode. This follows more the traditional unix model of a dedicated pulse user running a daemon to which other users can connect

Re: [pulseaudio-discuss] My default sound devices are not retained after suspend or reboot

2010-02-12 Thread Maarten Bosmans
2010/2/11 Sebastien PIMENTA p...@pulsradio.com: Hi, Can you tell me where is the log file of the debug mode ? I've edited /etc/pulse/daemon.conf and uncommented log-level=debug but I can't find the log file :S Lennart : concerning the card indexes, I'm not sure how to use the control

Re: [pulseaudio-discuss] My default sound devices are not retained after suspend or reboot

2010-02-16 Thread Maarten Bosmans
2010/2/12 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Sebastien Pimenta at 12/02/10 09:41 did gyre and gimble: Hi thanks for the tip, I'll give it a try but I'm using shoutcast, don't know if it will work too. Moreover my main problem is that the default pulseaudio card changes

Re: [pulseaudio-discuss] How to `wire' output of a program as if it is microphone input

2010-05-21 Thread Maarten Bosmans
of the sink. I have been staring at the documentation for two days but the Eurika moments still eludes me. Some ideas or pointers? Kind regards: al_shopov Maarten Bosmans pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio

[pulseaudio-discuss] Select channel for module-rtp-send

2010-05-31 Thread Maarten Bosmans
Hi, I have a USB soundcard as a stereo source in PulseAudio and want to send the audio as a RTP stream over the network. The two channels are unrelated (well not completely, one is the audio from the front-of-house mixing console and the other the combined signal from two room mics) so I would

Re: [pulseaudio-discuss] recommended PA config for an embedded box

2010-06-04 Thread Maarten Bosmans
2010/6/3 Manuel Klemenz m.klem...@gmx.at: Hi guys... I'm running a small Linux box/server in my home network where also my speaker are attached. this box is running 24/7 (at least almost) and only accessible over network - so no mouse, keyboard or video output is attached thereto. I've got

Re: [pulseaudio-discuss] Select channel for module-rtp-send

2010-06-18 Thread Maarten Bosmans
ping 2010/5/25 Maarten Bosmans mkbosm...@gmail.com: Hi, I have a USB soundcard as a stereo source in PulseAudio and want to send the audio as a RTP stream over the network. The two channels are unrelated (well not completely, one is the audio from the front-of-house mixing console

[pulseaudio-discuss] building PulseAudio from git master

2010-07-15 Thread Maarten Bosmans
Hi, I'm trying to build PulseAudio from the git master branch. After running autogen.sh, the configure script outputs the version as: pulseaudio 0.9.19-519-gf26c8. Shouldn't this be 0.9.21-*? Maarten ___ pulseaudio-discuss mailing list

Re: [pulseaudio-discuss] Assert error in pulsecore/resampler.c:trivial_resample

2010-07-22 Thread Maarten Bosmans
2010/7/20 Tanu Kaskinen ta...@iki.fi: On Fri, 2010-07-16 at 23:22 +0200, Maarten Bosmans wrote: Hi, PulseAudio from git master crashes when I play a sound to a combined sink of two 96000 Hz sinks. It does not happen if the slave sinks are set to 48kHz. Playing directly two one of the slaves

Re: [pulseaudio-discuss] silence on *.monitor

2010-12-20 Thread Maarten Bosmans
2010/12/20 Alexey Fisher bug-tr...@fisher-privat.net: Hallo all, i have a problem to capture anything on any *.monitor device. Steps to reproduce. Generate sound: gst-launch audiotestsrc ! audioconvert ! pulsesink capture sound: gst-launch pulsesrc

Re: [pulseaudio-discuss] [PATCH 3/3] Fighting rewinds: Make sure there is some headroom after an underrun

2010-12-20 Thread Maarten Bosmans
2010/12/9 David Henningsson david.hennings...@canonical.com: If the amount of data in the implementor buffer is very tiny, i e even less than what we will likely be asked for, don't ask for a rewind as that would lead to another underrun. In an audio context, the term headroom often refers to

Re: [pulseaudio-discuss] cannot control input devices from pavucontrol + crashes in recording tab

2010-12-22 Thread Maarten Bosmans
2010/12/22 Peter Hercek pher...@gmail.com: Well that says it alll. When I try to control muting or levels of input deviced in pavucontrol then it does not work. I can do this with 'alsamixer -c 0'. I use archlinux and I set pulseaudio based on this page:

Re: [pulseaudio-discuss] silence on *.monitor

2010-12-22 Thread Maarten Bosmans
2010/12/22 Alexey Fisher bug-tr...@fisher-privat.net: Am Mittwoch, den 22.12.2010, 13:06 +0100 schrieb Maarten Bosmans: 2010/12/20 Alexey Fisher bug-tr...@fisher-privat.net: Am Montag, den 20.12.2010, 14:10 +0100 schrieb Maarten Bosmans: 2010/12/20 Alexey Fisher bug-tr...@fisher-privat.net

[pulseaudio-discuss] [PATCH 00/10] Enable building for win32

2011-01-05 Thread Maarten Bosmans
operation on Linux. Maarten Maarten Bosmans (10): Clean up poll.h includes Use pulsecore/socket.h instead of sys/socket.h Adapt win32 specific code to current API Use pa_* instead of generic functions to improve portability Apply #ifdefs around functionality not available on win32 Use PCRE

[pulseaudio-discuss] [PATCH 02/10] Use pulsecore/socket.h instead of sys/socket.h

2011-01-05 Thread Maarten Bosmans
The check whether POSIX socket.h or WIN32 winsock2.h must be included can be made centrally. The downside is that some functionality of e.g. arpa/inet.h is also implemented in winsock.h, so that some files that don't use socket functions, but do use inet.h functions, must also include

[pulseaudio-discuss] [PATCH 03/10] Adapt win32 specific code to current API

2011-01-05 Thread Maarten Bosmans
--- src/daemon/main.c|4 ++-- src/pulsecore/mutex-win32.c | 25 - src/pulsecore/thread-win32.c |5 +++-- 3 files changed, 29 insertions(+), 5 deletions(-) diff --git a/src/daemon/main.c b/src/daemon/main.c index de809b1..cab275f 100644 ---

[pulseaudio-discuss] [PATCH 04/10] Use pa_* instead of generic functions to improve portability

2011-01-05 Thread Maarten Bosmans
--- src/pulse/context.c |2 +- src/pulse/thread-mainloop.c |2 +- src/pulsecore/inet_ntop.c |3 ++- src/pulsecore/inet_pton.c |4 +++- src/tests/asyncq-test.c |2 +- src/tests/rtstutter.c |3 ++- 6 files changed, 10 insertions(+), 6 deletions(-) diff

[pulseaudio-discuss] [PATCH 05/10] Apply #ifdefs around functionality not available on win32

2011-01-05 Thread Maarten Bosmans
--- configure.ac|3 +- src/daemon/daemon-conf.c|5 src/daemon/dumpmodules.c|2 + src/daemon/main.c |2 + src/modules/module-augment-properties.c |2 + src/modules/module-cli.c

[pulseaudio-discuss] [PATCH 09/10] Implement some functions for win32

2011-01-05 Thread Maarten Bosmans
And disable building binaries for win32 that make no sense --- src/Makefile.am | 22 -- src/pulsecore/authkey.c |2 +- src/pulsecore/core-util.c | 26 ++ 3 files changed, 43 insertions(+), 7 deletions(-) diff --git a/src/Makefile.am

Re: [pulseaudio-discuss] [PATCH 09/10] Implement some functions for win32

2011-01-10 Thread Maarten Bosmans
Thanks for the review 2011/1/11 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 06/01/11 01:39 did gyre and gimble: @@ -2565,7 +2569,11 @@ void pa_unset_env_recorded(void) {          if (!s)              break; +#ifdef OS_IS_WIN32 +        putenv(pa_sprintf_malloc

Re: [pulseaudio-discuss] [PATCH 08/10] Add AM_LDFLAGS more consistently to all commands

2011-01-10 Thread Maarten Bosmans
2011/1/11 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 06/01/11 01:39 did gyre and gimble: @@ -100,12 +100,12 @@ BINLDFLAGS = -static  endif  if OS_IS_WIN32 -AM_LDFLAGS+=-Wl,--export-all-symbols +AM_LDFLAGS+=-Wl,--export-all-symbols,--enable-auto-import

Re: [pulseaudio-discuss] [PATCH 09/10] Implement some functions for win32

2011-01-11 Thread Maarten Bosmans
2011/1/11 Colin Guthrie gm...@colin.guthr.ie: void pa_set_env(const char *key, const char *value) {     pa_assert(key);     pa_assert(value);     /* This is not thread-safe */     putenv(pa_sprintf_malloc(%s=%s, key, value)); } Hmm, that looks wrong too on first glance, but with a little

Re: [pulseaudio-discuss] Rethinking rate adjustments [was: Re: [PATCH] module-combine: limit the size of the rate adjustments]

2011-01-11 Thread Maarten Bosmans
2011/1/11 pl bossart bossart.nos...@gmail.com: The third shows the new algorithm. Doesn't it look pretty? The event halfway (after two minutes of playing) is change of song. Apparantly this introduces some extra samples in the RTP stream, but the algorithms deals nicely with that. The new

Re: [pulseaudio-discuss] Rethinking rate adjustments [was: Re: [PATCH] module-combine: limit the size of the rate adjustments]

2011-01-11 Thread Maarten Bosmans
2011/1/11 pl bossart bossart.nos...@gmail.com: Any idea what the sample rate drops linearly in the initial part? Maybe we ought to prevent SRC adjustments until there's enough history to make such changes? -Pierre Thats because initially the buffer with audio from the RTP stream only

Re: [pulseaudio-discuss] State of various rate adjustment patches

2011-01-15 Thread Maarten Bosmans
to test module-loopback? null-sink and alsa have very stable latencies, so its no good test for module-loopback. Maarten 2011/1/15 Maarten Bosmans mkbosm...@gmail.com: 2011/1/15 Colin Guthrie gm...@colin.guthr.ie: Hi, I know people (i.e. Maarten) are discussing and proposing various rate

Re: [pulseaudio-discuss] State of various rate adjustment patches

2011-01-17 Thread Maarten Bosmans
2011/1/17 pl bossart bossart.nos...@gmail.com: I've tested module-loopback by playing to a null-sink and looping its monitor to the real alsa sink. This showed good behaviour, but may be the algorithm I used for module-rtp-recv should also be used here. Does anyone has a better suggestion for

Re: [pulseaudio-discuss] module-alsa-source latency is 1199291857647 ms

2011-01-18 Thread Maarten Bosmans
2011/1/18 pl bossart bossart.nos...@gmail.com: My questions: 1)  The reported latency of 38 years must reflect a bug!(?)  Can this be fixed? This one made my day. Something's obviously broken here. can you provide the full log for us to figure out what went wrong? with only some excerpts

Re: [pulseaudio-discuss] module-alsa-source latency is 1199291857647 ms

2011-01-18 Thread Maarten Bosmans
2011/1/18 Dr. Adrian Wrigley m...@adrianwrigley.com: Hi all, I'm now having problems with latency. I'm trying to send the analog input on Machine2 to the speakers on Machine1.  I'm currenty trying with esd protocol and module-esound-sink on Macine2. (Unfortunately, native tcp protocol

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-20 Thread Maarten Bosmans
2011/1/20 Brian J. Murrell br...@interlinx.bc.ca: I have a laptop (Ubuntu 10.10) who's sound I would like to go to my workstation (also Ubuntu 10.10) so on the laptop in paprefs I have selected Make discoverable PulseAudio network sound devices available locally enabled. This alone should

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-20 Thread Maarten Bosmans
2011/1/20 Brian J. Murrell br...@interlinx.bc.ca: Maarten Bosmans mkbosmans at gmail.com writes: I assume you also loaded module-zeroconf-publish on the other machine. No, in fact it appears to be commented out in the /etc/pulse/default.pa of the workstation (where I hear the audio from

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-20 Thread Maarten Bosmans
contributing to the bandwidth. Do you still see excessive bandwidth usage with pavucontrol (and possibly other programs) closed? Maarten 2011/1/20 Brian J. Murrell br...@interlinx.bc.ca: Maarten Bosmans mkbosmans at gmail.com writes: It looks like both are from your PC. You probably SSH'ed into your

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-22 Thread Maarten Bosmans
2011/1/22 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Tanu Kaskinen at 21/01/11 19:34 did gyre and gimble: I had a look at some point at the peak detection resampler... I think the peak detection flag that you mentioned earlier doesn't do anything else than force the resampler of

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-22 Thread Maarten Bosmans
2011/1/22 Maarten Bosmans mkbosm...@gmail.com: 2011/1/22 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Tanu Kaskinen at 21/01/11 19:34 did gyre and gimble: I had a look at some point at the peak detection resampler... I think the peak detection flag that you mentioned earlier doesn't

Re: [pulseaudio-discuss] Build system: Makefile.am patches

2011-01-25 Thread Maarten Bosmans
2011/1/23 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Colin Guthrie at 20/01/11 18:10 did gyre and gimble: Yeah the first patch was as I expected. It broke make distcheck. It also generates all these warnings which are a pain...  CCLD   libpulsecore-1.0.la  CC     cli.lo cc1:

Re: [pulseaudio-discuss] Make discoverable PA network sound devices available locally results in huge network traffic

2011-01-26 Thread Maarten Bosmans
2011/1/22 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 22/01/11 10:11 did gyre and gimble: It turns out that (without pavucontrol loaded) starting an audio stream on the client on the tunnel sink, makes the audio stream over the network and stopping the program

Re: [pulseaudio-discuss] State of various rate adjustment patches

2011-02-09 Thread Maarten Bosmans
2011/1/31 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 31/01/11 10:36 did gyre and gimble: 2011/1/16 Maarten Bosmans mkbosm...@gmail.com: The branch is up at https://github.com/mkbosmans/pulseaudio/compare/master...rate-adjustment ready for merging, as far a I am

Re: [pulseaudio-discuss] PA Roadmap Planning and LAC Conference

2011-02-17 Thread Maarten Bosmans
2011/2/17 Colin Guthrie gm...@colin.guthr.ie: There have been a few discussions recently about getting a solid roadmap organised for the next medium term of PA development. While the stable-queue stuff has been bubbling along nicely enough, there are several new features in the 1.x master

Re: [pulseaudio-discuss] [PATCH 1/3] configure.ac: Typo. s/Console Kit/ConsoleKit/ [1]

2011-02-19 Thread Maarten Bosmans
These patches can probably be rolled into one patch, as they all concern spelling in configure.ac. You can alsa correct Jack-JACK and Alsa-ALSA. Maarten 2011/2/18 Paul Menzel paulepan...@users.sourceforge.net: Date: Fri, 18 Feb 2011 23:27:20 +0100 [1]

Re: [pulseaudio-discuss] [PATCH] src/Makefile.am: add missing space to fix build using uClibc

2011-02-19 Thread Maarten Bosmans
2011/2/19 Paul Menzel paulepan...@users.sourceforge.net: From: Henning Heinold hein...@inf.fu-berlin.de Date: Fri Jul 10 12:33:30 2009 +0200 Using uClibc        AM_LIBADD = $(PTHREAD_LIBS) $(INTLLIBS) is not empty because `$(INTLLIBS)` is set to `-lintl`. This uncovered a missing space

Re: [pulseaudio-discuss] [PATCH] dbusiface-stream: Send the Device property in the GetAll handler.

2011-02-22 Thread Maarten Bosmans
2011/2/22 Tanu Kaskinen tanu.kaski...@digia.com: ---  src/modules/dbus/iface-stream.c |    1 +  1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/modules/dbus/iface-stream.c b/src/modules/dbus/iface-stream.c index 681790b..cfba5ad 100644 --- a/src/modules/dbus/iface-stream.c

Re: [pulseaudio-discuss] PulseAudio module-rtp-recv

2011-02-23 Thread Maarten Bosmans
Please use the PulseAudio mailing list for these kind of questions. I'll be reading and answering there. 2011/2/23 ericlesoll ericles...@gmail.com: Hi Maarten, Thanks for your job, I reported some bugs on Launchpad about PulseAudio  module-rtp-recv, and I saw that you have commited 6 weeks

Re: [pulseaudio-discuss] A2DP - Bluetooth source to alsa sink audio routing

2011-02-23 Thread Maarten Bosmans
. Thanks for support and I really appreciate what you guys are doing. Cristina I'll do more testing the Maarten On 02/22/2011 09:36 PM, Maarten Bosmans wrote: 2011/2/22 Cristina Cristeadoina.crist...@gmail.com: Thanks Maarten, that really makes the sample rate adjustment

Re: [pulseaudio-discuss] [PATCH] Add a target to the PA log feature and improve PA log core

2011-02-23 Thread Maarten Bosmans
Here's my review of that patch. It will be handled in the meeting tomorrow, but I'm sure I've forgotten by then, so I sent it to the list. 2010/11/22 Becker, VincentX vincentx.bec...@intel.com: Hi all, Could you please review this patch ? This is a patch that adds a generic log target for PA

Re: [pulseaudio-discuss] [PATCH] Add a target to the PA log feature and improve PA log core

2011-02-23 Thread Maarten Bosmans
Here's my review of that patch. It will be handled in the meeting tomorrow, but I'm sure I've forgotten by then, so I sent it to the list. 2010/11/22 Becker, VincentX vincentx.bec...@intel.com: Hi all, Could you please review this patch ? This is a patch that adds a generic log target for PA

Re: [pulseaudio-discuss] [PATCH] Add a target to the PA log feature and improve PA log core

2011-02-23 Thread Maarten Bosmans
2011/2/23 Becker, VincentX vincentx.bec...@intel.com: Thanks for your review Maarten. So you suggest to split this into several patches (2). One for the outer implementation and the other for the inner one. I will try to attend to the irc meeting tomorrow so I can catch your remarks in real

Re: [pulseaudio-discuss] [PATCH] Add a target to the PA log feature and improve PA log core

2011-02-24 Thread Maarten Bosmans
2011/2/24 Becker, VincentX vincentx.bec...@intel.com: From: Maarten Bosmans [mailto:mkbosm...@gmail.com] 2011/2/23 Becker, VincentX vincentx.bec...@intel.com: Thanks for your review Maarten. So you suggest to split this into several patches (2). One for the outer implementation and the other

Re: [pulseaudio-discuss] [PATCH 1/2] Add a target to the PA log feature

2011-02-24 Thread Maarten Bosmans
2011/2/24 Becker, VincentX vincentx.bec...@intel.com: From: Maarten Bosmans [mailto:mkbosm...@gmail.com] 2011/2/23 Becker, VincentX vincentx.bec...@intel.com: Thanks for your review Maarten. So you suggest to split this into several patches (2). One for the outer implementation and the other

[pulseaudio-discuss] Windows binaries

2011-02-25 Thread Maarten Bosmans
As the patches that make it possible to build pulse on win32 should land in master any moment now, I thought it would be a good time to make the binaries available for download. http://bosmans.ch/pulseaudio/pulseaudio-1.0dev-1090.4.zip This is just a repackaging of the files found at

Re: [pulseaudio-discuss] [RFC] Allow read-only or non-existing sink input volume.

2011-02-27 Thread Maarten Bosmans
2011/2/27 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Tanu Kaskinen at 27/02/11 12:35 did gyre and gimble: I actually have started to feel that volume_readable and volume_writable would be better than has_volume and read_only_volume in pa_sink_input_info. What are others' opinions?

Re: [pulseaudio-discuss] MacOS X Support

2011-03-02 Thread Maarten Bosmans
2011/2/27 Stephen Lee sl33...@gmail.com: I git clone'd master yesterday, successfully compiled, and ran PA on my MacOS X machine.  I was pleasantly surprised to see the Coreaudio modules!  I successfully tunnelled audio from one of my Linux machines to my MacOS X w/o using esound, but I

[pulseaudio-discuss] Some patches towards the goal of a warning-free build

2011-03-02 Thread Maarten Bosmans
There is also a branch available: https://github.com/mkbosmans/pulseaudio/compare/warning-free-build The first commit of that branch is already in master, as it solved a nasty bug. [PATCH 1/4] Get rid of some unused-function compiler warnings Nothing to fancy here, I guess. [PATCH 2/4] Various

[pulseaudio-discuss] [PATCH 1/4] Get rid of some unused-function compiler warnings

2011-03-02 Thread Maarten Bosmans
--- src/daemon/daemon-conf.c |6 ++ src/pulsecore/core-util.c |4 src/pulsecore/memtrap.c |2 +- src/pulsecore/shm.c |2 ++ src/utils/pacat.c |2 ++ 5 files changed, 11 insertions(+), 5 deletions(-) diff --git a/src/daemon/daemon-conf.c

[pulseaudio-discuss] [PATCH 2/4] Various fixes for build warnings

2011-03-02 Thread Maarten Bosmans
--- src/modules/udev-util.c |2 +- src/pulse/volume.c |2 +- src/pulsecore/ltdl-helper.c |4 ++-- src/tests/mainloop-test.c |2 +- src/tests/mix-test.c| 10 ++ 5 files changed, 7 insertions(+), 13 deletions(-) diff --git a/src/modules/udev-util.c

[pulseaudio-discuss] [PATCH 3/4] configure: Drop some warnings

2011-03-02 Thread Maarten Bosmans
Drop -Winline and set -Wstrict-aliasing level to its default. --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 21dae30..77959a2 100644 --- a/configure.ac +++ b/configure.ac @@ -102,7 +102,7 @@ if test x$M4 = xno ; then

[pulseaudio-discuss] Some patches towards the goal of a warning-free build

2011-03-02 Thread Maarten Bosmans
There is also a branch available: https://github.com/mkbosmans/pulseaudio/compare/warning-free-build The first commit of that branch is already in master, as it solved a nasty bug. [PATCH 1/4] Get rid of some unused-function compiler warnings Nothing to fancy here, I guess. [PATCH 2/4] Various

Re: [pulseaudio-discuss] [PATCH 2/4] Various fixes for build warnings

2011-03-02 Thread Maarten Bosmans
2011/3/2 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 02/03/11 11:41 did gyre and gimble: diff --git a/src/tests/mainloop-test.c b/src/tests/mainloop-test.c index 3ec6d11..75b77b5 100644 --- a/src/tests/mainloop-test.c +++ b/src/tests/mainloop-test.c @@ -48,7

Re: [pulseaudio-discuss] pulseaudio: module.c: module-detect is deprecated: Please use module-udev-detect instead of module-detect!

2011-03-02 Thread Maarten Bosmans
2011/3/2 Andriy Gapon a...@freebsd.org: on 02/03/2011 17:18 Michal Varga said the following: On Wed, 2011-03-02 at 16:13 +0100, Michal Varga wrote: So instead of fixing things like these in ports, I'd say the issue should be reported/fix submitted to upstream, it's pretty much possible that

[pulseaudio-discuss] unused check_required function in modules/alsa/alsa-mixer.c

2011-03-03 Thread Maarten Bosmans
I intended to catch diwic on IRC, but didn't see him the last couple of days, so in order not to forget, here is my question to the list. In this commit some stuff got shuffled around in alsa-mixer http://git.0pointer.de/?p=pulseaudio.git;a=commitdiff;h=b0f72311 With the result that the

[pulseaudio-discuss] [RFC] allow conditionals in config input files like default.pa.in

2011-03-03 Thread Maarten Bosmans
It would be nice to have the ability to use @if HAVE_FEATURE@ in the configuration file templates. You could see it either as an extension of the current @VARIABLE@ replacement or as a compile-time variant of .ifexists. Incidentally, does anybody know why these files are currently generated in

Re: [pulseaudio-discuss] Problem in running locally compiled pulse utility programs

2011-03-04 Thread Maarten Bosmans
2011/3/4 ssrk ssrk...@gmail.com: Hi, I run Ubuntu 9.10, which by default has pulseaudio and utilities like pactl, pacmd. I downloaded pulseaudio source tarball http://0pointer.de/lennart/projects/pulseaudio/pulseaudio-0.9.22.tar.gz and extracted it, ran configure script, compiled it and

Re: [pulseaudio-discuss] [PATCH 1/2] Log feature:Add a new log target to a file descriptor

2011-03-08 Thread Maarten Bosmans
FWIW, I've tested this patch and it works as advertised. As we already decided in the IRC meeting that the concept of the patch was fine, this is ready to go in git master. (with some minor cosmetic adjustments?) Maarten 2011/3/8 Vincent Becker vincentx.bec...@intel.com: This patch enables

[pulseaudio-discuss] [PATCH 1/7] Fixup #include directives according to Coding Style

2011-03-09 Thread Maarten Bosmans
Use #include header.h if functionality of header.h is implemented and #include header.h if functionality of header.h is used. --- src/pulse/browser.c |2 +- src/pulse/context.c | 13 + src/pulse/ext-device-manager.c |5 ++---

[pulseaudio-discuss] [PATCH 2/7] build: Use silent rules for generating files

2011-03-09 Thread Maarten Bosmans
--- man/Makefile.am |4 ++-- src/Makefile.am | 20 ++-- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/man/Makefile.am b/man/Makefile.am index 7793fe7..eca1fb9 100644 --- a/man/Makefile.am +++ b/man/Makefile.am @@ -36,7 +36,7 @@ noinst_DATA = \

Re: [pulseaudio-discuss] [PATCH 2/7] build: Use silent rules for generating files

2011-03-09 Thread Maarten Bosmans
There are just 2 patches, the /7 was a mistake. Maarten 2011/3/9 Maarten Bosmans mkbosm...@gmail.com: ---  man/Makefile.am |    4 ++--  src/Makefile.am |   20 ++--  2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/man/Makefile.am b/man/Makefile.am index

Re: [pulseaudio-discuss] [PATCH 1/2] Log feature:Add a new log target to a file descriptor

2011-03-09 Thread Maarten Bosmans
2011/3/9 Vincent Becker vincentx.bec...@intel.com: @@ -185,7 +193,23 @@ int pa_daemon_conf_set_log_target(pa_daemon_conf *c, const char *string) {     } else if (!strcmp(string, stderr)) {         c-auto_log_target = 0;         c-log_target = PA_LOG_STDERR; -    } else +    } else if

[pulseaudio-discuss] [PATCH] Make pulse compile with clang

2011-03-10 Thread Maarten Bosmans
As discussed on IRC. This is basically a correctly formatted version of the patch attached to http://pulseaudio.org/ticket/883 with a small improvement in configure.ac added. Maarten 0001-Make-pulse-compile-with-clang.patch Description: Binary data

Re: [pulseaudio-discuss] [RFC] allow conditionals in config input files like default.pa.in

2011-03-12 Thread Maarten Bosmans
conditionally) could also be used for this. On Fri, 2011-03-04 at 00:26 +0100, Maarten Bosmans wrote: It would be nice to have the ability to use @if HAVE_FEATURE@ in the configuration file templates. You could see it either as an extension of the current @VARIABLE@ replacement or as a compile-time

[pulseaudio-discuss] Allow conditionals in config input files

2011-03-14 Thread Maarten Bosmans
This patch series changes the current use of sed by an awk script, with some enhancements on top. It is portable, or at least I've tested it with both gawk and mawk and it passes make distcheck. [PATCH 1/6] build: Replace sed by awk for processing .in files [PATCH 2/6] build: Get rid of

[pulseaudio-discuss] [PATCH 1/6] build: Replace sed by awk for processing .in files

2011-03-14 Thread Maarten Bosmans
--- Makefile.am |1 + configure.ac|3 +-- man/Makefile.am |4 +--- process-in.awk | 36 src/Makefile.am | 30 ++ 5 files changed, 53 insertions(+), 21 deletions(-) create mode 100644 process-in.awk diff

[pulseaudio-discuss] [PATCH 2/6] build: Get rid of default.pa.win32

2011-03-14 Thread Maarten Bosmans
--- src/Makefile.am | 15 --- src/daemon/default.pa.in| 15 +++ src/daemon/default.pa.win32 | 43 --- 3 files changed, 19 insertions(+), 54 deletions(-) delete mode 100644 src/daemon/default.pa.win32 diff

[pulseaudio-discuss] [PATCH 3/6] build: Use more conditional inclusions in default.pa

2011-03-14 Thread Maarten Bosmans
--- src/daemon/default.pa.in | 30 ++ 1 files changed, 26 insertions(+), 4 deletions(-) diff --git a/src/daemon/default.pa.in b/src/daemon/default.pa.in index 8ee3bdc..c0f396e 100755 --- a/src/daemon/default.pa.in +++ b/src/daemon/default.pa.in @@ -46,35 +46,58 @@

[pulseaudio-discuss] [PATCH 4/6] build: Only use relevant portions of daemon.conf.in

2011-03-14 Thread Maarten Bosmans
--- src/daemon/daemon.conf.in |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/daemon/daemon.conf.in b/src/daemon/daemon.conf.in index 9beba85..a21c93c 100644 --- a/src/daemon/daemon.conf.in +++ b/src/daemon/daemon.conf.in @@ -25,7 +25,9 @@ ; allow-exit = yes ;

[pulseaudio-discuss] [PATCH 5/6] Move -Ddefines from AM_CFLAGS to config.h

2011-03-14 Thread Maarten Bosmans
This makes the length of the compiler command line a bit shorter. --- configure.ac| 15 +-- m4/ax_define_dir.m4 | 49 + src/Makefile.am |6 -- 3 files changed, 58 insertions(+), 12 deletions(-) create mode 100644

[pulseaudio-discuss] [PATCH 6/6] Make pulse compile with clang

2011-03-14 Thread Maarten Bosmans
This fixes the checking of supported compiler flags and the following error message for svolume_mmx: pulsecore/svolume_mmx.c:157:76: error: invalid use of a cast in a inline asm context requiring an l-value: remove the cast or build with -fheinous-gnu-extensions : +r (samples), +r

[pulseaudio-discuss] [PATCH] Fix up some double spaces

2011-03-14 Thread Maarten Bosmans
This is another whitespace fixup. I promise that it's the last coding style fix I'll send to the list for now ;-) Actually it is not all just coding style, as what got my attention in the first place was an extra space in the log output (the hunk at pulsecore/module.c) As with the previous patch,

[pulseaudio-discuss] New win32 patch series

2011-03-16 Thread Maarten Bosmans
The updated branch is online at https://github.com/mkbosmans/pulseaudio/compare/mingw32-build [PATCH 1/9] module-waveout: Add device_name parameter [PATCH 2/9] Find modules and config files relative to the installed libraries. [PATCH 3/9] build: copy instead of link pacat to other utils on win32

[pulseaudio-discuss] [PATCH 1/9] module-waveout: Add device_name parameter

2011-03-16 Thread Maarten Bosmans
Also use the name in the source/sink description. Based on a patch by srirams, from github. --- src/modules/module-waveout.c | 32 ++-- 1 files changed, 30 insertions(+), 2 deletions(-) diff --git a/src/modules/module-waveout.c b/src/modules/module-waveout.c index

[pulseaudio-discuss] [PATCH 3/9] build: copy instead of link pacat to other utils on win32

2011-03-16 Thread Maarten Bosmans
This is necessary as symlinks are not supported on Windows. Also use the $(EXEEXT) variable. As a side effect on POSIX, an absolute symbolic link is now created instead of a relative one. --- src/Makefile.am | 21 + 1 files changed, 13 insertions(+), 8 deletions(-) diff

[pulseaudio-discuss] [PATCH 4/9] build: Protect some more variables by ifdefs

2011-03-16 Thread Maarten Bosmans
This avoids empty directories being created on builds without X11, ALSA, etc. --- src/Makefile.am | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 3736669..e0c5d0d 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -27,10

[pulseaudio-discuss] [PATCH 6/9] Fix pa_rtclock_from_wallclock

2011-03-16 Thread Maarten Bosmans
The HAVE_CLOCK_GETTIME macro protects timespec and related functions, nothing of which is used in pa_rtclock_from_wallclock. And silently just not converting was not the proper solution anyway. Also add an assert in pulse/mainloop.c to report the integer overflow that was triggered by the

[pulseaudio-discuss] [PATCH 7/9] module-waveout: Query device for supported samplerate

2011-03-16 Thread Maarten Bosmans
Instead of using a fixed list list of supported rates, ask the wave subsystem whether the rate can be used. --- src/modules/module-waveout.c | 59 + 1 files changed, 36 insertions(+), 23 deletions(-) diff --git a/src/modules/module-waveout.c

[pulseaudio-discuss] [PATCH 8/9] module-waveout: Move thread creation

2011-03-16 Thread Maarten Bosmans
--- src/modules/module-waveout.c | 22 +- 1 files changed, 13 insertions(+), 9 deletions(-) diff --git a/src/modules/module-waveout.c b/src/modules/module-waveout.c index 44b15da..90d0599 100644 --- a/src/modules/module-waveout.c +++ b/src/modules/module-waveout.c @@ -650,7

[pulseaudio-discuss] [PATCH 9/9] module-waveout: Fix record/playback args

2011-03-16 Thread Maarten Bosmans
--- src/modules/module-waveout.c | 20 1 files changed, 12 insertions(+), 8 deletions(-) diff --git a/src/modules/module-waveout.c b/src/modules/module-waveout.c index 90d0599..6c96921 100644 --- a/src/modules/module-waveout.c +++ b/src/modules/module-waveout.c @@ -256,20

[pulseaudio-discuss] [PATCH] Update PA_MODULE_USAGE to be in line with actual implementation

2011-03-18 Thread Maarten Bosmans
--- src/modules/alsa/module-alsa-sink.c|4 +++- src/modules/alsa/module-alsa-source.c |2 ++ src/modules/bluetooth/module-bluetooth-proximity.c |1 - src/modules/module-equalizer-sink.c|4 +++- src/modules/module-hal-detect.c

[pulseaudio-discuss] Get rid of some more warnings

2011-03-19 Thread Maarten Bosmans
Here are the last (relatively) easy fixes for compiler warnings. [PATCH 1/2] Get rid of some warnings: -Wunused-result [PATCH 2/2] Get rid of some warnings: -Wunsafe-loop-optimizations The war is not over yet, my friends. Prepare for the final battle and we shall be victorious! Maarten

  1   2   >