psi-plus / main

Main repository with patches and required resources
https://psi-plus.com/
GNU Lesser General Public License v2.1
68 stars 20 forks source link

Building on Parabola without WebKit not working #781

Closed jschwartzenberg closed 4 years ago

jschwartzenberg commented 4 years ago

Hey all, it seems there's an issue when building without WebKit.

[ 60%] Linking CXX executable ../psi/psi-plus
/usr/bin/ld: CMakeFiles/psi-plus.dir/chatview_te.cpp.o: in function `ChatView::renderMucMessage(MessageView const&, QTextCursor&)':
/usr/src/debug/psi/src/chatview_te.cpp:404: undefined reference to `ChatViewCommon::getMucNickColor(QString const&, bool)'
/usr/bin/ld: CMakeFiles/psi-plus.dir/chatview_te.cpp.o: in function `ChatView::ChatView(QWidget*)':
/usr/src/debug/psi/src/chatview_te.cpp:63: undefined reference to `ChatViewCommon::setLooks(QWidget*)'
/usr/bin/ld: CMakeFiles/psi-plus.dir/chatview_te.cpp.o: in function `ChatView::dispatchMessage(MessageView const&)':
/usr/src/debug/psi/src/chatview_te.cpp:283: undefined reference to `ChatViewCommon::updateLastMsgTime(QDateTime)'
collect2: fout: ld gaf exit-status 1 terug
make[2]: *** [src/CMakeFiles/psi-plus.dir/build.make:3658: psi/psi-plus] Fout 1
make[1]: *** [CMakeFiles/Makefile2:816: src/CMakeFiles/psi-plus.dir/all] Fout 2
make: *** [Makefile:149: all] Fout 2
==> FOUT: Er is een fout opgetreden in build().
    Afbreken...

This is with cmake -B build -S psi -DCMAKE_INSTALL_PREFIX=/usr -DCMAKE_BUILD_TYPE=Release -DPSI_PLUS=ON -DENABLE_WEBKIT=OFF .. with a slightly modified PKGBUILD from AUR.

Ri0n commented 4 years ago

I just checked the code. Looks like something impossible. chatviewcommon is included to the compilation unconditionally. So its symbols must be always available.

jschwartzenberg commented 4 years ago

The clean build is still running here. I have to leave soon, but should have the results when I'm back in about 1,5 hour.

jschwartzenberg commented 4 years ago

Seems there was a glitch in the matrix!