Re: [PATCH 02/10] alsabat: add standalone mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai@xxxxxxx]
> Sent: Tuesday, March 8, 2016 6:19 PM
> To: Lu, Han <han.lu@xxxxxxxxx>
> Cc: liam.r.girdwood@xxxxxxxxxxxxxxx; Gautier, Bernard
> <bernard.gautier@xxxxxxxxx>; Popescu, Edward C
> <edward.c.popescu@xxxxxxxxx>; alsa-devel@xxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 02/10] alsabat: add standalone mode
> 
> On Wed, 02 Mar 2016 09:53:12 +0100,
> han.lu@xxxxxxxxx wrote:
> >
> > From: "Lu, Han" <han.lu@xxxxxxxxx>
> >
> > 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@xxxxxxxxx>
> >
> > 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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux