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 <da...@fries.net> -----

Date: Sat, 25 Mar 2017 13:25:54 -0500
From: David Fries <da...@fries.net>
To: Maximiliano Curia <m...@gnuservers.com.ar>
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

Reply via email to