Skip to content

add needed include for qdatastream#57

Open
whyrusleeping wants to merge 1 commit intowoodenshark:masterfrom
whyrusleeping:fix/qtapp-build
Open

add needed include for qdatastream#57
whyrusleeping wants to merge 1 commit intowoodenshark:masterfrom
whyrusleeping:fix/qtapp-build

Conversation

@whyrusleeping
Copy link

Just got my lightpack and the application failed to build due to a missing include (i think its an issue caused by a newer version of qt). This fixed the build for me, figured i'd share.

Error i was experiencing:

qtsingleapplication/src/qtlocalpeer.cpp: In member function ‘bool QtLocalPeer::sendMessage(const QString&, int)’:
qtsingleapplication/src/qtlocalpeer.cpp:160:19: error: variable ‘QDataStream ds’ has initializer but incomplete type
     QDataStream ds(&socket);

@anonymous09
Copy link

I had same error in my Gentoo Linux.

@wadeduvall
Copy link

I can confirm this error, and that the pull request fixes it. I'm on Archlinux.

@whyrusleeping
Copy link
Author

I'll email them, see if that helps any

@dreamer-dead
Copy link
Contributor

Third party qtsingleapplication should be updated to fix this error.
I got the latest version from https://github.com/qtproject/qt-solutions.git and then was able to compile the main project.

@e-yes e-yes mentioned this pull request Sep 25, 2016
@hashworks
Copy link

Same applies for QPainterPath:

diff --git a/Software/src/GrabConfigWidget.cpp b/Software/src/GrabConfigWidget.cpp
index 4cd77aa..b8fb703 100644
--- a/Software/src/GrabConfigWidget.cpp
+++ b/Software/src/GrabConfigWidget.cpp
@@ -28,6 +28,7 @@
 #include <QPainter>
 #include <QBitmap>
 #include <QDesktopWidget>
+#include <QPainterPath>
 #include "debug.h"
 
 const unsigned GrabConfigWidget::MarginArrow = 20;
diff --git a/Software/src/qtsingleapplication/src/qtlocalpeer.cpp b/Software/src/qtsingleapplication/src/qtlocalpeer.cpp
index 332b064..19914f4 100644
--- a/Software/src/qtsingleapplication/src/qtlocalpeer.cpp
+++ b/Software/src/qtsingleapplication/src/qtlocalpeer.cpp
@@ -42,6 +42,7 @@
 #include "qtlocalpeer.h"
 #include <QCoreApplication>
 #include <QTime>
+#include <QDataStream>
 
 #if defined(Q_OS_WIN)
 #include <QLibrary>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants