summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorUlf Hermann <ulf.hermann@qt.io>2017-01-03 18:37:51 +0100
committerUlf Hermann <ulf.hermann@qt.io>2017-01-04 15:13:42 +0000
commitfa26283dfa36eba3e71f7c71e27d01f4a28c1d4c (patch)
tree21386f387adabfe8f1e80ba8166c45cf572bd353
parent858ae1139a5cf4f3a7dd1d0d7f056ec04846f3c8 (diff)
Be more explicit about linguist's module dependencies
The conditional addition of printsupport made it seem like linguist was intended to compile also without printsupport. However, QPrintDialog is extensively used, with no #ifdef guards whatsoever. Therefore, printsupport is apparently not optional. It doesn't make a lot of sense to build desktop tools with reduced Qt configurations. Doing so would clutter the code with #ifdefs for very little benefit. Change-Id: I815a0cfa90da6c302e894e0c10c18184128123fa Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
-rw-r--r--src/linguist/linguist/linguist.pro3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/linguist/linguist/linguist.pro b/src/linguist/linguist/linguist.pro
index de04e514a..742a04144 100644
--- a/src/linguist/linguist/linguist.pro
+++ b/src/linguist/linguist/linguist.pro
@@ -1,5 +1,4 @@
-QT += core-private widgets xml uitools-private
-qtHaveModule(printsupport): QT += printsupport
+QT += core-private widgets xml uitools-private printsupport
DEFINES += QT_NO_CAST_FROM_ASCII QT_NO_CAST_TO_ASCII