Various undefined reference problem in the test-build qevercloud package

Question

Various undefined reference problem in the test-build qevercloud package

hosiet opened this issue 8 years ago · comments

Hello,

I made the test-build packages for qevercloud and tested it with nixnote2 (with modifications of source code to use system qevercloud library and header files).

modified nixnote2: https://github.com/hosiet/nixnote2 , in debian branch with patches in debian/patches directory.
modified qevercloud: https://github.com/hosiet/qevercloud, in debian branch with git submodule and patches, etc.
pre-built qevercloud deb packages: http://debomatic-amd64.debian.net/distribution#unstable/qevercloud/3.0.2+ds-1/ , scroll to the bottom of the page and you will find various download links.

Chdir into nixnote2 and the build can be triggered by debuild -us -uc -i -b.

The result buildlog is attached. Error info starts at line 2848, both about the source code of nixnote2 and the qevercloud headers. For example:

release/nixnote.o: In function `NixNote::synchronize()':
./nixnote.cpp:1380: undefined reference to `qevercloud::EvernoteOAuthDialog::EvernoteOAuthDialog(QString, QString, QString, QWidget*)'
/usr/include/qt5qevercloud/EverCloudException.h:110: undefined reference to `vtable for qevercloud::EvernoteException'
/usr/include/qt5qevercloud/exceptions.h:27: undefined reference to `vtable for qevercloud::ThriftException'

So there must be something wrong :-( Do you have any idea?

The original build log:
nixnote2_testbuild.txt

Boyuan Yang · Answer 1 · Wed Sep 07 2016 12:42:50 GMT+0800 (China Standard Time)

update: This should not be packaging problem. The same problem happens when using bundled files in generated/ directory.

Dmitry Ivanov · Answer 2 · Wed Sep 07 2016 12:46:41 GMT+0800 (China Standard Time)

"undefined reference to vtable" usually occurs when there's some pure virtual method left unimplemented. Or its implementation didn't make it into the final binary.

First I'd like you to see what happens if you try to link nixnote2 to the vanilla QEverCloud library, without Debian patches at all, to understand whether the bug belongs to the vanilla version or to patches.

Dmitry Ivanov · Answer 3 · Wed Sep 07 2016 12:50:27 GMT+0800 (China Standard Time)

FWIW, I regulaly test the changes to QEverCloud via a NotePoster test app + one more app which I haven't open-sourced yet. I've never seen such symbols problem there.

Boyuan Yang · Answer 4 · Wed Sep 07 2016 13:00:25 GMT+0800 (China Standard Time)

It looks like only a few symbols could not be found (related to OAuth and
Exception). If the example project don't use them them maybe there won't be
errors. I'm not sure. Will dig into it later.

在 2016年9月7日週三 12:50，Dmitry Ivanov notifications@github.com 寫道：

FWIW, I regulaly test the changes to QEverCloud via a NotePoster
https://github.com/d1vanov/QEverCloud-example-NotePoster test app + one
more app which I haven't open-sourced yet. I've never seen such symbols
problem there.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#6 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIDYFkOjhyGIQXkgvDlNBxwcX_IbMsShks5qnkKUgaJpZM4J2hMe
.

Dmitry Ivanov · Answer 5 · Wed Sep 07 2016 13:02:17 GMT+0800 (China Standard Time)

The example project does use OAuth, otherwise it won't be able to communicate with Evernote servers 😃 It doesn't use the exceptions directly though.

Dmitry Ivanov · Answer 6 · Wed Sep 07 2016 15:24:00 GMT+0800 (China Standard Time)

In the build log you attached I don't actually see the linkage to QEverCloud library at all:
-L/usr/X11R6/lib64 -lhunspell -lcurl -L/usr/lib -lpoppler-qt5 -g -rdynamic -lQt5PrintSupport -lQt5WebKitWidgets -lQt5Widgets -lQt5WebKit -lQt5Gui -lQt5Network -lQt5Sql -lQt5Xml -lQt5DBus -lQt5Core -lGL -lpthread
I'd say that should explain the undefined references you have. Try to add the linkage to QEverCloud library into the NixNote.pro file:

greaterThan(QT_MAJOR_VERSION, 4) {
    QT       += core gui widgets printsupport webkit webkitwidgets sql network xml dbus
    DEFINES += QT_DISABLE_DEPRECATED_BEFORE=0
    INCLUDEPATH += /usr/include/poppler/qt5
    LIBS +=    -lhunspell -lcurl \
               -lpthread -L/usr/lib -lpoppler-qt5 -lqt5qevercloud -g -rdynamic
}

equals(QT_MAJOR_VERSION, 4) {
    QT       += core gui webkit sql network xml
    INCLUDEPATH += /usr/include/poppler/qt4
    LIBS +=    -lhunspell -lcurl \
               -lpthread -L/usr/lib -lpoppler-qt4 -lqt4qevercloud -g -rdynamic
}

Boyuan Yang · Answer 7 · Wed Sep 07 2016 15:56:14 GMT+0800 (China Standard Time)

Yes you are right, silly me. Things are working well now.

在 2016年9月7日週三 15:24，Dmitry Ivanov notifications@github.com 寫道：

In the build log you attached I don't actually see the linkage to
QEverCloud library at all:
-L/usr/X11R6/lib64 -lhunspell -lcurl -L/usr/lib -lpoppler-qt5 -g -rdynamic
-lQt5PrintSupport -lQt5WebKitWidgets -lQt5Widgets -lQt5WebKit -lQt5Gui
-lQt5Network -lQt5Sql -lQt5Xml -lQt5DBus -lQt5Core -lGL -lpthread
I'd say that should explain the undefined references you have. Try to add
the linkage to QEverCloud library into the NixNote.pro
https://github.com/hosiet/nixnote2/blob/debian/NixNote2.pro file:

greaterThan(QT_MAJOR_VERSION, 4) {
QT += core gui widgets printsupport webkit webkitwidgets sql network xml dbus
DEFINES += QT_DISABLE_DEPRECATED_BEFORE=0
INCLUDEPATH += /usr/include/poppler/qt5
LIBS += -lhunspell -lcurl
-lpthread -L/usr/lib -lpoppler-qt5 -lqt5qevercloud -g -rdynamic
}

equals(QT_MAJOR_VERSION, 4) {
QT += core gui webkit sql network xml
INCLUDEPATH += /usr/include/poppler/qt4
LIBS += -lhunspell -lcurl
-lpthread -L/usr/lib -lpoppler-qt4 -lqt4qevercloud -g -rdynamic
}

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#6 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIDYFlDNHLNyXFQuJEEs9-ZHa6dOW1-qks5qnmaQgaJpZM4J2hMe
.

Dmitry Ivanov · Answer 8 · Wed Sep 07 2016 16:01:00 GMT+0800 (China Standard Time)

Great, closing this one then 😃