Bug 485677

I am also having trouble building qbittorrent since the change to qt 4.5

https://bugs.launchpad.net/qbittorrent/+bug/332470

upstream seems to think it's a qt bug

That is a completely separate issue. It should be in a separate bug report.

But I don't think your issue is a bug in Qt. Something must be "#define"ing "subnet" as a macro, which is a bad thing to do.

It's these 2 lines in qhostaddress.h (lines 122, 124):

    bool isInSubnet(const QPair<QHostAddress, int> &subnet) const;

    static QPair<QHostAddress, int> parseSubnet(const QString &subnet);

The problem seems to be the <QHostAddress, int> contructs by the looks of things.

A quick-n-dirty hack of commenting out those lines in qhostaddress.h allows qbittorrent to compile.


On the surface, those lines look wrong to me... a (sort of) recursive template?  but I'm far from being a master of c++-fu.

Heh, looks like QPair is undefined, adding
#include <QPair>
to /usr/include/QtNetwork/qhostaddress.h
does the trick.

Leigh, looks like adding to httpserver.cpp:
#include <QPair>
would be a workaround until we can get this fixed in qt properly.

This report was originally about the scidavis failure before the qbittorrent maintainer hijacked it for his completely different bug!

I just cloned this bug as bug 486866 for the original issue.

But next time please:
* do not hijack bug reports for unrelated issues!
* do not answer such hijackers (other than mentioning it's off topic as I did in comment #2)!
* clone the bug if you want to track it (but the reporter should be responsible for filing their own bug report)! (And I probably should have done that when I posted my comment #2.)
and especially:
* don't change the topic of the existing bug report to match the hijack!

sorry, comment #1 didn't mention which app was at issue (for posterity:  reporter, please do so ; commenters (including me), please double-check, and make summary clear).

Created attachment 332872 [details]
workaround undefined QPair

(In reply to comment #6)
> This report was originally about the scidavis failure before the qbittorrent
> maintainer hijacked it for his completely different bug!



(In reply to comment #6)
> This report was originally about the scidavis failure before the qbittorrent
> maintainer hijacked it for his completely different bug!


Sorry if I hijacked it for a completely different bug, but I didn't know as both koji links are broken, and were when I posted.

The bug summary matched my problem so I posted it here.

(In reply to comment #9)
> Created an attachment (id=332872) [details]
> workaround undefined QPair


> Heh, looks like QPair is undefined, adding
>#include <QPair>
>to /usr/include/QtNetwork/qhostaddress.h
>does the trick


Thanks Rex, this works a treat.

>Leigh, looks like adding to httpserver.cpp:
>#include <QPair>
>would be a workaround until we can get this fixed in qt properly.

This didn't work, I still got the same error.

Did you add it BEFORE the #include which drags in qhostaddress.h? The order matters.

(In reply to comment #12)
> Did you add it BEFORE the #include which drags in qhostaddress.h? The order
> matters.

Yes I think so


--- qbittorrent-1.3.1/src/httpserver.cpp.orig 
+++ qbittorrent-1.3.1/src/httpserver.cpp 
@@ -18,7 +18,7 @@
  *   59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  */
 
-
+#include <QPair>
 #include "httpserver.h"
 #include "httpconnection.h"
 #include "eventmanager.h"

Well I got there :) , I was patching the wrong file.


--- qbittorrent-1.3.1/src/httpserver.h.orig 
+++ qbittorrent-1.3.1/src/httpserver.h 
@@ -22,6 +22,7 @@
 #ifndef HTTPSERVER_H
 #define HTTPSERVER_H
 
+#include <QPair>
 #include <QTcpServer>
 #include <QByteArray>

Well I finally got it to build @ Koji

http://koji.fedoraproject.org/koji/buildinfo?buildID=83808

I find it strange that I had to add a new build requires (glib2-devel)

Should be fixed in qt-4.5.0-0.4.rc1 and newer