aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOswald Buddenhagen <oswald.buddenhagen@qt.io>2018-10-12 16:30:03 +0200
committerLiang Qi <liang.qi@qt.io>2018-10-18 10:59:31 +0000
commit4c2fea3d5d22c6b499e82cd1dd21547e92b19edc (patch)
treeb1921d2a254925a64e457b697bd4e38c82ce6051
parent4061156381e8049f27840b8e064908f4d6056c99 (diff)
fix setting QTDIR_build for default_pre.prf
adding it to the super cache makes no sense, as that's read before spec_pre.prf, which clears CONFIG. use the regular cache instead. submodules do not inherit that setting, and they don't need to as per comment. also, don't keep adding it to the cache - the cache() function doesn't check whether the added value is already present, so we need to do that manually. amends 31590c0b65. Fixes: QTBUG-63452 Change-Id: I608559a80a7c772dbdaacfaaadb4fc268b442c39 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
-rw-r--r--qt.pro12
1 files changed, 9 insertions, 3 deletions
diff --git a/qt.pro b/qt.pro
index 9ce0a694..81553664 100644
--- a/qt.pro
+++ b/qt.pro
@@ -1,7 +1,13 @@
# Create the super cache so modules will add themselves to it.
-# Also suppress the license check on subsequent "visits". The first
-# visit will skip it anyway due to not having a compiler set up yet.
-cache(CONFIG, add super, $$list(QTDIR_build))
+cache(, super)
+
+# Suppress the license check on subsequent "visits". The first
+# visit will skip it anyway due to not having a compiler set up
+# yet. This cannot be added to the super cache, because that is
+# read before spec_pre.prf, which flushes CONFIG. This does not
+# affect submodules, as they come with a .qmake.conf. But that
+# one sets the flag via qt_build_config.prf anyway.
+!QTDIR_build: cache(CONFIG, add, $$list(QTDIR_build))
TEMPLATE = subdirs