[alsa-devel] [PATCH 02/10] alsabat: add standalone mode

[Lu, Han]

> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai at suse.de]
> Sent: Tuesday, March 8, 2016 6:19 PM
> To: Lu, Han <han.lu at intel.com>
> Cc: liam.r.girdwood at linux.intel.com; Gautier, Bernard
> <bernard.gautier at intel.com>; Popescu, Edward C
> <edward.c.popescu at intel.com>; alsa-devel at alsa-project.org
> Subject: Re: [PATCH 02/10] alsabat: add standalone mode
> 
> On Wed, 02 Mar 2016 09:53:12 +0100,
> han.lu at intel.com wrote:
> >
> > From: "Lu, Han" <han.lu at intel.com>
> >
> > Add support for standalone mode where alsabat will run on a different
> > machine to the one being tested.
> > In standalone mode, the sound data can be generated, playback and
> > captured just like in normal mode, but will not be analyzed.
> > The alsabat being built without libfftw3 support is always work in
> > standalone mode.
> > The alsabat in normal mode can also bypass data analysis using option
> > "--standalone".
> >
> > Signed-off-by: Lu, Han <han.lu at intel.com>
> >
> > diff --git a/bat/Makefile.am b/bat/Makefile.am index 8dfafa9..5646e9a
> > 100644
> > --- a/bat/Makefile.am
> > +++ b/bat/Makefile.am
> > @@ -6,7 +6,6 @@ EXTRA_DIST = alsabat.1  alsabat_SOURCES = \
> >  	bat.c \
> >  	common.c \
> > -	analyze.c \
> >  	signal.c \
> >  	convert.c \
> >  	alsa.c
> > @@ -15,8 +14,12 @@ noinst_HEADERS = \
> >  	common.h \
> >  	bat-signal.h \
> >  	alsa.h \
> > -	convert.h \
> > -	analyze.h
> > +	convert.h
> > +
> > +if HAVE_LIBFFTW3
> > +alsabat_SOURCES += analyze.c
> > +noinst_HEADERS += analyze.h
> > +endif
> 
> I wonder how "make dist" works if running on a system without fftw3...
> Does it pack these conditional files, too?
> 
> 
> Takashi

[han] after remove libfftw3 by "sudo apt-get remove libfftw3-dev", I cloned
a new alsabat repo, patch and build with
	./gitcompile
	./configure
	make
	sudo make install
and then ran "make dist", and the analyze.c and analyze.h are included in the
.bz2 and the .gz packages. I suppose it's the result we expected?

BR,
Han