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

Unable to compile anymore Kiwix-Desktop #615

Closed
kelson42 opened this issue Mar 19, 2021 · 3 comments · Fixed by #616
Closed

Unable to compile anymore Kiwix-Desktop #615

kelson42 opened this issue Mar 19, 2021 · 3 comments · Fixed by #616
Assignees
Labels

Comments

@kelson42
Copy link
Collaborator

kelson42 commented Mar 19, 2021

$ make 
g++ -c -pipe -std=c++11 -I/usr/local/include/ -I/usr/local/include -I/usr/include/x86_64-linux-gnu -O2 -Wall -W -D_REENTRANT -fPIC -DVERSION=2.0.5 -DQT_DEPRECATED_WARNINGS -DQT_NO_DEBUG -DQT_WEBENGINEWIDGETS_LIB -DQT_WEBENGINECORE_LIB -DQT_QUICK_LIB -DQT_PRINTSUPPORT_LIB -DQT_WIDGETS_LIB -DQT_GUI_LIB -DQT_WEBCHANNEL_LIB -DQT_QML_LIB -DQT_NETWORK_LIB -DQT_POSITIONING_LIB -DQT_CORE_LIB -I. -Isubprojects/QtSingleApplication/src -isystem /usr/include/x86_64-linux-gnu/qt5 -isystem /usr/include/x86_64-linux-gnu/qt5/QtWebEngineWidgets -isystem /usr/include/x86_64-linux-gnu/qt5/QtWebEngineCore -isystem /usr/include/x86_64-linux-gnu/qt5/QtQuick -isystem /usr/include/x86_64-linux-gnu/qt5/QtPrintSupport -isystem /usr/include/x86_64-linux-gnu/qt5/QtWidgets -isystem /usr/include/x86_64-linux-gnu/qt5/QtGui -isystem /usr/include/x86_64-linux-gnu/qt5/QtWebChannel -isystem /usr/include/x86_64-linux-gnu/qt5/QtQml -isystem /usr/include/x86_64-linux-gnu/qt5/QtNetwork -isystem /usr/include/x86_64-linux-gnu/qt5/QtPositioning -isystem /usr/include/x86_64-linux-gnu/qt5/QtCore -I. -I. -I/usr/lib/x86_64-linux-gnu/qt5/mkspecs/linux-g++ -o contenttypefilter.o src/contenttypefilter.cpp
In file included from /usr/local/include/kiwix/library.h:31,
                 from src/library.h:5,
                 from src/kiwixapp.h:4,
                 from src/contenttypefilter.cpp:2:
/usr/local/include/xapian.h:31:4: error: #error Include <xapian.h> before Qt headers, or put 'CONFIG += no_keywords' in your .pro file and use Q_SLOTS instead of slots, etc
   31 | #  error Include <xapian.h> before Qt headers, or put 'CONFIG += no_keywords' in your .pro file and use Q_SLOTS instead of slots, etc
      |    ^~~~~
In file included from /usr/local/include/xapian.h:76,
                 from /usr/local/include/kiwix/library.h:31,
                 from src/library.h:5,
                 from src/kiwixapp.h:4,
                 from src/contenttypefilter.cpp:2:
/usr/local/include/xapian/keymaker.h:114:10: error: a template-id may not appear in a using-declaration
  114 |     std::vector<KeySpec> slots;
      |          ^~~~~~~~~~~~~~~
/usr/local/include/xapian/keymaker.h: In member function ‘void Xapian::MultiValueKeyMaker::add_value(Xapian::valueno, bool, const string&)’:
/usr/local/include/xapian/keymaker.h:145:7: error: expected primary-expression before ‘.’ token
  145 |  slots.push_back(KeySpec(slot, reverse, defvalue));
      |       ^
make: *** [Makefile:1004: contenttypefilter.o] Error 1

With latest libkiwix

@kelson42 kelson42 added the bug label Mar 19, 2021
@kelson42 kelson42 self-assigned this Mar 19, 2021
@mgautierfr
Copy link
Member

It is a bug in kiwix-lib, not kiwix-desktop.
Last PR (kiwix/libkiwix#460) add "xapian.h" in the public lib-kiwix's includes and we should not (as xapian is a private/internal dependency).

@kelson42
Copy link
Collaborator Author

kelson42 commented Mar 19, 2021

@mgautierfr OK and we should fix things following your recommendation #616 (comment) and this probably means rollbacking my commit. At least now, Kiwix Desktop compilation is not totally broken anymore.

@kelson42 kelson42 assigned mgautierfr and unassigned kelson42 Mar 19, 2021
@kelson42
Copy link
Collaborator Author

kelson42 commented Apr 8, 2021

Was a problem with libkiwix

@kelson42 kelson42 closed this as completed Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants