On 2 April 2017 at 15:12, Yann E. MORIN <[email protected]> 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" <[email protected]>
>> Reviewed-by: Eric Engestrom <[email protected]>
>> Tested-by: Eric Engestrom <[email protected]>
>
> Ping? ;-)
>
Yes please. Fwiw
Reviewed-by: Emil Velikov <[email protected]>

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
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to