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

2011-05-27 Thread Maarten Bosmans
2011/5/27 Luke Yelavich : > 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 out values

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

2011-05-26 Thread Maarten Bosmans
2011/5/16 Colin Guthrie : > 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 libraries, it i

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

2011-05-24 Thread Maarten Bosmans
2011/5/23 Quinn Plattel : > 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 > "PULSE_SERVER=lo

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

2011-05-20 Thread Maarten Bosmans
2011/5/20 Colin Guthrie : > '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

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

2011-05-20 Thread Maarten Bosmans
2011/5/20 Colin Guthrie : > 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 feature would

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

2011-05-18 Thread Maarten Bosmans
2011/5/18 Colin Guthrie : > '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 >> wrote: >>> I'm trying to setup Pulseaudio to receive RTP/Multicast steams. Sender >>> should be vlc, (and receiver should be pulseau

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 (esp

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

2011-05-09 Thread Maarten Bosmans
2011/5/9 Maarten Lankhorst : > 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 admit I don't under

Re: [pulseaudio-discuss] ORC buildsystem problems

2011-05-03 Thread Maarten Bosmans
2011/5/3 Paul Menzel : > 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. >> > >>

[pulseaudio-discuss] ORC buildsystem problems

2011-05-02 Thread Maarten Bosmans
Recently, I encountered some problems when enabling orc in some less usual situations. When compiling with --enable-orc from a tarball generated from a --disable-orc configured tree, the following error occurs. make[2]: *** No rule to make target `pulsecore/svolume-orc-gen.c', needed by `all'. St

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

[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") # Lin

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

[pulseaudio-discuss] [PATCH 6/7] build-system: Rearrange database selection

2011-04-30 Thread Maarten Bosmans
--- configure.ac | 64 +- 1 files changed, 32 insertions(+), 32 deletions(-) diff --git a/configure.ac b/configure.ac index 9755800..79df100 100644 --- a/configure.ac +++ b/configure.ac @@ -654,48 +654,45 @@ AS_IF([test "x$HAVE_LIBSAMPLERA

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

[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 1/7] build-system: Simplify AC_ARG_ENABLE usage

2011-04-30 Thread Maarten Bosmans
Make use of the enable_[feature] variable automatically defined by AC_ARG_ENABLE instead of defining our own variable. --- configure.ac | 415 +++-- 1 files changed, 109 insertions(+), 306 deletions(-) diff --git a/configure.ac b/configure.ac i

[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] Cleanup of configure.ac

2011-04-30 Thread Maarten Bosmans
This patch series cleans up the --enable- 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 simplification o

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

2011-04-24 Thread Maarten Bosmans
2011/4/23 Colin Guthrie : > 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 break windows

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 : > 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 delay/latency problem pe

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 : > 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 Icecast, exist

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

2011-04-15 Thread Maarten Bosmans
2011/4/15 pl bossart : >> 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 better support for e

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 : > On Thu, Apr 14, 2011 at 12:46 AM, Maarten Bosmans 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 patc

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 : > 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, F_GETFL) != -1

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] Semaphore lockup when using threaded mainloops excessively

2011-04-05 Thread Maarten Bosmans
2011/4/5 Daniel Mack : > Was anyone able to reproduce this? The test is called "connect-stress" > in the mainline tree. As I said on IRC, I was able to reproduce this on Ubuntu, but it needed two connect-stress running in parallel to get the daemon to fail. How are you coming along with investiga

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 : > Hallo liebe Listenleser, > > ich habe eine SLES11-Kvm-Maschine mit openSuse11.2/3/4 Gastsystemen. > Das einloggen erfolgt über ThinClients an den

Re: [pulseaudio-discuss] Help with pulsaudio

2011-03-28 Thread Maarten Bosmans
sink on the Linux side. > Thanks again. > > Regards > Amgad Yousif Maarten > -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

2011-03-27 Thread Maarten Bosmans
2011/3/27 Daniel Mack : > On Thu, Mar 24, 2011 at 6:39 PM, Yousif, Amgad (UK) > wrote: >> Hi >> >> I wanted some help with using pulsaudio to getting sound from a server >> machine to a client machine. Please let me know the best person to contact >> for this. > > This should help you: > >  http:/

[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

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

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

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 2011/3

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

2011-03-24 Thread Maarten Bosmans
2011/3/24 Maarten Bosmans : > 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 with the absolute dB

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

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

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 : > '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(__O

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 : >>>'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 your p

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

2011-03-23 Thread Maarten Bosmans
2011/3/21 Vincent Becker : > 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 > implementation of a sound device

Re: [pulseaudio-discuss] Pulseaudio passthrough branch doesn't read /etc/pulse/default.pa

2011-03-23 Thread Maarten Bosmans
2011/3/21 Dark Shadow : > On Sun, Mar 20, 2011 at 5:58 PM, Dark Shadow > wrote: >> I have successfully bitstreamed DTS-HD from xbmc through PA but in >> using the passthrough git branch it doesn't read that config file >> which causes problems for me forcing a sink. >> >> How can I fix this since

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 : > On Sun, Mar 20, 2011 at 7:14 PM, Tanu Kaskinen 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 insertions(+

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 mailin

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

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

2011-03-23 Thread Maarten Bosmans
And add #include , 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 --- a/src/daemon/daemon-conf.c ++

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

2011-03-20 Thread Maarten Bosmans
2011/3/20 Colin Guthrie : > '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 th

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

2011-03-20 Thread Maarten Bosmans
2011/3/20 Tanu Kaskinen : > 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 insertions(+), 0 deletions(-) >> >> diff --git a/src/Makefile.am b/src/Ma

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

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

2011-03-19 Thread Maarten Bosmans
nite loops [-Wunsafe-loop-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 : > Mostly warnings about

[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 ‘a

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 : > --- >  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 533c4c3..f7aed51 100644 > --- a/s

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

[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’, d

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

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

2011-03-19 Thread Maarten Bosmans
2011/3/18 Colin Guthrie : > 'Twas brillig, and Maarten Bosmans at 16/03/11 09:55 did gyre and gimble: >> 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

Re: [pulseaudio-discuss] [PATCH 2/9] Find modules and config files relative to the installed libraries.

2011-03-19 Thread Maarten Bosmans
2011/3/18 Colin Guthrie : > 'Twas brillig, and Maarten Bosmans at 16/03/11 09:55 did gyre and gimble: >> +#ifdef OS_IS_WIN32 >> +    { >> +        char *toplevel = pa_win32_get_toplevel(NULL); >> +        c->dl_search_path = pa_sprintf_malloc("%s"

[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] [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 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 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 b/src/modules

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

[pulseaudio-discuss] [PATCH 5/9] Use pulsecore/arpa-inet.h to make arpa/inet.h functionality available

2011-03-16 Thread Maarten Bosmans
Automatically use replacement function on platforms (win32) where not all arpa/inet.h is available natively. --- configure.ac |2 +- src/Makefile.am|3 +- src/modules/module-protocol-stub.c |4 +- src/modules/raop/raop_client.c |1 - s

[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 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 2/9] Find modules and config files relative to the installed libraries.

2011-03-16 Thread Maarten Bosmans
Do not use replace %PULSE_ROOT% from the environment. --- src/Makefile.am |4 -- src/daemon/daemon-conf.c |7 +++ src/pulsecore/core-util.c | 123 src/pulsecore/core-util.h |4 ++ src/pulsecore/dllmain.c |4 +- 5 files chan

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

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

[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 @@ lo

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

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

[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 default.

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

2011-03-12 Thread Maarten Bosmans
onditionally) 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@ replace

[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] [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 : > @@ -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 (pa_startswith(strin

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 : > --- >  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 > i

[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 = \ defau

[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 if functionality of header.h is used. --- src/pulse/browser.c |2 +- src/pulse/context.c | 13 + src/pulse/ext-device-manager.c |5 ++--- src/pulse/ext-stream-restore.c

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 : > This patch enables logging of text debug messa

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

2011-03-04 Thread Maarten Bosmans
2011/3/4 ssrk : > 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 tried to run

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

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

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 : > 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 they d

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

2011-03-02 Thread Maarten Bosmans
2011/3/2 Colin Guthrie : > '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/test

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

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

[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 b/src/daemon/d

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

Re: [pulseaudio-discuss] MacOS X Support

2011-03-02 Thread Maarten Bosmans
2011/2/27 Stephen Lee : > 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 noticed that iTun

Re: [pulseaudio-discuss] Windows binaries

2011-02-27 Thread Maarten Bosmans
2011/2/27 Sean McNamara : > Hi, > > On Fri, Feb 25, 2011 at 8:28 AM, Maarten Bosmans wrote: >> 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 availabl

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

2011-02-27 Thread Maarten Bosmans
2011/2/27 Colin Guthrie : > '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? One >>

[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 https://build

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

2011-02-25 Thread Maarten Bosmans
2011/2/25 Becker, VincentX : >>From: Maarten Bosmans [mailto:mkbosm...@gmail.com] >>Sent: Friday, February 25, 2011 1:10 AM >>To: Becker, VincentX >>Cc: General PulseAudio Discussion >>Subject: Re: [pulseaudio-discuss] [PATCH 1/2] Add a target to the PA log >>f

  1   2   >