git: 089da4a7a764 - main - www/qt6-webengine: Really disable PCH

From: Jason E. Hale <jhale_at_FreeBSD.org>
Date: Mon, 06 Nov 2023 20:50:01 UTC
The branch main has been updated by jhale:

URL: https://cgit.FreeBSD.org/ports/commit/?id=089da4a7a764183a8d63d22dedc12537f90b50db

commit 089da4a7a764183a8d63d22dedc12537f90b50db
Author:     Jason E. Hale <jhale@FreeBSD.org>
AuthorDate: 2023-11-06 19:53:12 +0000
Commit:     Jason E. Hale <jhale@FreeBSD.org>
CommitDate: 2023-11-06 20:49:55 +0000

    www/qt6-webengine: Really disable PCH
    
    Due to the way qt_evaluate_config_expression()[1] parses conditions,
    De Morgan's Laws do not apply. This function is internal to the Qt
    build process and this behavior may be by design for simplicity.
    NOT (A OR B) defies logic and always evaluates to NOT A OR B, so we
    must express this as NOT A AND NOT B instead to get the desired result.
    
    Disabling PCH (pre-compiled headers) greatly decreases build time,
    especially with ccache and should fix the build on arm64 (not tested).
    
    ../../../../../qtwebengine-everywhere-src-6.5.3/src/3rdparty/chromium/base/strings/string_util.h:51:3:
    error: non-const lvalue reference to type '__builtin_va_list' cannot bind to a value of unrelated type 'va_list' (aka 'std::__va_list')
      va_start(arguments, format);
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~
    /usr/include/sys/_stdarg.h:45:49: note: expanded from macro 'va_start'
      #define       va_start(ap, last)      __builtin_va_start((ap), (last))
                                                               ^~~~
    
    [1] https://code.qt.io/cgit/qt/qtbase.git/tree/cmake/QtFeature.cmake?h=6.5.3#n84
---
 www/qt6-webengine/files/patch-cmake_Functions.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/qt6-webengine/files/patch-cmake_Functions.cmake b/www/qt6-webengine/files/patch-cmake_Functions.cmake
index 2ebeb4684add..919c24219f0b 100644
--- a/www/qt6-webengine/files/patch-cmake_Functions.cmake
+++ b/www/qt6-webengine/files/patch-cmake_Functions.cmake
@@ -23,7 +23,7 @@
      extend_gn_list(gnArgArg
          ARGS enable_precompiled_headers
 -        CONDITION BUILD_WITH_PCH AND NOT LINUX
-+	CONDITION BUILD_WITH_PCH AND NOT (LINUX OR FREEBSD)
++        CONDITION BUILD_WITH_PCH AND NOT LINUX AND NOT FREEBSD
      )
      extend_gn_list(gnArgArg
          ARGS dcheck_always_on