Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Octopi cannot be build (probably on Qt5.11) #327

Closed
pavbaranov opened this issue May 2, 2018 · 4 comments
Closed

Octopi cannot be build (probably on Qt5.11) #327

pavbaranov opened this issue May 2, 2018 · 4 comments

Comments

@pavbaranov
Copy link

I'm Qt5.11beta4 user, so maybe it's something with this. Octopi cannot be build. Relevant part of "build" is:
src/transactiondialog.cpp: In member function 'void TransactionDialog::setDetailedText(QString)': src/transactiondialog.cpp:78:63: error: invalid use of incomplete type 'class QRegularExpression' if (detailedtext.contains(QRegularExpression("pacman-[0-9]+"))) ^ In file included from /usr/include/qt/QtCore/qhashfunctions.h:44:0, from /usr/include/qt/QtCore/qlist.h:47, from /usr/include/qt/QtCore/qvariant.h:45, from /usr/include/qt/QtCore/QVariant:1, from build/ui_transactiondialog.h:12, from src/transactiondialog.h:23, from src/transactiondialog.cpp:20: /usr/include/qt/QtCore/qstring.h:83:7: note: forward declaration of 'class QRegularExpression' class QRegularExpression; ^~~~~~~~~~~~~~~~~~ make: *** [Makefile:1827: build/transactiondialog.o] Error 1 make: *** Waiting for unfinished jobs.... ==> ERROR: A failure occurred in build().
Yes, I know, that there isn't Qt5.11 stable version so now, but maybe this issue is useful for upcoming version of octopi.

@HeinzDo
Copy link

HeinzDo commented May 27, 2018

Yes, the same here with Qt5.11. release.

@demmm
Copy link
Contributor

demmm commented May 28, 2018

This issue is getting slightly more urgent, since pacman 5.1.0 is now out, which shipped with libalpm.so.11, up from libalpm.so.10, so Octopi is broken on systems that update to new pacman, but can't be rebuild due to incompatibility with Qt 5.11.
KaOS will keep pacman at 5.0.2 until Octopi is compatible with Qt 5.11.

@demmm
Copy link
Contributor

demmm commented May 28, 2018

Fix is very simple, just needed to add one new include:

--- a/src/transactiondialog.cpp	2018-05-07 00:55:25.000000000 +0200
+++ b/src/transactiondialog.cpp	2018-05-28 19:18:28.492578121 +0200
@@ -26,6 +26,7 @@
 #include <QPushButton>
 #include <QDialog>
 #include <QCloseEvent>
+#include <QRegularExpression>
 
 /*
  * This is the dialog used to show the transaction summary

Patch used:
https://github.com/KaOSx/apps/blob/master/octopi/transactiondialog.diff

@aarnt
Copy link
Owner

aarnt commented May 29, 2018

Thank you all !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants