Bug 211671

Summary: Wrongly complains not compiled with exiv2 library
Product: [Applications] kphotoalbum Reporter: Olivier Berger <oberger>
Component: generalAssignee: KPhotoAlbum Bugs <kpabugs>
Status: RESOLVED DUPLICATE    
Severity: normal CC: oberger
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Fix the wrong #ifdef

Description Olivier Berger 2009-10-24 18:11:22 UTC
Version:           4.0.2 (using KDE 4.3.1)
OS:                Linux
Installed from:    Debian testing/unstable Packages

Upon startup, a dialog complains that exiv2 library wasn't compiled with, which is not true.

I think the #ifdef is wrong in the code.

See more details in http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=552236 :

Upon startup, kphotoalbum prompts me with a dialog saying that "Images/Videos Are Not Sorted" : "... you don't have EXIF support available... It is strongly recommended to recompile KPhotoAlbum with the exiv2 library...".

Looking at the code, in XMLDB/FileReader.cpp, it seems it's the only place in the sources where a #ifdef HASEXIV2 is used, whereas all the other sources seem to use #ifdef HAVE_EXIV2

I suspect you may be interested in having a look at http://websvn.kde.org/trunk/extragear/graphics/kphotoalbum/XMLDB/FileReader.cpp?r1=698273&r2=698274&
Comment 1 Olivier Berger 2009-10-25 17:07:47 UTC
Created attachment 37823 [details]
Fix the wrong #ifdef
Comment 2 Olivier Berger 2009-10-25 17:08:30 UTC
The proposed patch seems to fix the problem here.
Comment 3 Olivier Berger 2009-10-26 10:23:54 UTC

*** This bug has been marked as a duplicate of bug 211907 ***