[pulseaudio-discuss] [PATCH v2 1/6] build-sys: add `check` test framework

[rong deng]

2012/8/9, Tanu Kaskinen <tanuk at iki.fi>:
> On Wed, 2012-08-08 at 22:16 +0800, rong deng wrote:
>> 2012/8/8, David Henningsson <david.henningsson at canonical.com>:
>> > But what about a user of e g Linux From Scratch or other minimalist
>> > distros, who just wants to set up and build PulseAudio as quick as
>> > possible? Why should (s)he be required to also bring in the check
>> > package just for building it?
>>
>> Other packages would already require the check package.
>> From the webpage of check framework: http://check.sourceforge.net/
>> there's a section called "Projects Using Check", there are several
>> common projects using it. e.g. XCB and gstreamer. So if the user wants
>> to install X environment he/she would already install XCB and the
>> check dependency is there.
>
> Note that check is only an optional dependency in gstreamer (I didn't
> check whether this is also the case with XCB, but I'd be surprised if it
> wasn't):
> http://cgit.freedesktop.org/gstreamer/gstreamer/tree/configure.ac#n557

Hi Tanu,

Thanks for your reminder, OK. Now I think it's a valid point. So I've
sent out a patch to make check optional. But by default it's enabled,
I think running tests is a good practice that should be encouraged.

>From my last time trying LFS, it was years ago, it runs tests for each
package to make sure the installation from LFS isn't misbehaved. But I
haven't checked the current status for LFS lately.

If you have time, or David has the time, please help to review that
patch. I've done some simple configurations it works for me.

Thanks!