Vadim Zhukov <persg...@gmail.com> wrote: > > This minor release, under the guise of adding support for Qt5, now > > casually requires Qt5 as the minimum version. Since we don't have > > a Qt5 port, this means dropping the Qt client. If somebody has a > > strong opinion to the contrary and a better suggestion, speak up. > > This is an improved version of your patch, that allows to build with > Qt4. Actual Qt4 support needed only two small patches.
Great! > I fixed WANTLIB handling and RUN_DEPENDS handling while there, too. What are you trying to fix here? The extra libs reported by lib-depends-check for the -qt subpackage? I can assure you that transmission-qt pulls in these libraries. It's just that they are reached indirectly, e.g., transmission-qt links against libcurl, which links against libidn. If you want to fix the extras, I'd rather link the libraries explicitly in qtr.pro. > -MODULES= devel/gettext textproc/intltool > +MODULES= devel/gettext textproc/intltool x11/qt4 No, we don't want a build dependency on Qt4 despite FLAVOR=no_qt. > -RUN_DEPENDS-gtk=${PKGNAME-main}:net/transmission,-main \ > +RUN_DEPENDS-gtk=${BUILD_PKGPATH} \ No, we don't want the PSEUDO_FLAVORS to make their way into the dependency. New try: Index: Makefile =================================================================== RCS file: /cvs/ports/net/transmission/Makefile,v retrieving revision 1.89 diff -u -p -r1.89 Makefile --- Makefile 9 Aug 2013 19:13:08 -0000 1.89 +++ Makefile 12 Aug 2013 14:22:08 -0000 @@ -4,7 +4,7 @@ COMMENT-main= BitTorrent command line an COMMENT-gtk= BitTorrent client with GTK+ interface COMMENT-qt= BitTorrent client with Qt interface -VER= 2.81 +VER= 2.82 DISTNAME= transmission-${VER} PKGNAME-main= transmission-${VER} PKGNAME-gtk= transmission-gtk-${VER} @@ -44,7 +44,7 @@ WANTLIB-gtk= ${WANTLIB} ICE SM X11 Xcomp xcb-render xcb-shm WANTLIB-qt= ${WANTLIB} ICE SM QtCore QtDBus QtGui QtNetwork QtXml \ X11 Xext Xi Xinerama Xrender \ - expat fontconfig freetype stdc++ + fontconfig freetype stdc++ MODULES= devel/gettext textproc/intltool Index: distinfo =================================================================== RCS file: /cvs/ports/net/transmission/distinfo,v retrieving revision 1.46 diff -u -p -r1.46 distinfo --- distinfo 9 Aug 2013 19:13:08 -0000 1.46 +++ distinfo 12 Aug 2013 14:22:08 -0000 @@ -1,2 +1,2 @@ -SHA256 (transmission-2.81.tar.xz) = u7Vx9YJUZqpF5u9K0oYXYTWhqcAjT/6jUn69gAHMVqs= -SIZE (transmission-2.81.tar.xz) = 3167816 +SHA256 (transmission-2.82.tar.xz) = OZZlEIffZ6hfHhtKkrG1GN3v3YTGVLjfb7zLC5HwNSI= +SIZE (transmission-2.82.tar.xz) = 3172024 Index: patches/patch-libtransmission_platform-quota_c =================================================================== RCS file: patches/patch-libtransmission_platform-quota_c diff -N patches/patch-libtransmission_platform-quota_c --- patches/patch-libtransmission_platform-quota_c 9 Aug 2013 19:13:08 -0000 1.1 +++ /dev/null 1 Jan 1970 00:00:00 -0000 @@ -1,30 +0,0 @@ -$OpenBSD: patch-libtransmission_platform-quota_c,v 1.1 2013/08/09 19:13:08 naddy Exp $ ---- libtransmission/platform-quota.c.orig Tue Jul 23 13:37:31 2013 -+++ libtransmission/platform-quota.c Tue Jul 23 13:39:38 2013 -@@ -18,7 +18,7 @@ - - #ifndef WIN32 - #include <sys/types.h> /* types needed by quota.h */ -- #ifdef __FreeBSD__ -+ #if defined(__FreeBSD__) || defined(__OpenBSD__) - #include <ufs/ufs/quota.h> /* quotactl() */ - #elif defined (__sun) - #include <sys/fs/ufs_quota.h> /* quotactl */ -@@ -203,7 +203,7 @@ getquota (const char * device) - int64_t freespace; - int64_t spaceused; - --#if defined(__FreeBSD__) || defined(SYS_DARWIN) -+#if defined(__FreeBSD__) || defined(__OpenBSD__) || defined(SYS_DARWIN) - if (quotactl(device, QCMD(Q_GETQUOTA, USRQUOTA), getuid(), (caddr_t) &dq) == 0) - { - #elif defined(__sun) -@@ -235,7 +235,7 @@ getquota (const char * device) - /* No quota enabled for this user */ - return -1; - } --#if defined(__FreeBSD__) -+#if defined(__FreeBSD__) || defined(__OpenBSD__) - spaceused = (int64_t) dq.dqb_curblocks >> 1; - #elif defined(SYS_DARWIN) - spaceused = (int64_t) dq.dqb_curbytes; Index: patches/patch-qt_favicon_cc =================================================================== RCS file: patches/patch-qt_favicon_cc diff -N patches/patch-qt_favicon_cc --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ patches/patch-qt_favicon_cc 12 Aug 2013 14:22:08 -0000 @@ -0,0 +1,28 @@ +$OpenBSD$ +--- qt/favicon.cc.orig Mon Aug 12 13:27:14 2013 ++++ qt/favicon.cc Mon Aug 12 13:28:49 2013 +@@ -14,7 +14,12 @@ + #include <QNetworkAccessManager> + #include <QNetworkReply> + #include <QNetworkRequest> ++ ++#if QT_VERSION >= 0x050000 + #include <QStandardPaths> ++#else ++#include <QDesktopServices> ++#endif + + #include "favicon.h" + +@@ -40,7 +45,11 @@ Favicons :: ~Favicons( ) + QString + Favicons :: getCacheDir( ) + { ++#if QT_VERSION >= 0x050000 + const QString base = QStandardPaths::writableLocation (QStandardPaths::CacheLocation); ++#else ++ const QString base = QDesktopServices::storageLocation( QDesktopServices::CacheLocation ); ++#endif + return QDir( base ).absoluteFilePath( "favicons" ); + } + Index: patches/patch-qt_file-tree_cc =================================================================== RCS file: patches/patch-qt_file-tree_cc diff -N patches/patch-qt_file-tree_cc --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ patches/patch-qt_file-tree_cc 12 Aug 2013 14:22:08 -0000 @@ -0,0 +1,15 @@ +$OpenBSD$ +--- qt/file-tree.cc.orig Mon Aug 12 13:29:51 2013 ++++ qt/file-tree.cc Mon Aug 12 13:30:33 2013 +@@ -885,7 +885,11 @@ FileTreeView :: FileTreeView (QWidget * parent, bool i + for (int i=0; i<NUM_COLUMNS; ++i) + { + setColumnHidden (i, (i<FIRST_VISIBLE_COLUMN) || (LAST_VISIBLE_COLUMN<i)); ++#if QT_VERSION >= 0x050000 + header()->setSectionResizeMode(i, QHeaderView::Interactive); ++#else ++ header()->setResizeMode(i, QHeaderView::Interactive); ++#endif + } + + connect (this, SIGNAL(clicked(const QModelIndex&)), Index: patches/patch-qt_qtr_pro =================================================================== RCS file: /cvs/ports/net/transmission/patches/patch-qt_qtr_pro,v retrieving revision 1.4 diff -u -p -r1.4 patch-qt_qtr_pro --- patches/patch-qt_qtr_pro 7 Apr 2012 10:55:04 -0000 1.4 +++ patches/patch-qt_qtr_pro 12 Aug 2013 14:22:08 -0000 @@ -1,6 +1,6 @@ $OpenBSD: patch-qt_qtr_pro,v 1.4 2012/04/07 10:55:04 naddy Exp $ ---- qt/qtr.pro.orig Thu Mar 29 17:30:57 2012 -+++ qt/qtr.pro Thu Mar 29 17:31:22 2012 +--- qt/qtr.pro.orig Fri Aug 9 04:45:47 2013 ++++ qt/qtr.pro Mon Aug 12 16:14:03 2013 @@ -8,7 +8,7 @@ target.path = /bin INSTALLS += target @@ -9,13 +9,13 @@ $OpenBSD: patch-qt_qtr_pro,v 1.4 2012/04 +man.path = /man/man1/ man.files = transmission-qt.1 - CONFIG += qt qdbus thread debug link_pkgconfig + CONFIG += qt thread debug link_pkgconfig @@ -26,7 +26,7 @@ LIBS += $${LIBUTP_LIBS} LIBS += $${TRANSMISSION_TOP}/third-party/dht/libdht.a LIBS += $${LIBUPNP_LIBS} LIBS += $${LIBNATPMP_LIBS} -unix: LIBS += -L$${EVENT_TOP}/lib -lz -lrt -+unix: LIBS += -L$${EVENT_TOP}/lib -lz -levent_core -levent_extra ++unix: LIBS += -L$${EVENT_TOP}/lib -lidn -liconv -lz -levent_core -levent_extra win32:DEFINES += QT_DBUS win32:LIBS += -levent-2.0 -lws2_32 -lintl win32:LIBS += -lidn -liconv -lwldap32 -liphlpapi -- Christian "naddy" Weisgerber na...@mips.inka.de