From 56c203fc09e6e7aeb6ce474e30fe4a93c8d412b4 Mon Sep 17 00:00:00 2001 From: Kai Koehne Date: Wed, 6 May 2020 15:09:37 +0200 Subject: Snippets: Fix dependency issues The widget snippet was sometimes ignored, because the printsupport dependency was not explicit in the top-level src.pro file. This lead to a situation that, if printsupport by chance was already built, it was tested, otherwise silently ignored. This shows that having requires(qtHaveModule()) inside src/ is actually harmful, and they are therefore removed from snippets.pro. Also, the dependencies for the snippets projects are now moved to a central place so that the correctness is easier to check. Pick-to: 5.15 Change-Id: Ice051fa04848040e206c78361fbbcf680383c6b2 Reviewed-by: Paul Wicking --- src/testlib/doc/snippets/snippets.pro | 3 --- 1 file changed, 3 deletions(-) (limited to 'src/testlib/doc') diff --git a/src/testlib/doc/snippets/snippets.pro b/src/testlib/doc/snippets/snippets.pro index d1ca7ca3ec..28b6682b72 100644 --- a/src/testlib/doc/snippets/snippets.pro +++ b/src/testlib/doc/snippets/snippets.pro @@ -1,6 +1,3 @@ -requires(qtHaveModule(sql)) -requires(qtHaveModule(widgets)) - TEMPLATE = app TARGET = testlib_cppsnippet QT = core testlib sql widgets -- cgit v1.2.3