On Fri, Jul 08, 2005 at 05:37:17PM +1000, Erik de Castro Lopo wrote:
>
> It looks like you (or rather sweep) is a victim of the ever changing
> ALSA API. The last Sweep release is over a year old and the ALSA
> API was still in flux back then.
>
> I'll see if I can generate a patch for this over the weekennd (unless
> Conrad beats me to it).
there's a patch in CVS that adds:
#define ALSA_PCM_OLD_HW_PARAMS_API
before the line:
#include <alsa/asoundlib.h>
which should make sweep build (check the list archives for March for
details).
Of course updating driver_alsa.c to use the newer api would probably be
even better.
cheers,
Conrad.

Niki Kovacs wrote:
> Hi,
Hi Niki,
<snip>
> somewhere. I use ALSA (built into the 2.6.12 kernel), and my KDE
<snip>
> -D_REENTRANT-I/usr/X11R6/include -Wall -c `test -f 'driver_alsa.c' || echo
> './'`driver_alsa.c
> driver_alsa.c: In function `alsa_device_setup':
> driver_alsa.c:213: warning: passing arg 3 of
> `snd_pcm_hw_params_set_rate_near'
> makes pointer from integer without a cast
> driver_alsa.c:222: warning: passing arg 3 of
> `snd_pcm_hw_params_set_channels_near' makes pointerfrom integer without
> a cast
> driver_alsa.c:231: warning: passing arg 3 of
> `snd_pcm_hw_params_set_period_size_near' makes pointer from integer
> without a
> cast
> driver_alsa.c:241: warning: passing arg 3 of
> `snd_pcm_hw_params_set_periods_near' makes pointer from integer without
> a cast
> driver_alsa.c:276: error: too few arguments to function
> `snd_pcm_hw_params_get_rate'
> driver_alsa.c:277: error: too few arguments to function
> `snd_pcm_hw_params_get_channels'
It looks like you (or rather sweep) is a victim of the ever changing
ALSA API. The last Sweep release is over a year old and the ALSA
API was still in flux back then.
I'll see if I can generate a patch for this over the weekennd (unless
Conrad beats me to it).
Erik
--
+-----------------------------------------------------------+
Erik de Castro Lopo nospam@... (Yes it's valid)
+-----------------------------------------------------------+
"The RIAA is obsessed to the point of comedy with the frustration
of having its rules broken, without considering whether such rules
might be standing in the way of increased revenues. Indeed,
Napster and Gnutella may turn out to be the two best music-marketing
gimmicks yet devised, if only the RIAA would take its head out of
its ass long enough to realise it."
-- Thomas C Greene on http://www.theregister.co.uk

On Fri, Jul 08, 2005 at 05:37:17PM +1000, Erik de Castro Lopo wrote:
>
> It looks like you (or rather sweep) is a victim of the ever changing
> ALSA API. The last Sweep release is over a year old and the ALSA
> API was still in flux back then.
>
> I'll see if I can generate a patch for this over the weekennd (unless
> Conrad beats me to it).
there's a patch in CVS that adds:
#define ALSA_PCM_OLD_HW_PARAMS_API
before the line:
#include <alsa/asoundlib.h>
which should make sweep build (check the list archives for March for
details).
Of course updating driver_alsa.c to use the newer api would probably be
even better.
cheers,
Conrad.