On 2 April 2017 at 15:12, Yann E. MORIN <yann.morin.1...@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.1...@free.fr> >> Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com> >> Tested-by: Eric Engestrom <eric.engest...@imgtec.com> > > Ping? ;-) > Yes please. Fwiw Reviewed-by: Emil Velikov <emil.veli...@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 _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel