From 72107fc39f15e3827297530dfad3c63f8e4ffcaa Mon Sep 17 00:00:00 2001 From: Adrian Herrmann Date: Fri, 11 Nov 2022 20:31:37 +0100 Subject: Fix debug build problems on Win re: limited-api There is a build error on Windows when limited-api=yes and debug where the linker does not know the right paths to look for libraries. Supporting this configuration is not desirable anyway because there are no released Python builds on Windows with this configuration. Therefore: 1. Display a warning to the user that the combination debug + limited=api=yes does not really "exist" if they choose it explicitly on Windows. 2. Have the default for debug builds be limited-api=no on Windows. Pick-to: 6.4 Change-Id: Ifed99f80abb02c9d512a1321526e949d56ff08a6 Reviewed-by: Friedemann Kleint Reviewed-by: Shyamnath Premnadh Reviewed-by: Cristian Maureira-Fredes --- build_scripts/main.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'build_scripts/main.py') diff --git a/build_scripts/main.py b/build_scripts/main.py index 8c24ceb75..f1a151caa 100644 --- a/build_scripts/main.py +++ b/build_scripts/main.py @@ -683,10 +683,12 @@ class PysideBuild(_build, CommandMixin, BuildInfoCollectorMixin): elif OPTION["LIMITED_API"] == "no": cmake_cmd.append("-DFORCE_LIMITED_API=no") elif not OPTION["LIMITED_API"]: - pass + if sys.platform == 'win32' and self.debug: + cmake_cmd.append("-DFORCE_LIMITED_API=no") else: raise SetupError("option limited-api must be 'yes' or 'no' " - "(default yes if applicable, i.e. python version >= 3.7)") + "(default yes if applicable, i.e. Python " + "version >= 3.7 and release build if on Windows)") if OPTION["VERBOSE_BUILD"]: cmake_cmd.append("-DCMAKE_VERBOSE_MAKEFILE:BOOL=ON") -- cgit v1.2.3