|Summary:||failed build against libgphoto2-2.5.0 [patch]|
|Product:||[Applications] digikam||Reporter:||nucleo <nucleo>|
|Component:||Import-Gphoto2||Assignee:||Digikam Developers <digikam-bugs-null>|
|Severity:||normal||CC:||benv-kde.org, caulier.gilles, dodonvictor, johu, kevin.kofler, marcus, melko, rdieter, wazery|
|Latest Commit:||commit b224133f53e8428da7162c604374c7f73283d215||Version Fixed In:||2.9.0|
updated and tested libgphoto 2.5/2.4 patch
Description nucleo 2012-07-12 18:15:06 UTC
Comment 1 Rex Dieter 2012-07-12 18:16:47 UTC
Comment 2 nucleo 2012-07-12 18:25:21 UTC
Note that KDE 4.8.97 used in thi build.
Comment 3 Wouter de Geus 2012-07-17 11:42:35 UTC
Just tried this on slackware64-current with KDE 4.8.4 (and libgphoto2-2.5.0) with the same result. Since this new libgphoto2 version finally gets rid of the silly 1024 files per directory limit I'd say it's a very welcome upgrade.
Comment 4 Marcus Meissner 2012-07-24 15:20:40 UTC
Created attachment 72725 [details] 0001-support-both-libgphoto2-2.5.0-and-2.4.0.patch please try this patch against "core" part of digikam. it tries to handle both 2.4 and 2.5.
Comment 5 Marcus Meissner 2012-07-24 15:21:36 UTC
(untested yet... will do this in some minutes)
Comment 6 nucleo 2012-07-25 00:05:40 UTC
Patch from Comment 4 fixes build but not tested how digikam works. http://kojipkgs.fedoraproject.org//packages/digikam/2.7.0/5.fc18/data/logs/i686/build.log
Comment 7 Marcus Meissner 2012-07-27 15:18:31 UTC
it builds, but there is some GPPortInfo lifetime handling off, i need to do some more fixes there
Comment 8 Marcus Meissner 2012-08-15 09:42:21 UTC
Created attachment 73176 [details] updated and tested libgphoto 2.5/2.4 patch this patch should work with both 2.4 and 2.5. i had to rewrite findConnectedUSBCamera a bit ;)
Comment 9 Marcus Meissner 2012-08-15 12:02:50 UTC
(patch is against master ... if any other branch is required i can rebase it there too if necessary ;)
Comment 10 caulier.gilles 2012-08-15 14:48:57 UTC
Thanks Marcus. git/master is for production. next 2.9.0 will be last stable release without any major changes. Your current patch will be applied into. There is another 3.0.0 branch where similar patch must be applied, and where we working in-deep currently. Look here for details : http://community.kde.org/Digikam/GSoC2012#Repositories_Branching_and_Dates If you can provide a patch for this branch, it will be great. Best Gilles Caulier
Comment 11 Marcus Meissner 2012-08-20 08:26:41 UTC
applies as-is to 3.0.0 branch, but I fail to compile the branch, error is this: /home/marcus/projects/digikam-sc/core/digikam/utils/componentsinfo.h:81:71: error: ‘class KIPI::PluginLoader’ has no member named ‘kipiPluginsVersion’
Comment 12 Marcus Meissner 2012-08-20 08:27:43 UTC
taking that back, it needs utilities/importui/backend/gpcamera.cpp not seen ...
Comment 13 Victor Dodon 2012-08-20 08:38:22 UTC
(In reply to comment #11) > applies as-is to 3.0.0 branch, but I fail to compile the branch, error is > this: > > /home/marcus/projects/digikam-sc/core/digikam/utils/componentsinfo.h:81:71: > error: ‘class KIPI::PluginLoader’ has no member named ‘kipiPluginsVersion’ digiKam from development/3.0.0 branch requires libkipi 2.0.0 which you can get from development/2.0.0 branch from libkipi repo. Victor.
Comment 14 caulier.gilles 2012-08-20 13:32:31 UTC
Islam, I CC you about 3.0.0 branch where Marcus try to apply patch about Gphoto 2.5.0 support... Gilles Caulier
Comment 15 Islam Wazery 2012-08-21 02:07:49 UTC
I applied it to development/3.0.0 signed with Marcus Meissner name.
Comment 16 caulier.gilles 2012-08-21 07:24:37 UTC
Git commit 1e3fe5480e5311d4d564da79c6eee7ecf7cac774 by Gilles Caulier. Committed on 21/08/2012 at 09:23. Pushed by cgilles into branch 'master'. apply patch from Marcus Meissner to support libgphoto 2.5.0 API M +23 -11 CMakeLists.txt M +5 -1 NEWS M +3 -0 digikam/utils/config-digikam.h.cmake M +60 -62 utilities/cameragui/devices/gpcamera.cpp http://commits.kde.org/digikam/1e3fe5480e5311d4d564da79c6eee7ecf7cac774
Comment 17 Marcus Meissner 2012-08-21 11:29:43 UTC
Islam ... one question, is "utiltiies/importui/backend/gpcamera.cpp" or utilities/cameragui/devices/gpcamera.cpp active? or both? I see you merged it into importui/backend/gpcamera.cpp, which is probably the newer one?
Comment 18 caulier.gilles 2012-08-21 11:45:06 UTC
Marcus, Islam, who work to revamp digiKam camera GUI for 3.0.0 through GoSC 2012 project, as just renamed cameragui dir to importui... gpcamera.cpp files from 2.x and 3.x are the same in fact... Gilles Caulier