On Thu, Apr 28, 2022 at 05:37:20PM +0200, Rafael Sadowski wrote:
> Based on the idea and the feedback from kn@ and tb@. Here is an patch to
> disable QML just-in-time compiler support in OpenBSD. Is that something
> we want?

This works for me in neochat and kdeconnect with sysctl.knwxabort=1
where the first attempt was adding WX_NEEDED which developed into
disabling he W^X probe in qtdeclarative.

Your patch has the same effect and seems more appropiate, although I
can't speak for the whole of Qt and its QML JIT compiler.

OK kn

> diff --git a/x11/qt5/qtdeclarative/Makefile b/x11/qt5/qtdeclarative/Makefile
> index bec5deb4701..9f3c62c9761 100644
> --- a/x11/qt5/qtdeclarative/Makefile
> +++ b/x11/qt5/qtdeclarative/Makefile
> @@ -1,8 +1,6 @@
> -# qmlplugindump
> -USE_WXNEEDED =               Yes
> -
>  QT5NAME =            QtDeclarative
>  COMMENT-main =               Qt5 Quick and QML components
> +REVISION =           0
>  
>  MULTI_PACKAGES =     -main -examples
>  DPB_PROPERTIES =     parallel
> diff --git a/x11/qt5/qtdeclarative/patches/patch-src_qml_configure_json 
> b/x11/qt5/qtdeclarative/patches/patch-src_qml_configure_json
> new file mode 100644
> index 00000000000..2bbe4a9791b
> --- /dev/null
> +++ b/x11/qt5/qtdeclarative/patches/patch-src_qml_configure_json
> @@ -0,0 +1,12 @@
> +Index: src/qml/configure.json
> +--- src/qml/configure.json.orig
> ++++ src/qml/configure.json
> +@@ -103,7 +103,7 @@
> +                      && (config.linux || config.ios || config.tvos || 
> config.qnx || config.integrity))"
> +             ],
> +             "output": [ "privateFeature" ],
> +-            "autoDetect": "!config.ios && !config.tvos",
> ++            "autoDetect": "!config.ios && !config.tvos && !config.openbsd",
> +             "comment": "On arm and arm64 we need a specialization of 
> cacheFlush() for each OS to be
> +                         enabeled. Therefore the config white list.
> +                         Also Mind that e.g. x86_32 has arch.x86_64 but 
> 32bit pointers. Therefore
> 

Reply via email to