Index | Thread | Search

From:
Rafael Sadowski <rafael@sizeofvoid.org>
Subject:
Re: UPDATE: devel/zeal
To:
ports <ports@openbsd.org>, Adam Wolk <awolk@openbsd.org>
Cc:
Klemens Nanni <kn@openbsd.org>
Date:
Wed, 7 Feb 2024 21:10:26 +0100

Download raw body.

Thread
On Sun Feb 04, 2024 at 01:20:18PM +0000, Klemens Nanni wrote:
> On Sat, Feb 03, 2024 at 08:18:31AM +0100, Rafael Sadowski wrote:
> > Update zeal to 0.7.0. Upstream switched to Qt6 and Qt6Webengine.
> > 
> > Could someone test it with the current Qt6 version, I'm sitting on a new
> > version that needs a little more time.
> 
> I removed USE_WXNEEDED=Yes and replaced your five new patches with the
> single, identical upstream commit in PATCHFILES;  that took an extra
> post-extract to basically get FIX_CRLF_FILES before dispatch, see diff...
> 
> /usr/ports/pobj/zeal-0.7.0/zeal-0.7.0/src/app/main.cpp:180:5: error: 'critical' is deprecated: Use the overload taking StandardButtons instead. [-Werror,-Wdeprecated-declarations]
>     QT_REQUIRE_VERSION(argc, argv, QT_VERSION_STR)
>     ^
> /usr/local/include/X11/qt6/QtWidgets/qmessagebox.h:315:58: note: expanded from macro 'QT_REQUIRE_VERSION'
> str)).arg(QString::fromLatin1(qVersion())); QMessageBox::critical(0, QApplication::tr(\
>                                                          ^
> /usr/local/include/X11/qt6/QtWidgets/qmessagebox.h:242:5: note: 'critical' has been explicitly marked deprecated here
>     QT_DEPRECATED_VERSION_X_6_2("Use the overload taking StandardButtons instead.")
>     ^
> /usr/local/include/X11/qt6/QtCore/qtdeprecationmarkers.h:141:44: note: expanded from macro 'QT_DEPRECATED_VERSION_X_6_2'
> # define QT_DEPRECATED_VERSION_X_6_2(text) QT_DEPRECATED_X(text)
>                                            ^
> /usr/local/include/X11/qt6/QtCore/qtdeprecationmarkers.h:27:33: note: expanded from macro 'QT_DEPRECATED_X'
> #  define QT_DEPRECATED_X(text) Q_DECL_DEPRECATED_X(text)
>                                 ^
> /usr/local/include/X11/qt6/QtCore/qcompilerdetection.h:971:36: note: expanded from macro 'Q_DECL_DEPRECATED_X'
> #  define Q_DECL_DEPRECATED_X(x) [[deprecated(x)]]
>                                    ^
> 1 error generated.

I tested my diff with Qt 6.6.1. When packages are present could you run
a second try, please. zeal was my test case for qtwebengine.


> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/zeal/Makefile,v
> diff -u -p -r1.20 Makefile
> --- Makefile	21 Sep 2023 09:50:08 -0000	1.20
> +++ Makefile	4 Feb 2024 13:11:34 -0000
> @@ -1,13 +1,14 @@
> -# qt5 webkit
> -USE_WXNEEDED =	Yes
> -
>  USE_NOBTCFI =	Yes
>  
>  COMMENT =	simple offline API documentation browser
>  
> -V =	0.6.1
> -DISTNAME = zeal-${V}
> -REVISION = 3
> +V =		0.7.0
> +DISTNAME =	zeal-${V}
> +
> +# fix build with Qt6
> +SITES.qt =		https://github.com/zealdocs/zeal/commit/
> +PATCHFILES.qt =		9630cc94c155d87295e51b41fbab2bd5798f8229.patch
> +PATCH_DIST_STRIP =	-p1
>  
>  CATEGORIES =	devel
>  
> @@ -18,24 +19,44 @@ MAINTAINER =	Adam Wolk <awolk@openbsd.or
>  # GPLv3
>  PERMIT_PACKAGE =	Yes
>  
> -WANTLIB += ${COMPILER_LIBCXX} ICE Qt5Concurrent Qt5Core Qt5Gui
> -WANTLIB += Qt5Network Qt5WebKit Qt5WebKitWidgets Qt5Widgets Qt5X11Extras
> -WANTLIB += SM X11 Xext archive c m sqlite3 xcb xcb-keysyms
> +WANTLIB += ${COMPILER_LIBCXX} GL ICE Qt6Concurrent Qt6Core Qt6Gui
> +WANTLIB += Qt6Network Qt6OpenGL Qt6PrintSupport Qt6Qml Qt6QmlModels
> +WANTLIB += Qt6Quick Qt6WebChannel Qt6WebEngineCore Qt6WebEngineWidgets
> +WANTLIB += Qt6Widgets SM X11 Xext archive c m sqlite3 xcb xcb-keysyms
> +WANTLIB += xkbcommon
>  
>  SITES =		https://github.com/zealdocs/zeal/releases/download/v${V}/
>  
>  EXTRACT_SUFX =	.tar.xz
>  
> -MODULES =	devel/kf5
> +MODULES =	x11/qt6 devel/cmake
> +
> +BUILD_DEPENDS =	devel/kf5/extra-cmake-modules
>  
> -RUN_DEPENDS +=	devel/desktop-file-utils \
> +RUN_DEPENDS =	devel/desktop-file-utils \
>  		x11/gtk+4,-guic
>  
> -LIB_DEPENDS +=	archivers/libarchive \
> +LIB_DEPENDS =	archivers/libarchive \
>  		databases/sqlite3 \
> -		x11/qt5/qtwebkit \
> -		x11/qt5/qtx11extras
> +		x11/qt6/qtdeclarative \
> +		x11/qt6/qtwebengine
> +
> +CONFIGURE_ARGS =	-DZEAL_RELEASE_BUILD=ON
> +
> +FIX_CRLF_FILES =	src/libs/registry/*.cpp \
> +			src/libs/ui/qxtglobalshortcut/*.cpp \
> +			src/libs/ui/*.cpp
>  
>  NO_TEST =	Yes
> +
> +# XXX FIX_CRLF_FILES runs after distpatch, but patch(1) and gpatch(1) won't
> +# apply PATCHFILES due to wrong line endings
> +post-extract:
> +	@cd ${WRKDIST} && perl -i -pe 's/\r$$//' -- \
> +	    src/libs/registry/docset.cpp \
> +	    src/libs/registry/docsetmetadata.cpp \
> +	    src/libs/registry/docsetregistry.cpp \
> +	    src/libs/ui/docsetsdialog.cpp \
> +	    src/libs/ui/qxtglobalshortcut/qxtglobalshortcut_x11.cpp
>  
>  .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/devel/zeal/distinfo,v
> diff -u -p -r1.7 distinfo
> --- distinfo	5 Jan 2019 10:10:32 -0000	1.7
> +++ distinfo	4 Feb 2024 12:57:00 -0000
> @@ -1,2 +1,4 @@
> -SHA256 (zeal-0.6.1.tar.xz) = +7knkjPSuCRbSd4CKK4NBEWEqK50z9UjnBbVCIsfx4s=
> -SIZE (zeal-0.6.1.tar.xz) = 671432
> +SHA256 (9630cc94c155d87295e51b41fbab2bd5798f8229.patch) = 5AzvKb1jzWiDHwtLJ0KciN0ySarfALJVnMMmuz2urmE=
> +SHA256 (zeal-0.7.0.tar.xz) = Qs7+iydnvZTly3ANpKN9mM+xhp3W8EOxD3cx3zGcr0w=
> +SIZE (9630cc94c155d87295e51b41fbab2bd5798f8229.patch) = 4768
> +SIZE (zeal-0.7.0.tar.xz) = 784204
> Index: patches/patch-src_libs_ui_searchitemdelegate_cpp
> ===================================================================
> RCS file: patches/patch-src_libs_ui_searchitemdelegate_cpp
> diff -N patches/patch-src_libs_ui_searchitemdelegate_cpp
> --- patches/patch-src_libs_ui_searchitemdelegate_cpp	11 Mar 2022 18:53:39 -0000	1.2
> +++ /dev/null	1 Jan 1970 00:00:00 -0000
> @@ -1,11 +0,0 @@
> -Index: src/libs/ui/searchitemdelegate.cpp
> ---- src/libs/ui/searchitemdelegate.cpp.orig
> -+++ src/libs/ui/searchitemdelegate.cpp
> -@@ -27,6 +27,7 @@
> - #include <QFontMetrics>
> - #include <QHelpEvent>
> - #include <QPainter>
> -+#include <QPainterPath>
> - #include <QToolTip>
> - 
> - using namespace Zeal::WidgetUi;
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/devel/zeal/pkg/PLIST,v
> diff -u -p -r1.5 PLIST
> --- pkg/PLIST	11 Mar 2022 18:53:40 -0000	1.5
> +++ pkg/PLIST	4 Feb 2024 12:54:32 -0000
> @@ -1,5 +1,5 @@
>  @bin bin/zeal
> -share/applications/org.zealdocs.Zeal.desktop
> +share/applications/org.zealdocs.zeal.desktop
>  share/icons/hicolor/128x128/apps/zeal.png
>  share/icons/hicolor/16x16/apps/zeal.png
>  share/icons/hicolor/24x24/apps/zeal.png
> @@ -8,4 +8,4 @@ share/icons/hicolor/64x64/apps/zeal.png
>  @tag update-desktop-database
>  @tag gtk-update-icon-cache %D/share/icons/hicolor
>  share/metainfo/
> -share/metainfo/org.zealdocs.Zeal.appdata.xml
> +share/metainfo/org.zealdocs.zeal.appdata.xml
>