[PATCH v2] configure: add option to disable tests

Emil Velikov emil.l.velikov at gmail.com
Mon Apr 3 17:35:34 UTC 2017


On 2 April 2017 at 15:12, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> Hello,
>
> On 2017-03-08 18:25 +0100, Yann E. MORIN spake thusly:
>> When building for a product, tests are not needed.
>>
>> Besides, one test requires a C++ compiler, which is not always
>> available.
>>
>> So, add an option to configure to disable building tests altogether.
>>
>> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
>> Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
>> Tested-by: Eric Engestrom <eric.engestrom at imgtec.com>
>
> Ping? ;-)
>
Yes please. Fwiw
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

Personally, I would toggle that to disabled and force-enable via
AM_DISTCHECK_CONFIGURE_FLAGS as mentioned in the other patch.
We really want to folding the the C++ check/test here, IMHO.

-Emil


More information about the wayland-devel mailing list