undefined symbol: _ZN11CommHistory10EventModel20enableContactChangesEb

gregoa commented 9 years ago

I just tried to import a database created with glogarchive on my N900. Output on the commandline:

$ jolla-morse
[D] QWaylandEglClientBufferIntegration::QWaylandEglClientBufferIntegration:62 - Using Wayland-EGL 
[W] QQmlImportDatabase::registerPluginTypes:1875 - Module 'Sailfish.Silica' does not contain a module identifier directive - it cannot be protected from external registrations.
[D] DeclarativeCoverWindow::DeclarativeCoverWindow:63 - DeclarativeCoverWindow: I have a default alpha buffer

[D] onClicked:86 - Proceed. /home/nemo/Documents/liveDB-20141222-135637.csv
[D] CSVHandler::parseFile:167 - CSVHandler::parseFile() called, registered QList<GlogEvent*> 
[D] CSVWorker::run:38 - CSVWorker::run( "/home/nemo/Documents/liveDB-20141222-135637.csv" ) 
[D] CSVWorker::run:44 - 637454 
[D] CSVWorker::run:52 - OK! 
[D] CSVWorker::run:59 - 3636 
[D] CSVHandler::insertGlogEvents:199 - Inserting glogevents: 3636 
[D] InsertWorker::setGrouped:47 - Getting groups 
[D] InsertWorker::handleGroups:145 - Got groups: 139 
[W] CommHistoryDatabase::open:301 - Opened commhistory database: "/home/nemo/.local/share/commhistory/commhistory.db" 
jolla-morse: symbol lookup error: jolla-morse: undefined symbol: _ZN11CommHistory10EventModel20enableContactChangesEb

Am I missing a package, or is the exported CSV broken, or - and that's my guess - does jolla-morse need a rebuild against the current version of libcommhistory-qt5 or something?

System:

$ rpm -q jolla-morse
jolla-morse-0.0.5-1.armv7hl
$ rpm -q libcommhistory-qt5
libcommhistory-qt5-1.7.18.1-1.21.2.armv7hl
$ cat /etc/sailfish-release 
NAME=SailfishOS
ID=sailfishos
VERSION="1.1.1.27 (Vaarainjärvi) (armv7hl)"
[..]

Thanks for writing morse!

Cheers, gregor

gregoa commented 9 years ago

Ok, I got the SDK working, and indeed a rebuild is enough. Could you please upload a -2 version to openrepos.net? - I guess the yml/spec should include a buildtime and runtime dependency on libcommhistory-qt5{,-devel}, besides that I locally bumped the version:

diff --git a/qml/pages/about.qml b/qml/pages/about.qml
index 7b881e8..1ba29e8 100644
--- a/qml/pages/about.qml
+++ b/qml/pages/about.qml
@@ -54,7 +54,7 @@ Page {
                 width: parent.width
             }
             Label {
-                text: "0.0.5-1\n\nCopyright 2013 Markus Törnqvist\n<mjt@nysv.org>\n\nhttps://github.com/mjtorn/jolla-morse/"
+                text: "0.0.5-2\n\nCopyright 2013 Markus Törnqvist\n<mjt@nysv.org>\n\nhttps://github.com/mjtorn/jolla-morse/"
                 horizontalAlignment: Text.AlignHCenter
                 elide: Text.ElideMiddle
                 width: parent.width
diff --git a/rpm/jolla-morse.spec b/rpm/jolla-morse.spec
index c9613d9..6d85771 100644
--- a/rpm/jolla-morse.spec
+++ b/rpm/jolla-morse.spec
@@ -14,7 +14,7 @@ Name:       jolla-morse
 %{?qtc_builddir:%define _builddir %qtc_builddir}
 Summary:    Jolla Morse
 Version:    0.0.5
-Release:    1
+Release:    2
 Group:      Qt/Qt
 License:    LICENSE
 URL:        http://example.org/
diff --git a/rpm/jolla-morse.yaml b/rpm/jolla-morse.yaml
index 41fa201..586f82d 100644
--- a/rpm/jolla-morse.yaml
+++ b/rpm/jolla-morse.yaml
@@ -1,7 +1,7 @@
 Name: jolla-morse
 Summary: Jolla Morse
-Version: 0.1
-Release: 1
+Version: 0.0.5
+Release: 2
 Group: Qt/Qt
 URL: http://example.org/
 License: LICENSE

Cheers, gregor

mjtorn commented 9 years ago

Hi!

Not sure I understand what's happening (real busy atm, sorry!). When you say you also added the version info, that's fine, but I don't see the dependency for libcommhistory. OTOH it should be shipped with the phone anyway, right?

Is your patch only to update some RPM metadata that I missed when pushing?

Again, sorry, got a lot on my plate right now and can't focus on this probably until the holidays are over, so any extra info helps :)

Thanks!

gregoa commented 9 years ago

On Mon, 22 Dec 2014 21:38:20 -0800, Markus Törnqvist wrote:

Hi!

Hi Markus,

thanks for the quick reply.

Not sure I understand what's happening (real busy atm, sorry!). When you say you also added the version info, that's fine, but I don't see the dependency for libcommhistory. OTOH it should be shipped with the phone anyway, right? Is your patch only to update some RPM metadata that I missed when pushing?

Sorry for being unclear. Let's try again :)

The main message is: jolla-morse needs a rebuild against the current libcommhistory / the current SDK, and an upload to openrepos.net.
When I rebuilt it locally, I bumped the Release to 2 to get a newer RPM version; and noticed that the Version in the .yaml is probably wrong.
As a purely aesthetic side remark I mentioned that the libcommhistory build and runtime history should probably be also mentioned in the spec/yaml; this is not in the patch because after getting the SDK running I didn't want to learn the details of rpm packaging :)

Again, sorry, got a lot on my plate right now and can't focus on this probably until the holidays are over, so any extra info helps :)

No worries. When you find the time, please just rebuild + upload the package if possible, and feel free to ignore my side remarks about the metadata :)

Cheers, gregor

.''. Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06 : :' : Debian GNU/Linux user, admin, and developer - http://www.debian.org/ . ' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe - NP: Funny van Dannen: Nur du nicht

mjtorn / jolla-morse

undefined symbol: _ZN11CommHistory10EventModel20enableContactChangesEb #4