Re: [pulseaudio-discuss] [PACKAGERS] New dep

2011-05-27 Thread Maarten Bosmans
2011/5/27 Luke Yelavich them...@ubuntu.com: On Fri, May 27, 2011 at 04:52:02AM EST, Arun Raghavan wrote: The website worked fine as of a month ago. I don't know why it's down now. The library serves the purpose we need well (light weight, doesn't invent its own type system, allows you to parse

Re: [pulseaudio-discuss] [PACKAGERS] New dep

2011-05-26 Thread Maarten Bosmans
2011/5/16 Colin Guthrie gm...@colin.guthr.ie: Hi, I've just pushed Arun's (mostly, Pierre-Louis also had a hand!) passthrough work to master. This carries with it a new dependency: json-c What is the upstream location of that software? Googling for json implementations gives a lot of small

Re: [pulseaudio-discuss] How to debug a protocol error message?

2011-05-24 Thread Maarten Bosmans
2011/5/23 Quinn Plattel qie...@gmail.com: Hi, I am currently having problems with communication between a version 0.9.22 pulseaudio client and a version 0.9.15 pulseaudio server. Here are the steps to reproduce the error message: pulseaudio server side: ssh -X -R 4713:localhost:4713

Re: [pulseaudio-discuss] [RFC] Source Output Volumes

2011-05-20 Thread Maarten Bosmans
2011/5/20 Colin Guthrie gm...@colin.guthr.ie: Hello, As some of you know I've been working on restoring a little more symmetry to our API to allow the adjustment of source output (capture stream) volumes. In the past, when stream volumes were added to sink inputs, it was thought that this

Re: [pulseaudio-discuss] [RFC] Source Output Volumes

2011-05-20 Thread Maarten Bosmans
2011/5/20 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 20/05/11 10:23 did gyre and gimble: With flat volumes, adding per-stream volumes to capture streams makes sense. With these changes, would it still be possible to record audio without software gain

Re: [pulseaudio-discuss] Receiving RTP streams with pulseaudio

2011-05-18 Thread Maarten Bosmans
2011/5/18 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Rémi Denis-Courmont at 18/05/11 07:37 did gyre and gimble: On Tue, 17 May 2011 21:51:35 +0200, Michael Trunner mich...@trunner.de wrote: I'm trying to setup Pulseaudio to receive RTP/Multicast steams. Sender should be vlc, (and

Re: [pulseaudio-discuss] Reciving RTP streams with pulseaudio

2011-05-17 Thread Maarten Bosmans
The catch is that module-rtp-receive waits for a SDP/SAP announcement on the specified multicast group before it begins playing the RTP audio stream. VLC probably only send an RTP stream. I have no clear recipe for you how to instruct VLC to send SAP messages. You could of course send VLC output

Re: [pulseaudio-discuss] Is it possible to compress pulseaudio's network streams?

2011-05-11 Thread Maarten Bosmans
There are currently no plans and no active development on compressed network streams for PulseAudio. That said, it would be nice to have some day. It seems that the consensus is that the CELT codec would be best suited for this, because of it's tight control over latency. When streaming audio

Re: [pulseaudio-discuss] module combine not respecting requested latencies

2011-05-09 Thread Maarten Bosmans
2011/5/9 Maarten Lankhorst m.b.lankho...@gmail.com: Hi all, I was toying around with the combine module, but it appears it adds a lot latency for no good reason, changing it from 200 ms to 10 ms makes some applications like wine happier. I wanted to see if I could fix it myself, but I must

Re: [pulseaudio-discuss] ORC buildsystem problems

2011-05-03 Thread Maarten Bosmans
2011/5/3 Paul Menzel paulepan...@users.sourceforge.net: Am Dienstag, den 03.05.2011, 08:39 +0530 schrieb Arun Raghavan: On Mon, 2011-05-02 at 11:52 +0200, Maarten Bosmans wrote: Recently, I encountered some problems when enabling orc in some less usual situations. When compiling

[pulseaudio-discuss] Cleanup of configure.ac

2011-04-30 Thread Maarten Bosmans
This patch series cleans up the --enable-feature handling part of configure.ac. It changes the testing of features so that they are all done in the same way. The linecount of configure.ac is reduced from around 1800 with 450 lines. This is partly because os using macro's, partly due to

[pulseaudio-discuss] [PATCH 2/7] build-system: Use AS_IF macro for configure output

2011-04-30 Thread Maarten Bosmans
And add some HAVE_[feature] variables for clarity. --- configure.ac | 198 ++ 1 files changed, 48 insertions(+), 150 deletions(-) diff --git a/configure.ac b/configure.ac index a816052..403b935 100644 --- a/configure.ac +++ b/configure.ac

[pulseaudio-discuss] [PATCH 4/7] build-system: Move dependency error messages to outer scope

2011-04-30 Thread Maarten Bosmans
--- configure.ac | 107 +++-- 1 files changed, 43 insertions(+), 64 deletions(-) diff --git a/configure.ac b/configure.ac index 72cfbb7..18a136d 100644 --- a/configure.ac +++ b/configure.ac @@ -576,13 +576,12 @@ if test x$enable_x11 != xno;

[pulseaudio-discuss] [PATCH 3/7] build-system: Move AC_DEFINE to separate line with AS_IF

2011-04-30 Thread Maarten Bosmans
--- configure.ac | 59 - 1 files changed, 17 insertions(+), 42 deletions(-) diff --git a/configure.ac b/configure.ac index 403b935..72cfbb7 100644 --- a/configure.ac +++ b/configure.ac @@ -584,14 +584,11 @@ else HAVE_X11=0 fi -if

[pulseaudio-discuss] [PATCH 5/7] build-system: Replace some more conditionals with AS_IF

2011-04-30 Thread Maarten Bosmans
--- configure.ac | 311 +++--- 1 files changed, 103 insertions(+), 208 deletions(-) diff --git a/configure.ac b/configure.ac index 18a136d..9755800 100644 --- a/configure.ac +++ b/configure.ac @@ -571,15 +571,10 @@ PKG_PROG_PKG_CONFIG

[pulseaudio-discuss] [PATCH 7/7] build-system: Small fixes

2011-04-30 Thread Maarten Bosmans
--- configure.ac | 24 ++-- 1 files changed, 10 insertions(+), 14 deletions(-) diff --git a/configure.ac b/configure.ac index 79df100..16fb03d 100644 --- a/configure.ac +++ b/configure.ac @@ -382,7 +382,6 @@ AM_CONDITIONAL(HAVE_AF_UNIX, test x$HAVE_AF_UNIX = x1) # Linux

[pulseaudio-discuss] [PATCH] Make connect-stress test compile for win32

2011-04-30 Thread Maarten Bosmans
--- src/tests/connect-stress.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/tests/connect-stress.c b/src/tests/connect-stress.c index 5476675..eadcf88 100644 --- a/src/tests/connect-stress.c +++ b/src/tests/connect-stress.c @@ -118,7 +118,7 @@ static void

Re: [pulseaudio-discuss] More patches for better OS X support

2011-04-24 Thread Maarten Bosmans
2011/4/23 Colin Guthrie gm...@colin.guthr.ie: This should still work for you on OSX (if not, then I apologise!). I added the OS_IS_DARWIN bit as the API for pthread_setname_np does not appear to be totally stable (two args here, and I presume just one arg on OSX). I figured this may actually

Re: [pulseaudio-discuss] Pulseaudio recording device re-direct from local machine over cable broadband to remote server / latency problem

2011-04-18 Thread Maarten Bosmans
2011/4/18 Nick Holloway ourm...@hotmail.com: Thanks Sean and Marten, this has answered my question, very much appreciated. I had hoped to send my local machine's microphone (for voice) over the Internet to the remote machine, but through all the various configuration I've done the

Re: [pulseaudio-discuss] Pulseaudio recording device re-direct from local machine over cable broadband to remote server / latency problem

2011-04-17 Thread Maarten Bosmans
2011/4/17 Sean McNamara smc...@gmail.com: I don't know if pulseaudio supports any kind of protocol compression these days, but traditionally it does not. And due to that, it is generally unsuitable for use over the public internet. Lossy compression such as mp3, and protocols such as RTP and

Re: [pulseaudio-discuss] RFC: Filter module loader

2011-04-15 Thread Maarten Bosmans
2011/4/15 pl bossart bossart.nos...@gmail.com: Here is my first draft of a filter module to automatically load equalizer and/or echo-cancel modules if automagically and in a manual but convenient way. Thanks for these patches, this is interesting, Arun and I were talking last week about

Re: [pulseaudio-discuss] [PATCH] pa_poll(): Simplify detectin of invalid fds in select() emulation mode

2011-04-14 Thread Maarten Bosmans
2011/4/14 Daniel Mack zon...@gmail.com: On Thu, Apr 14, 2011 at 12:46 AM, Maarten Bosmans mkbosm...@gmail.com wrote: return (fd = 0) (fcntl(fd, F_GETFL) != -1); For Windows, both fcntl and F_GETFL are not available. Sigh. Ok, thanks for the feedback. Attached is a new patch which keeps

Re: [pulseaudio-discuss] [PATCH] pa_poll(): Simplify detectin of invalid fds in select() emulation mode

2011-04-13 Thread Maarten Bosmans
2011/4/13 Daniel Mack zon...@gmail.com: Attached is a patch to fix for one out of the few problems I still fight with on OS X. I'm not sure whether this change breaks Windows support, so I'd be happy if someone could give it a try. This is indeed a problem. return (fd = 0) (fcntl(fd,

Re: [pulseaudio-discuss] [PATCH] call-state-tracker: New component.

2011-04-05 Thread Maarten Bosmans
What are the users of this state tracker? Is it only some out-of-tree Meego modules? Are you planning to submit those too? At least a snippet of how it should be used would be nice. If I read correctly it is a global boolean state tracker that is a thin wrapper around pa_shared. Is it possible to

Re: [pulseaudio-discuss] Serversound 2 ThinClient

2011-04-04 Thread Maarten Bosmans
Please note this is a English-speaking mailing list. Below is my answer in English, hopefully you will be able to understand it. 2011/4/4 badgerman bad-ger-...@web.de: Hallo liebe Listenleser, ich habe eine SLES11-Kvm-Maschine mit openSuse11.2/3/4 Gastsystemen. Das einloggen erfolgt über

Re: [pulseaudio-discuss] Help with pulsaudio

2011-03-28 Thread Maarten Bosmans
-Original Message- From: Maarten Bosmans [mailto:mkbosm...@gmail.com] Sent: 27 March 2011 19:54 To: General PulseAudio Discussion Cc: Daniel Mack; Yousif, Amgad (UK) Subject: Re: [pulseaudio-discuss] Help with pulsaudio                    *** WARNING ***  This message has originated

Re: [pulseaudio-discuss] Getting rid of annoying (but harmless) linking warnings

2011-03-26 Thread Maarten Bosmans
Check whether output of pkg-config --libs dbus-1 pkg-config --libs sndfile contains any offensive parameters. If there is anything out of the ordinary, cat libpulsecommon-1.0.la could also reveal something, as I suspect your problem has more to do with libtool than with autotools. Maarten

[pulseaudio-discuss] pactl improvements

2011-03-26 Thread Maarten Bosmans
This series superseeds the two patches previously sent to the list. [PATCH 1/4] pactl: Accept more volume specification formats This has relative volume adjustments now. I opted for +/- prefix to denote a relative adjustment. [PATCH 2/4] pactl: Add subcommands to the list command Roughly the

[pulseaudio-discuss] [PATCH 1/4] pactl: Accept more volume specification formats

2011-03-26 Thread Maarten Bosmans
With this you can specify the volume with 6554, 10%, 0.001 or -60dB, all resulting in the same volume change. --- src/utils/pactl.c | 229 +++-- 1 files changed, 171 insertions(+), 58 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c

[pulseaudio-discuss] [PATCH 2/4] pactl: Add subcommands to the list command

2011-03-26 Thread Maarten Bosmans
--- src/utils/pactl.c | 67 +--- 1 files changed, 53 insertions(+), 14 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c index 9b6050a..e3c2aa2 100644 --- a/src/utils/pactl.c +++ b/src/utils/pactl.c @@ -48,6 +48,7 @@ static

[pulseaudio-discuss] [PATCH 3/4] pactl: Separate stat and info actions

2011-03-26 Thread Maarten Bosmans
--- src/utils/pactl.c | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c index e3c2aa2..cfa96fe 100644 --- a/src/utils/pactl.c +++ b/src/utils/pactl.c @@ -90,6 +90,7 @@ static enum { NONE, EXIT, STAT, +INFO,

[pulseaudio-discuss] [PATCH 4/4] pactl: Add short output format for list action

2011-03-26 Thread Maarten Bosmans
--- src/utils/pactl.c | 88 +++- 1 files changed, 79 insertions(+), 9 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c index cfa96fe..194183d 100644 --- a/src/utils/pactl.c +++ b/src/utils/pactl.c @@ -62,6 +62,7 @@ static uint32_t

Re: [pulseaudio-discuss] [PATCH 2/2] Move compile-time checks around pa_run_from_build_tree to core-util

2011-03-24 Thread Maarten Bosmans
2011/3/24 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Colin Guthrie at 24/03/11 09:02 did gyre and gimble: 'Twas brillig, and Maarten Bosmans at 22/03/11 15:06 did gyre and gimble: Is there a better way than #if defined(__linux__) !defined(__OPTIMIZE__) to check for a debug build

[pulseaudio-discuss] [PATCH 1/2] pactl: Accept more volume specification formats

2011-03-24 Thread Maarten Bosmans
With this you can specify the volume with 6554, 10%, 0.001 or -60dB, all resulting in the same volume change. --- src/utils/pactl.c | 98 +--- 1 files changed, 62 insertions(+), 36 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c

[pulseaudio-discuss] [PATCH 2/2] pactl: Add subcommands to the list command

2011-03-24 Thread Maarten Bosmans
--- src/utils/pactl.c | 62 +++- 1 files changed, 51 insertions(+), 11 deletions(-) diff --git a/src/utils/pactl.c b/src/utils/pactl.c index 672bfbb..11ddcb3 100644 --- a/src/utils/pactl.c +++ b/src/utils/pactl.c @@ -48,6 +48,7 @@ static

Re: [pulseaudio-discuss] [PATCH 1/2] pactl: Accept more volume specification formats

2011-03-24 Thread Maarten Bosmans
2011/3/24 Maarten Bosmans mkbosm...@gmail.com: With this you can specify the volume with 6554, 10%, 0.001 or -60dB, all resulting in the same volume change. I was also going to add relative volumes, such as +3dB and -5%, by detecting a + or - sign in the volume. But that clashes

[pulseaudio-discuss] [PATCH 1/2] win32: Simplify dl_search_path code

2011-03-23 Thread Maarten Bosmans
And add #include sys/stat.h, needed by the code introduced in f7acd4bd. --- src/daemon/daemon-conf.c | 19 ++- 1 files changed, 6 insertions(+), 13 deletions(-) diff --git a/src/daemon/daemon-conf.c b/src/daemon/daemon-conf.c index 9b530a8..59579d9 100644 ---

[pulseaudio-discuss] [PATCH 2/2] Move compile-time checks around pa_run_from_build_tree to core-util

2011-03-23 Thread Maarten Bosmans
To make the code cleaner and have the checks all in one place. --- src/daemon/daemon-conf.c | 11 +-- src/modules/alsa/alsa-mixer.c | 12 src/pulsecore/core-util.c |3 --- src/pulsecore/core-util.h |5 + 4 files changed, 10 insertions(+), 21

Re: [pulseaudio-discuss] [PATCH 2/2] Move compile-time checks around pa_run_from_build_tree to core-util

2011-03-23 Thread Maarten Bosmans
Is there a better way than #if defined(__linux__) !defined(__OPTIMIZE__) to check for a debug build? By default the CFLAGS contain -g -O2, so __OPTIMIZE__ will not be defined and running uninstalled does not work. Maarten ___ pulseaudio-discuss mailing

Re: [pulseaudio-discuss] [PATCH 3/5] osx: add -headerpad_max_install_names to LDFLAGS

2011-03-23 Thread Maarten Bosmans
2011/3/21 Daniel Mack dan...@zonque.org: On Sun, Mar 20, 2011 at 7:14 PM, Tanu Kaskinen ta...@iki.fi wrote: On Sun, 2011-03-20 at 18:39 +0100, Daniel Mack wrote: This is needed for sufficient padding of library names in linked binaries. ---  src/Makefile.am |    5 +  1 files changed, 5

Re: [pulseaudio-discuss] [RFC PATCH] Log PCM samples to files

2011-03-23 Thread Maarten Bosmans
2011/3/21 Vincent Becker vincentx.bec...@intel.com: Hi, I recently created a new PA module (module-log-pcm) which is able to log PCM samples from any source/sink and its corresponding outputs/inputs. For testing audio quality, checking signal processing or verifying the right

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

2011-03-23 Thread Maarten Bosmans
2011/3/21 Becker, VincentX vincentx.bec...@intel.com: 'Twas brillig, and Vincent Becker at 18/03/11 10:23 did gyre and gimble: This patch enables logging of text debug messages (pa_log feature) into a file or a device driver. Example : pulseaudio --log-target=file:./mylog.txt Many thanks for

Re: [pulseaudio-discuss] [PATCH] Get rid of some warnings

2011-03-20 Thread Maarten Bosmans
2011/3/20 Colin Guthrie gm...@colin.guthr.ie: 'Twas brillig, and Maarten Bosmans at 19/03/11 15:26 did gyre and gimble: Mostly warnings about unused stuff. Furthermore, the first hunk is a fix for the change in 177948a6. Finally, comment in AEC_dtd was translated and the code simplified

[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

[pulseaudio-discuss] [PATCH 1/2] Get rid of some warnings: -Wunused-result

2011-03-19 Thread Maarten Bosmans
modules/module-default-device-restore.c: In function ‘load’: modules/module-default-device-restore.c:67: warning: ignoring return value of ‘fgets’, declared with attribute warn_unused_result [-Wunused-result] modules/module-default-device-restore.c:88: warning: ignoring return value of ‘fgets’,

[pulseaudio-discuss] [PATCH 2/2] Get rid of some warnings: -Wunsafe-loop-optimizations

2011-03-19 Thread Maarten Bosmans
pulsecore/core-util.c: In function ‘pa_hexstr’: pulsecore/core-util.c:1858: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations] modules/alsa/alsa-mixer.c: In function ‘pa_alsa_decibel_fix_dump’: modules/alsa/alsa-mixer.c:3678: warning: cannot optimize

Re: [pulseaudio-discuss] [PATCH 1/2] daemon: Fix missing include - cpu-orc.h

2011-03-19 Thread Maarten Bosmans
Can this go in fast? as it's annoying to have to patch this every time myself. Maarten 2011/3/19 Arun Raghavan arun.ragha...@collabora.co.uk: ---  src/daemon/main.c |    1 +  1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/daemon/main.c b/src/daemon/main.c index

[pulseaudio-discuss] [PATCH] Get rid of some warnings

2011-03-19 Thread Maarten Bosmans
Mostly warnings about unused stuff. Furthermore, the first hunk is a fix for the change in 177948a6. Finally, comment in AEC_dtd was translated and the code simplified slightly. CC module_bluetooth_device_la-module-bluetooth-device.lo modules/bluetooth/module-bluetooth-device.c: In function

Re: [pulseaudio-discuss] [PATCH] Get rid of some warnings

2011-03-19 Thread Maarten Bosmans
-optimizations] Not our code I think I have had enough of this. But if there are others that are interested in fixing warnings: compiling with clang works now in git master and gives plenty of warnings to play with ;-) Maarten 2011/3/19 Maarten Bosmans mkbosm...@gmail.com: Mostly warnings about unused

[pulseaudio-discuss] Merge of pavucontrol and paprefs

2011-03-19 Thread Maarten Bosmans
With the approaching switch to freedesktop.org infrastructure, only pulseaudio and and pavucontrol are going to be migrated, if I understand it correctly. Of the other utils, paman and padevchooser were already deprecated for several years. (I have closed all the tickets as wontfix, to emphasize

[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] 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] 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,

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] [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

[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

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

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

[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] 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

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?

[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] [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

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] 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] [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] 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] 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] 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

  1   2   >