2012/8/8, David Henningsson <david.henningsson at canonical.com>: > On 07/17/2012 08:45 AM, Deng Zhengrong wrote: >> +#### check test framework #### >> + >> +PKG_CHECK_MODULES(LIBCHECK, [ check ]) >> +AC_SUBST(LIBCHECK_CFLAGS) >> +AC_SUBST(LIBCHECK_LIBS) >> + > > Is it possible to make this build dependency optional? If so, please do, > because > 1) if you just want to compile PulseAudio and not the tests, you > should be able to, and > 2) I have to go through some administrative work if I have to bring in > yet another build dependency to PulseAudio. :-/ Hi David, Thanks for the review, I have a little different opinion: 1. when we're in the process of development, we should always run the tests all the time to track any possible regressions. It's not a good idea to bypass the tests, because that's the value of all these tests, otherwise we could just delete them all. :) 2. it's sad to bring in yet another build dependency, but you know life is not perfect... We've already added e.g. json lib before. Besides, this check framework is already widely used in other programs, e.g. gstreamer. and I've checked with several major linux distribution that this can be easily installed. mostly install check or check-devel or something similar. One command and it's all set. :) deng
