Summary: | Invalid error message for SQLite version check after an update of SQLite | ||
---|---|---|---|
Product: | [Applications] skrooge | Reporter: | Benjamin Robin <benjarobin+kde> |
Component: | general | Assignee: | Stephane MANKOWSKI <stephane> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | carlos.bessa, stephane |
Priority: | NOR | ||
Version: | 1.12 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/skrooge/dd18ae016fb3eacf4894f30fd0dc15e1187cc53e | Version Fixed In: |
Description
Benjamin Robin
2015-05-15 14:21:33 UTC
I understand the point but I added this check because: - Some users have a Qt built without system-sqlite. - The system SQLite version is different than the Qt sqlite version. - This causes a crash of Skrooge. Because Skrooge needs to call specific SQLite functions not callable from Qt, I must be sure that the Qt and the SQLite versions are aligned. I can not call dynamically SQLIte because, in case of non-alignment, this will cause the crash. This is why I did the check statically (Compare the version of Qt version and SQLite version AT BUILD time). In you case, you have a false positive, but this is better than having a crash. In conclusion: I don't know how to do better. Do you have an idea? Yeah! i found. I will do the same but compare the MAJOR version only !!!!!!!!!!!!!!! Are you sure that using the symbol sqlite3_version will crash the application ? I don't see how this can happen. The other solution that I can found was to create an executable that print the version of the SQLite system lib... Nevertheless comparing only the major version should be enough... Git commit dd18ae016fb3eacf4894f30fd0dc15e1187cc53e by Stephane Mankowski. Committed on 15/05/2015 at 19:11. Pushed by smankowski into branch 'kf5'. Invalid error message for SQLite version check after an update of SQLite M +1 -0 CHANGELOG M +2 -2 skgbasemodeler/skgdocument.cpp M +13 -0 skgbasemodeler/skgservices.cpp M +7 -0 skgbasemodeler/skgservices.h M +5 -0 tests/skgbasemodelertest/skgtestbase.cpp http://commits.kde.org/skrooge/dd18ae016fb3eacf4894f30fd0dc15e1187cc53e Hi, will this also be corrected for the KDE4 version? I have not yet moved to the new Plasma, and cannot start Skrooge due to this problem in my current install. regards, Carlos Hi, Yes, this has been corrected on the KDE4 version too: http://quickgit.kde.org/?p=skrooge.git&a=commit&h=483f7409e06870780654c17b6f335addc378e99e Hi, Will we (users) get it only when a new version is released or if I ask the packager of my distro to build a new version the patch will already be included? (sorry to ask, I don't really get how this works yet). Btw, thank you for such a great program. Really very intuitive to use. Carlos Hi, I will publish a new 1.12.1 version including this correction. Regards. |