Bug 211671

Summary:	Wrongly complains not compiled with exiv2 library
Product:	[Applications] kphotoalbum	Reporter:	Olivier Berger <oberger>
Component:	general	Assignee:	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:
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 ***