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

Reply via email to