Hi,I've received a patch for xawtv made by David Fries (I maintain xawtv in Debian) that I think it would be nice to include in the xawtv3 [1] git repository and in the next bug fix release. Currently I don't have commit access and I'm not subscribed to the linux-media mailing list because I haven't received a reply about my subscription request.
I'm contacting you (Mauro) as you committed the latest patches in the git repository, could you please review and/or push the patch below?
I'm also CCing the mailing list but I don't expect it to reach it. If the mail reaches the list, please cc me in the reply.
Please let me know if there is an easier way to contact the linux-media group. Happy hacking, [1]: https://git.linuxtv.org/xawtv3.git ----- Forwarded message from David Fries <David@xxxxxxxxx> ----- Date: Sat, 25 Mar 2017 13:25:54 -0500 From: David Fries <David@xxxxxxxxx> To: Maximiliano Curia <maxy@xxxxxxxxxxxxxxxxx> User-Agent: Mutt/1.5.23 (2014-03-12) Subject: [PATCH] xawtv allow ./configure --disable-alsa to compile when alsa is available alsa_loopback is used outside of the HAVE_ALSA check, always define it. Disable alsa_stream.c or the alsa functions are missing symbols. --- I'm debugging a webcam problem, the 'motion' program works once, then fails, xawtv unwedges the camera so it can run again. In trying to figure out what xawtv is doing that motion isn't, I went to compile without audio to cut down on the ioctls to look at and turns out xawtv using audio IS what is unwedging the camera. That's no good for the uvc USB camera driver, or camera, to require audio be setup for it to work properly. Here's a patch to fixup xawtv to compile without alsa. Thanks for supporting this small little program, I would have never thought to look at audio otherwise. common/alsa_stream.c | 2 +- console/radio.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/common/alsa_stream.c b/common/alsa_stream.c index 3e33b5e..85e10b4 100644 --- a/common/alsa_stream.c +++ b/common/alsa_stream.c @@ -28,7 +28,7 @@ #include "config.h" -#ifdef HAVE_ALSA_ASOUNDLIB_H +#if defined(HAVE_ALSA_ASOUNDLIB_H) && defined(HAVE_ALSA) #include <stdio.h> #include <stdlib.h> diff --git a/console/radio.c b/console/radio.c index 186fd3c..d4f7d57 100644 --- a/console/radio.c +++ b/console/radio.c @@ -62,8 +62,8 @@ USB radio devices benefit from a larger default latency */ #define DEFAULT_LATENCY 500 -#if defined(HAVE_ALSA) int alsa_loopback = 1; +#if defined(HAVE_ALSA) char *alsa_playback = NULL; char *alsa_capture = NULL; int alsa_latency = DEFAULT_LATENCY; -- 2.11.0 ----- End forwarded message ----- -- "C makes it easy to shoot yourself in the foot; C++ makes it harder, but when you do it blows your whole leg off." -- Bjarne Stroustrup Saludos /\/\ /\ >< `/
Attachment:
signature.asc
Description: PGP signature
